RFR: 8370481: C2 SuperWord: Long/Integer.compareUnsigned return wrong value in SLP [v4]
Emanuel Peter
epeter at openjdk.org
Thu Oct 23 15:23:21 UTC 2025
On Thu, 23 Oct 2025 15:17:25 GMT, Hamlin Li <mli at openjdk.org> wrote:
>> src/hotspot/share/opto/superword.cpp line 1757:
>>
>>> 1755: case Op_CmpI:
>>> 1756: case Op_CmpL:
>>> 1757: break;
>>
>> We could add a comment why these are ok without any action.
>> Suggestion:
>>
>> // The mask of signed int/long scalar comparisons has the same semantics
>> // as the mask for vector elementwise int/long comparison with VectorMaskCmp.
>> break;
>
> Nice! I can commit your suggestion by simply click the button.
Yes, the benefits of GitHub ;)
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/27942#discussion_r2455580088
More information about the hotspot-compiler-dev
mailing list