RFR: 8373134: C2: Min/Max users of Min/Max uses should be enqueued for GVN [v8]
Galder Zamarreño
galder at openjdk.org
Mon Jan 12 08:38:27 UTC 2026
On Mon, 12 Jan 2026 07:29:09 GMT, Tobias Hartmann <thartmann at openjdk.org> wrote:
>> Galder Zamarreño has updated the pull request incrementally with two additional commits since the last revision:
>>
>> - Update test/hotspot/jtreg/compiler/igvn/TestMinMaxIdentity.java
>>
>> Co-authored-by: Emanuel Peter <emanuel.peter at oracle.com>
>> - Fix style
>
> Do we risk hitting [JDK-8374896](https://bugs.openjdk.org/browse/JDK-8374896) now that the verification will be enabled by this PR?
@TobiHartmann and I just had a chat and he is right. If the changes in `PhaseIterGVN::verify_Identity_for` are applied, we will hit [JDK-8374896](https://bugs.openjdk.org/browse/JDK-8374896). So, I will revert that part of the change. Note that the IR test does not rely on `VerifyIterativeGVN` being enabled.
-------------
PR Comment: https://git.openjdk.org/jdk/pull/28895#issuecomment-3737402164
More information about the hotspot-compiler-dev
mailing list