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

Manuel Hässig mhaessig at openjdk.org
Thu Jan 8 11:11:47 UTC 2026


On Thu, 8 Jan 2026 10:51:55 GMT, Hannes Greule <hgreule at openjdk.org> wrote:

>> 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?

Moving it to the test library and cleaning up in a follow-up RFE sounds good

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

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


More information about the hotspot-compiler-dev mailing list