[lworld] RFR: 8335256: [lworld] C2: Remove larval InlineTypeNode [v5]
Tobias Hartmann
thartmann at openjdk.org
Mon May 12 11:58:06 UTC 2025
On Fri, 9 May 2025 18:10:40 GMT, Quan Anh Mai <qamai at openjdk.org> wrote:
>> I'm not sure what effect it will have on mainline. For this patch, without this, `TestNullableInlineTypes::test85` tries to enter EA with 10k nodes, which it bails out. Trying to eliminate all trivially removable allocations reduces the number to 2k, allows EA to complete successfully. I think it may be more significant on Valhalla because in general we seem to emit a lot more nodes with value objects.
>
> Please let me know what you think about this, perhaps we need to measure the effects on mainline first.
Makes sense. I'd say we put it into Valhalla only for now. We can still consider upstreaming it later.
-------------
PR Review Comment: https://git.openjdk.org/valhalla/pull/1447#discussion_r2084508877
More information about the valhalla-dev
mailing list