Integrated: 8343600: RISC-V: enable CRC32 intrinsic when either Zba or RVV are supported

Hamlin Li mli at openjdk.org
Tue Nov 12 10:21:48 UTC 2024


On Tue, 5 Nov 2024 09:44:21 GMT, Hamlin Li <mli at openjdk.org> wrote:

> Hi,
> Can you help to review this patch?
> 
> There were 2 intrinsic optimizations, scalar one and vector one, 
> * plain scalar one optimizes cases when data size is small, scalar one with zba optimizes cases when data size is big;
> * vector one optimizes cases when data size is big (better than scalar one with zba).
> 
> With RVV supported, when the size is small it will go to scalar code path, when the size is big it will go to RVV path then scalar path, so that means the RVV itself are sufficient enough to enable the intrinsic even without Zba supported.
> 
> Thanks!

This pull request has now been integrated.

Changeset: 2c1e4c38
Author:    Hamlin Li <mli at openjdk.org>
URL:       https://git.openjdk.org/jdk/commit/2c1e4c381615ce52276f4bf331a1e7a845af4b6e
Stats:     34 lines in 1 file changed: 18 ins; 16 del; 0 mod

8343600: RISC-V: enable CRC32 intrinsic when either Zba or RVV are supported

Reviewed-by: fyang

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

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


More information about the hotspot-dev mailing list