On Wed, 25 Feb 2026 13:41:26 GMT, Tobias Hartmann <[email protected]> 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

Reply via email to