On Mon, 22 Jun 2026 07:19:51 GMT, Aleksey Shipilev <[email protected]> wrote:

>> Shawn Emery has updated the pull request incrementally with one additional 
>> commit since the last revision:
>> 
>>   Update based on adinn's comments
>
> src/hotspot/cpu/aarch64/stubGenerator_aarch64.cpp line 7721:
> 
>> 7719:     const int32_t columns    = limbs * 2;
>> 7720:     const uint64_t mask      = -1UL >> rem;
>> 7721:     const uint64_t CARRY_ADD = 1UL << (bpl - 1);
> 
> On MSVC, `UL` is `unsigned long`, which is apparently only 32-bit long. So 
> shifting `<< 51` breaks.

Similar issue with `mask` above, I think.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/31409#discussion_r3450420704

Reply via email to