[jdk17u-dev] RFR: 8313322: RISC-V: implement MD5 intrinsic
Ilya Gavrilin
duke at openjdk.org
Thu Sep 14 11:48:13 UTC 2023
Please review this backport of md5 intrinsics to jdk17u-dev.
Backport is not clean because:
- There was no no iterator for `class AbstractRegSet`, as an example `RegSetIterator` from aarch64 used: https://github.com/openjdk/jdk17u-dev/blob/0ddf1ebe3df119be2029b12feee93d0128e7cae0/src/hotspot/cpu/aarch64/register_aarch64.hpp#L379-L421
- Method `uint AbstractRegSet::size()` was missing
- Also for `size()` method we need `population_count()` function, so included `population_count.hpp` in `register_riscv.hpp`
- MD5 generation requires huge `codeBuffer`, with old codeBuffer sizes got error during intrinsic generation, so extended them:
code_size1: 19000 -> 20000
code_size2: 28000 -> 30000
-------------
Commit messages:
- Backport md5 sum to jdk17
Changes: https://git.openjdk.org/jdk17u-dev/pull/1739/files
Webrev: https://webrevs.openjdk.org/?repo=jdk17u-dev&pr=1739&range=00
Issue: https://bugs.openjdk.org/browse/JDK-8313322
Stats: 463 lines in 6 files changed: 456 ins; 1 del; 6 mod
Patch: https://git.openjdk.org/jdk17u-dev/pull/1739.diff
Fetch: git fetch https://git.openjdk.org/jdk17u-dev.git pull/1739/head:pull/1739
PR: https://git.openjdk.org/jdk17u-dev/pull/1739
More information about the jdk-updates-dev
mailing list