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

Roland Westrelin roland at openjdk.org
Mon Sep 22 13:37:55 UTC 2025


On Thu, 11 Sep 2025 07:48:10 GMT, Roberto Castañeda Lozano <rcastanedalo at openjdk.org> wrote:

>>> @rose00 @robcasloz I updated the change with a new way to avoid redundant projections. At matching time, before a `NarrowMemProj` is matched into a `MachProj`, new logic checks whether a `MachProj` already exists. That guarantees that no redundant `MachProj` are ever added. It also performs the new normalization at a major cut-point. What do you think?
>> 
>> 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.
>
>> 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 review. New commit addresses most of your comments.

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

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


More information about the hotspot-compiler-dev mailing list