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

>> 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.

Good catch!  Fixed.  The windows-aarch64 build is not part of my regression 
tests :(

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

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

Reply via email to