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