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