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

Quan Anh Mai qamai at openjdk.org
Fri May 9 17:55:47 UTC 2025


On Fri, 9 May 2025 09:36:50 GMT, Tobias Hartmann <thartmann at openjdk.org> wrote:

>> 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`.

Yes, I think it is cleaner that way instead of trailing each `mex.eliminate_macro_nodes` with an `igvn.optimize()`

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

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


More information about the valhalla-dev mailing list