RFR: 8373555: C2: Optimize redundant input calculations for sign comparisons [v2]

Hannes Greule hgreule at openjdk.org
Thu Jan 8 10:55:18 UTC 2026


On Thu, 8 Jan 2026 10:00:19 GMT, Manuel Hässig <mhaessig at openjdk.org> wrote:

>> Hannes Greule has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   update copyright year
>
> test/hotspot/jtreg/compiler/c2/gvn/BoolNodeSimplifySignInvariantTests.java line 711:
> 
>> 709:     }
>> 710: 
>> 711:     record IntRange(int lo, int hi) {
> 
> Do you think this would be useful beyond this test? If so, it would be good to add it to the test library.

Yes, there are quite a few tests with the same code (sometimes `IntRange` and `LongRange`, sometimes just `Range`). Do you want me to extract the record in this PR so the other existing tests can be cleaned up afterwards?

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

PR Review Comment: https://git.openjdk.org/jdk/pull/28782#discussion_r2671870388


More information about the hotspot-compiler-dev mailing list