RFR: 8339738: RISC-V: Vectorize crc32 intrinsic [v4]

Hamlin Li mli at openjdk.org
Wed Sep 11 08:59:23 UTC 2024


On Wed, 11 Sep 2024 08:24:21 GMT, Fei Yang <fyang at openjdk.org> wrote:

>> I was thinking the similar thing.
>> But it's bit tedious, as I need to figure out which jump should be far and which not, or I could just simply change any where suspicious, but it's not good either.
>> So I pick the simplest solution to move it to the end, in other intrinsics I saw the similar code layout, they don't have the comment for it, but I guess they are doing the similar things.
>
> Yeah, I understand what you mean. I am just a bit concerned about the two `j(L_exit)` added. Seems the one in the `if (UseRVV)` block is not necessary?

Thanks, fixed.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/20910#discussion_r1753678733


More information about the hotspot-dev mailing list