On Fri, 11 Nov 2022 01:26:40 GMT, Vladimir Ivanov <[email protected]> wrote:
>> Volodymyr Paprotski has updated the pull request incrementally with one
>> additional commit since the last revision:
>>
>> live review with Sandhya
>
> src/hotspot/cpu/x86/macroAssembler_x86.hpp line 733:
>
>> 731: void andptr(Register src1, Register src2) { LP64_ONLY(andq(src1,
>> src2)) NOT_LP64(andl(src1, src2)) ; }
>> 732:
>> 733: #ifdef _LP64
>
> Why is it x64-specific?
I believe its needed.
TLDR.. Couple of check ins ago, I broke the 32-bit build, and that was the
'easy' fix..
> src/hotspot/cpu/x86/stubGenerator_x86_64_poly.cpp line 161:
>
>> 159: const XMMRegister P2_H = xmm5;
>> 160: const XMMRegister TMP1 = xmm6;
>> 161: const Register polyCP = r13;
>
> Could be renamed to `rscratch` (or `tmp`) since it doesn't hold constant base
> address anymore.
done
-------------
PR: https://git.openjdk.org/jdk/pull/10582
