RFR: 8348561: Add aarch64 intrinsics for ML-DSA [v2]
Andrew Dinn
adinn at openjdk.org
Tue Feb 4 11:48:09 UTC 2025
On Mon, 3 Feb 2025 18:11:51 GMT, Ferenc Rakoczi <duke at openjdk.org> wrote:
>> @ferakocz I'm afraid you lucked out on getting your change committed before my reorganization of the stub generation code. If you are unsure of how to do the merge so your new stub is declared and generated following the new model (see the doc comments in stubDeclarations.hpp for details) let me know and I'll be happy to help you sort it out.
>
>> @ferakocz I'm afraid you lucked out on getting your change committed before my reorganization of the stub generation code. If you are unsure of how to do the merge so your new stub is declared and generated following the new model (see the doc comments in stubDeclarations.hpp for details) let me know and I'll be happy to help you sort it out.
>
> @adinn I think I managed to figure it out. Please take a look at the PR and let me know if I should have done anything differently.
@ferakocz Yes, the stub declaration part of it looks to be correct.
The rest of the patch will need at least two reviewers (@theRealAph? @martinuy? @franferrax) and may take some time to review, given that they will probably need to read up on the maths and algorithms. As an aid for reviewers and maintainers it would be good to insert a comment into the generator file linking the implementations to the relevant maths and algorithm. I found the FIPS-204 spec and the CRYSTALS-Dilithium Algorithm Specifications and Supporting Documentation paper, Shi Bai, Léo Ducas et al, 2021 - are they the best ones to look at?
-------------
PR Comment: https://git.openjdk.org/jdk/pull/23300#issuecomment-2633666753
More information about the hotspot-dev
mailing list