RFR: 8370502: C2: segfault while adding node to IGVN worklist [v4]

[Emanuel Peter]

On Fri, 28 Nov 2025 10:16:06 GMT, Manuel Hässig <mhaessig at openjdk.org> wrote:

>> Kerem Kat has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   fix rename
>
> test/hotspot/jtreg/compiler/c2/TestUnlockNodeNullMemprof.java line 37:
> 
>> 35:     public static void main(String[] args) {
>> 36:         int[] a = new int[0]; // test valid only when size is 0.
>> 37:         for (int i = 0; i < Integer.valueOf(10000); i++)
> 
> Suggestion:
> 
>         for (int i = 0; i < Integer.valueOf(10000); i++) // test only valid with boxed loop limit
> 
> This surprised me, so I would appreciate the comment, but feel free to leave it.

Is that maybe because it delays the discovery of the constant until CCP?

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

PR Review Comment: https://git.openjdk.org/jdk/pull/28432#discussion_r2571167713