RFR: 8327963: C2: fix construction of memory graph around Initialize node to prevent incorrect execution if allocation is removed [v8]

Tue Oct 21 13:51:52 UTC 2025

On Mon, 29 Sep 2025 08:40:01 GMT, Roland Westrelin <roland at openjdk.org> wrote:

>>> That sounds good to me, thank you for enforcing this Roland! I will re-run testing and have a new look at the changeset within the next days.
>> 
>> Test results of b701d03ed335286587c4d2539dde715b091d30bd on top of jdk-26+14 look good. Will have a look at the code within the next days.
>
> @robcasloz Thanks for the patches. I added them.

Hi @rwestrel, could you please have a look at the merge conflicts of this PR so that we can progress further with the review of this work?

> Hi @rwestrel, could you please have a look at the merge conflicts of this PR so that we can progress further with the review of this work?

The conflict is caused by the integration of [JDK-8360031](https://bugs.openjdk.org/browse/JDK-8360031), which relaxes the assertion in https://github.com/openjdk/jdk/blob/430041d366ddf450c2480c81608dde980dfa6d41/src/hotspot/share/opto/memnode.cpp#L4232 which is also touched by this changeset. Is the current assertion in mainline (after JDK-8360031) still valid in the context of this changeset?

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

PR Comment: https://git.openjdk.org/jdk/pull/24570#issuecomment-3423349747
PR Comment: https://git.openjdk.org/jdk/pull/24570#issuecomment-3425213782