Integrated: 8340732: RISC-V: Refactor crc32 scalar version

Hamlin Li mli at openjdk.org
Wed Oct 2 07:50:43 UTC 2024


On Tue, 24 Sep 2024 07:22:51 GMT, Hamlin Li <mli at openjdk.org> wrote:

> Hi,
> Can you help to review this patch?
> As discussed in https://github.com/openjdk/jdk/pull/20910#discussion_r1755150447, it's helpful to refactor the existing scalar version of crc32 intrinsic.
> Several refactoring are done in this pr,
> 1. Simplify the `len` usage, now it only decreases (i.e. change in one direction)
> 2. Simplify the code paths
> 3. Remove one instruction in `L_by4_loop`
> 4. Remove unnecessary code
> 5. Other misc
> 
> Thanks!

This pull request has now been integrated.

Changeset: 52c2ea65
Author:    Hamlin Li <mli at openjdk.org>
URL:       https://git.openjdk.org/jdk/commit/52c2ea65c4ab6a2d25e9de465fbf20034fe79028
Stats:     57 lines in 2 files changed: 9 ins; 26 del; 22 mod

8340732: RISC-V: Refactor crc32 scalar version

Reviewed-by: fyang

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

PR: https://git.openjdk.org/jdk/pull/21150


More information about the hotspot-dev mailing list