Integrated: 8350471: Unhandled compilation bailout in GraphKit::builtin_throw

[Manuel Hässig]

On Wed, 26 Mar 2025 08:55:10 GMT, Manuel Hässig <duke at openjdk.org> wrote:

> # Issue Summary
> 
> When creating a builtin exception node, a stress test decided to bail out as if the allocation of the builtin exception objects had failed. Since these are preallocated at VM creation, the test failure is a false positive. 
> 
> # Change Rationale
> 
> `GraphKit::builtin_throw()` features a bailout check after getting an appropriate exception object. However, up to that point, the execution in `builtin_throw()` cannot fail. In particular, there can be no failure to allocate the exception because these are all preallocated during `Threads::create_vm()` startup in `universe_post_init()` and `Threads:initialize_java_lang_classes()`. Further, none of the three callers handles a possible bailout in `builtin_throw()`. Hence, this PR removes the bailout check responsible for the test failure
> 
> # Testing
> 
>  - [Github Actions](https://github.com/mhaessig/jdk/actions/runs/14078715650)
>  - tier1 through tier3 and Oracle internal testing

This pull request has now been integrated.

Changeset: 8ef78323
Author:    Manuel Hässig <manuel.hassig at oracle.com>
Committer: Tobias Hartmann <thartmann at openjdk.org>
URL:       https://git.openjdk.org/jdk/commit/8ef78323b1177782a645155fda19544fae24c279
Stats:     1 line in 1 file changed: 0 ins; 0 del; 1 mod

8350471: Unhandled compilation bailout in GraphKit::builtin_throw

Reviewed-by: thartmann, chagedorn, kvn

-------------

PR: https://git.openjdk.org/jdk/pull/24243