On Fri, 22 May 2026 13:06:55 GMT, Ferenc Rakoczi <[email protected]> wrote:

>> src/hotspot/cpu/aarch64/stubGenerator_aarch64.cpp line 8377:
>> 
>>> 8375:       VSeq<4> b_vec(20);
>>> 8376: 
>>> 8377:       __ ldr(a9, Address(aLimbs, 64));
>> 
>> It makes things more obvious to a maintainer and avoids mistakes should 
>> anything need patching if you use 8 * BytesPerLong here instead of 64 and 
>> likewise for all the other hard-wired Address offsets used throughout this 
>> method. That way the offsets directly relate to the vector and GPR counts 
>> mentioned in the comments.
>
> Accepted.

But not yet changed?

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

PR Review Comment: https://git.openjdk.org/jdk/pull/30941#discussion_r3413407998

Reply via email to