RFR: 8370481: C2 SuperWord: Long/Integer.compareUnsigned return wrong value in SLP [v5]

Hamlin Li mli at openjdk.org
Fri Oct 24 09:02:20 UTC 2025


On Fri, 24 Oct 2025 07:12:29 GMT, Emanuel Peter <epeter at openjdk.org> wrote:

>> Added tests to cover UI{GE|LT|LE}forF and UL{GE|LT|LE}forD.
>> 
>> Other tests for example UI{GE|LT|LE}forD UL{GE|LT|LE}forF could be added when I work on https://github.com/openjdk/jdk/pull/25336 or https://github.com/openjdk/jdk/pull/25341, as currently they are not vectorized.
>
> If you already have the tests in code, it may be good to just put all tests in now. Of course with adjusted IR rules. That would allow us to verify correctness on all combinations, and backport the tests as well. What do you think?

Sure, added all the combination for unsigned comparison.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/27942#discussion_r2459407325


More information about the hotspot-compiler-dev mailing list