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

Hamlin Li mli at openjdk.org
Tue Nov 5 09:49:43 UTC 2024


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!

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

Commit messages:
 - Initial commit
 - Merge branch 'openjdk:master' into master
 - Merge branch 'openjdk:master' into master
 - Merge branch 'openjdk:master' into master
 - Merge branch 'openjdk:master' into master
 - Revert "initial commit"
 - initial commit

Changes: https://git.openjdk.org/jdk/pull/21897/files
  Webrev: https://webrevs.openjdk.org/?repo=jdk&pr=21897&range=00
  Issue: https://bugs.openjdk.org/browse/JDK-8343600
  Stats: 34 lines in 1 file changed: 18 ins; 16 del; 0 mod
  Patch: https://git.openjdk.org/jdk/pull/21897.diff
  Fetch: git fetch https://git.openjdk.org/jdk.git pull/21897/head:pull/21897

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


More information about the hotspot-dev mailing list