RFR: 8373508: C2: sinking CreateEx out of loop breaks the graph [v2]

[Roland Westrelin]

On Wed, 17 Dec 2025 08:23:21 GMT, Christian Hagedorn <chagedorn at openjdk.org> wrote:

>> Roland Westrelin has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Update test/hotspot/jtreg/compiler/loopopts/TestCreateExSunkOutOfLoop.java
>>   
>>   Co-authored-by: Christian Hagedorn <christian.hagedorn at oracle.com>
>
> test/hotspot/jtreg/compiler/loopopts/TestCreateExSunkOutOfLoop.java line 30:
> 
>> 28:  * @library /test/lib
>> 29:  * @run main/othervm -Xbatch ${test.main.class}
>> 30:  * @run main ${test.main.class}
> 
> Since this test runs for 4s at least, I'm not sure if it's worth to have an Xbatch and non-Xbatch version. Does it trigger with both?

It does. Which would you keep then?

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

PR Review Comment: https://git.openjdk.org/jdk/pull/28842#discussion_r2627280788