RFR: 8370794: C2 SuperWord: Long/Integer.compareUnsigned return wrong value for EQ/NE in SLP

Hamlin Li mli at openjdk.org
Mon Nov 3 10:38:47 UTC 2025


On Mon, 3 Nov 2025 10:27:40 GMT, Ludovic Henry <luhenry at openjdk.org> wrote:

>> Hi,
>> Can you help to review this patch?
>> 
>> [JDK-8370481](https://bugs.openjdk.org/browse/JDK-8370481) introduces this regression for unsigned I/L EQ/NE in SLP.
>> 
>> ====================
>> 
>> In [JDK-8370481](https://bugs.openjdk.org/browse/JDK-8370481), we fixed an issue related to transformation from (Bool + CmpU + CMove) to (VectorMaskCmp + VectorBlend), and added tests for unsigned ones.
>> As discussion in [1], we should also add more tests for transformation from (Bool + Cmp + CMove) to (VectorMaskCmp + VectorBlend) for the signed ones.
>> 
>> [1] https://github.com/openjdk/jdk/pull/27942#discussion_r2468750039
>> 
>> Thanks!
>> 
>> Tests running...
>
> Marked as reviewed by luhenry (Committer).

@luhenry Thank you for reviewing!

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

PR Comment: https://git.openjdk.org/jdk/pull/28047#issuecomment-3479868473


More information about the hotspot-compiler-dev mailing list