[riscv-port] RFR: 8278644: riscv: Intrinsify mulAdd [v2]
Feilong Jiang
fjiang at openjdk.java.net
Tue Dec 21 09:06:27 UTC 2021
On Tue, 21 Dec 2021 08:41:46 GMT, Fei Yang <fyang at openjdk.org> wrote:
>> Feilong Jiang has updated the pull request incrementally with one additional commit since the last revision:
>>
>> change the type of i in for loop to match the type of unroll
>
> src/hotspot/cpu/riscv/macroAssembler_riscv.cpp line 3062:
>
>> 3060: blt(len, tmp, L_tail_loop);
>> 3061: bind(L_unroll);
>> 3062: for (unsigned i = 0; i < unroll; i++) {
>
> Better to change type of "i" into signed, i.e., "for (int i = 0; i < unroll; i++)"
Thanks, the type of `i` is the same as `unroll` now.
-------------
PR: https://git.openjdk.java.net/riscv-port/pull/36
More information about the riscv-port-dev
mailing list