RFR: 8318217: RISC-V: C2 VectorizedHashCode [v3]
Yuri Gaevsky
duke at openjdk.org
Fri Nov 17 11:27:32 UTC 2023
On Thu, 16 Nov 2023 17:00:45 GMT, Hamlin Li <mli at openjdk.org> wrote:
>> Yuri Gaevsky has updated the pull request incrementally with one additional commit since the last revision:
>>
>> Addressed most of suggestions for code improvements from @Hamlin-Li
>
> src/hotspot/cpu/riscv/c2_MacroAssembler_riscv.cpp line 1492:
>
>> 1490: beqz(cnt, DONE);
>> 1491:
>> 1492: lw(pow31_2, ExternalAddress(StubRoutines::riscv::arrays_hashcode_powers_of_31()
>
> Now you don't need this `lw` anymore, as 961 will fit in an immediate, so `mv pow31_2, 961` should be fine.
Agreed.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/16629#discussion_r1397121746
More information about the hotspot-dev
mailing list