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