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

Yuri Gaevsky duke at openjdk.org
Fri Dec 8 14:17:23 UTC 2023


On Fri, 8 Dec 2023 13:43:33 GMT, Fei Yang <fyang at openjdk.org> wrote:

>> We need this check because after wide "unrolling" loop the cnt could be 0,1,2 or 3, see "don't forget about tail" comment at the line 1502.
>
> The control flow will be directed to `DONE` by the `beqz` check at L1493 when `cnt` is zero.

I meant other case(s): if '`cnt`' is equal to 4/8/... then we pass the initial `cnt==zero` check at L1493 and go further into wide loop, where the 'cnt' is zero (L1501)- that's why that check is needed.

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

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


More information about the hotspot-dev mailing list