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