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