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