Integrated: 8344393: RISC-V: Remove option UseRVVForBigIntegerShiftIntrinsics
Fei Yang
fyang at openjdk.org
Tue Nov 19 10:23:52 UTC 2024
On Mon, 18 Nov 2024 11:25:15 GMT, Fei Yang <fyang at openjdk.org> wrote:
> Seem that it's more reasonable to control BigInteger shift intrinsics with`UseRVV` option.
> I witnessed performance benefit on BPI-F3 with the RISC-V vector extension. And it's still
> possible to disable these two intrinsics with `-XX:DisableIntrinsic` option if we want.
>
> JMH on BPI-F3 (has vector 1.0) for reference:
>
> Without instrinsic:
> BigIntegers.testLeftShift N/A avgt 15 4083.865 ± 224.139 ns/op
> BigIntegers.testRightShift N/A avgt 15 1745.833 ± 44.855 ns/op
>
> With instrinsic:
> BigIntegers.testLeftShift N/A avgt 15 2243.095 ± 24.821 ns/op
> BigIntegers.testRightShift N/A avgt 15 1558.770 ± 36.191 ns/op
>
>
> Test on linux-riscv64 platform:
> - [x] tier1 (release)
This pull request has now been integrated.
Changeset: 0d666891
Author: Fei Yang <fyang at openjdk.org>
URL: https://git.openjdk.org/jdk/commit/0d66689177b880035e4047399e3e64f461713562
Stats: 10 lines in 6 files changed: 0 ins; 8 del; 2 mod
8344393: RISC-V: Remove option UseRVVForBigIntegerShiftIntrinsics
Reviewed-by: mli, fjiang
-------------
PR: https://git.openjdk.org/jdk/pull/22202
More information about the hotspot-dev
mailing list