[lworld] RFR: 8335256: [lworld] C2: Remove larval InlineTypeNode [v5]

Tobias Hartmann thartmann at openjdk.org
Fri May 9 10:33:17 UTC 2025


On Fri, 9 May 2025 07:27:47 GMT, Tobias Hartmann <thartmann at openjdk.org> wrote:

>> Quan Anh Mai has updated the pull request with a new target base due to a merge or a rebase. The pull request now contains six commits:
>> 
>>  - move progress check
>>  - Merge branch 'lworld' into larvaloop
>>  - fast path for non intrinsics
>>  - fix test failures
>>  - Merge branch 'lworld' into larvaloop
>>  - remove larval InlineTypeNode
>
> src/hotspot/share/opto/compile.cpp line 2900:
> 
>> 2898:         }
>> 2899:         igvn.set_delay_transform(false);
>> 2900:         print_method(PHASE_ITER_GVN_AFTER_ELIMINATION, 2);
> 
> Why did you remove the `igvn.optimize();` here?

Ah, you added it to `eliminate_macro_nodes`.

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

PR Review Comment: https://git.openjdk.org/valhalla/pull/1447#discussion_r2081312701


More information about the valhalla-dev mailing list