8350471: Unhandled compilation bailout in GraphKit::builtin_throw

mhaessig · TobiHartmann · commit 8ef78323b117 · 2025-03-28T08:53:37.000Z
Reviewed-by: thartmann, chagedorn, kvn
diff --git a/src/hotspot/share/opto/graphKit.cpp b/src/hotspot/share/opto/graphKit.cpp
@@ -587,7 +587,7 @@ void GraphKit::builtin_throw(Deoptimization::DeoptReason reason) {
     default:
       break;
     }
-    if (failing()) { stop(); return; }  // exception allocation might fail
+    // If we have a preconstructed exception object, use it.
     if (ex_obj != nullptr) {
       if (env()->jvmti_can_post_on_exceptions()) {
         // check if we must post exception events, take uncommon trap if so

Original file line number	Diff line number	Diff line change
`@@ -587,7 +587,7 @@ void GraphKit::builtin_throw(Deoptimization::DeoptReason reason) {`
`587`	`587`	`default:`
`588`	`588`	`break;`
`589`	`589`	`}`
`590`		`- if (failing()) { stop(); return; } // exception allocation might fail`
	`590`	`+ // If we have a preconstructed exception object, use it.`
`591`	`591`	`if (ex_obj != nullptr) {`
`592`	`592`	`if (env()->jvmti_can_post_on_exceptions()) {`
`593`	`593`	`// check if we must post exception events, take uncommon trap if so`