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

Emanuel Peter epeter at openjdk.org
Fri Oct 24 07:15:36 UTC 2025


On Thu, 23 Oct 2025 19:39:16 GMT, Hamlin Li <mli at openjdk.org> wrote:

>> Make sense, I'll add more tests.
>
> 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?

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

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


More information about the hotspot-compiler-dev mailing list