RFR: 8318217: RISC-V: C2 VectorizedHashCode [v2]

Yuri Gaevsky duke at openjdk.org
Wed Nov 15 16:00:37 UTC 2023


On Tue, 14 Nov 2023 15:40:46 GMT, Hamlin Li <mli at openjdk.org> wrote:

>> Yuri Gaevsky has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Minor cosmetic fixes.
>
> src/hotspot/cpu/riscv/c2_MacroAssembler_riscv.cpp line 1512:
> 
>> 1510: 
>> 1511:   bind(WIDE_LOOP);
>> 1512:   DO_ELEMENT_LOAD(tmp1, 0)
> 
> Can you add `;` at the end of the statement? similar comments for other DO_ELEMENT_LOAD's

Agreed.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/16629#discussion_r1394412176


More information about the hotspot-dev mailing list