RFR: 8327964: Simplify BigInteger.implMultiplyToLen intrinsic [v2]
Dean Long
dlong at openjdk.org
Tue Mar 19 19:46:20 UTC 2024
On Tue, 19 Mar 2024 19:06:36 GMT, Yudi Zheng <yzheng at openjdk.org> wrote:
>> Moving array construction within BigInteger.implMultiplyToLen intrinsic candidate to its caller simplifies the intrinsic implementation in JIT compiler.
>
> Yudi Zheng has updated the pull request incrementally with one additional commit since the last revision:
>
> address comment.
src/hotspot/cpu/aarch64/stubGenerator_aarch64.cpp line 4670:
> 4668: const Register tmp6 = r15;
> 4669: const Register tmp7 = r16;
> 4670: const Register tmp8 = r17;
It looks like tmp8 is never used. The call to multiply_to_len() below needs to be adjusted.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/18226#discussion_r1530986811
More information about the core-libs-dev
mailing list