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