[jdk17u-dev] Integrated: 8313322: RISC-V: implement MD5 intrinsic
Ilya Gavrilin
igavrilin at openjdk.org
Wed Sep 27 17:07:30 UTC 2023
On Thu, 14 Sep 2023 11:41:18 GMT, Ilya Gavrilin <igavrilin at openjdk.org> wrote:
> Please review this backport of md5 intrinsics to jdk17u-dev.
>
> Backport is not clean because:
>
> - There was 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
This pull request has now been integrated.
Changeset: e7fc89db
Author: Ilya Gavrilin <igavrilin at openjdk.org>
Committer: Vladimir Kempik <vkempik at openjdk.org>
URL: https://git.openjdk.org/jdk17u-dev/commit/e7fc89db52a1ca4ce4c347612b19b8054b1c8d76
Stats: 464 lines in 6 files changed: 457 ins; 1 del; 6 mod
8313322: RISC-V: implement MD5 intrinsic
Reviewed-by: fyang
Backport-of: b093880acd89d8d0bccd4b8b260b721f4dcfc161
-------------
PR: https://git.openjdk.org/jdk17u-dev/pull/1739
More information about the jdk-updates-dev
mailing list