RFR: 8344393: RISC-V: Remove option UseRVVForBigIntegerShiftIntrinsics
Fei Yang
fyang at openjdk.org
Mon Nov 18 11:32:33 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.
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
-------------
Commit messages:
- 8344393: RISC-V: Remove option UseRVVForBigIntegerShiftIntrinsics
Changes: https://git.openjdk.org/jdk/pull/22202/files
Webrev: https://webrevs.openjdk.org/?repo=jdk&pr=22202&range=00
Issue: https://bugs.openjdk.org/browse/JDK-8344393
Stats: 4 lines in 3 files changed: 0 ins; 3 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