[lworld] RFR: 8378672: [lworld] Consolidate a factory to create a GraphKit from a SafePointNode [v2]
Quan Anh Mai
qamai at openjdk.org
Wed Feb 25 14:09:55 UTC 2026
On Wed, 25 Feb 2026 13:41:26 GMT, Tobias Hartmann <thartmann at openjdk.org> wrote:
>> It is because originally, the 2 `If`s are too far from each other for `IfNode::Ideal` to fold. This PR induces a random change in the IGVN order which prevents the dominated `If` from being enqueued and folded in the first IGVN invocation. As a result, it is only processed later.
>
> Thanks for the explanation - makes sense. Could you please file a tracking RFE in Valhalla to revert this once JDK-8375442 got merged?
I faced another failure at `vectorization/TestSubwordTruncation.java`, so I decided to minimize the unintended change by only adding to igvn worklist if the current `_gvn` is an `IterGVN`.
-------------
PR Review Comment: https://git.openjdk.org/valhalla/pull/2163#discussion_r2853253101
More information about the valhalla-dev
mailing list