[jdk17u-dev] RFR: 8318953: RISC-V: Small refactoring for MacroAssembler::test_bit

Gui Cao gcao at openjdk.org
Thu Nov 2 11:09:08 UTC 2023


On Thu, 2 Nov 2023 03:40:59 GMT, Fei Yang <fyang at openjdk.org> wrote:

>> Hi, this backport to make the test_bit assember function more simpler and to improve performance when the parameter (1UL << bit_pos) exceeds 32-bit range. This is a risc-v specific change.
>> Backport is not clean. because the fastlock needs to call test_bit, in the mainline version, the fastlock function is renamed and moved to c2_MacroAssembler_riscv.cpp, but in jdk17u-dev, fastlock is still in riscv.ad.
>> ### Testing:
>> qemu 8.1.50:
>> - [x] Tier1 tests (release)
>
> Looks good.

@RealFYang Thanks for your review.

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

PR Comment: https://git.openjdk.org/jdk17u-dev/pull/1941#issuecomment-1790519449


More information about the jdk-updates-dev mailing list