RFR: 8344393: RISC-V: Remove option UseRVVForBigIntegerShiftIntrinsics [v2]

Fei Yang fyang at openjdk.org
Tue Nov 19 00:24:14 UTC 2024


> 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

Fei Yang has updated the pull request incrementally with one additional commit since the last revision:

  Remove jvmci use cases

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

Changes:
  - all: https://git.openjdk.org/jdk/pull/22202/files
  - new: https://git.openjdk.org/jdk/pull/22202/files/95043a3b..987a535f

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jdk&pr=22202&range=01
 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=22202&range=00-01

  Stats: 6 lines in 3 files changed: 0 ins; 5 del; 1 mod
  Patch: https://git.openjdk.org/jdk/pull/22202.diff
  Fetch: git fetch https://git.openjdk.org/jdk.git pull/22202/head:pull/22202

PR: https://git.openjdk.org/jdk/pull/22202


More information about the hotspot-dev mailing list