RFR: 8344393: RISC-V: Remove option UseRVVForBigIntegerShiftIntrinsics [v2]
Feilong Jiang
fjiang at openjdk.org
Tue Nov 19 02:33:55 UTC 2024
On Tue, 19 Nov 2024 00:24:14 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
>
> Fei Yang has updated the pull request incrementally with one additional commit since the last revision:
>
> Remove jvmci use cases
Marked as reviewed by fjiang (Committer).
-------------
PR Review: https://git.openjdk.org/jdk/pull/22202#pullrequestreview-2444160013
More information about the hotspot-dev
mailing list