RFR: 8350589: Investigate cleaner implementation of AArch64 ML-DSA intrinsic introduced in JDK-8348561 [v5]

Andrew Dinn adinn at openjdk.org
Wed Mar 19 17:26:12 UTC 2025


On Wed, 19 Mar 2025 17:14:08 GMT, Ferenc Rakoczi <duke at openjdk.org> wrote:

>> Andrew Dinn has updated the pull request with a new target base due to a merge or a rebase. The pull request now contains six commits:
>> 
>>  - use references and const to avoid VSeq copying and fix int array arg issue
>>  - fix comment
>>  - fix invalid register argument
>>  - fix errors in comments
>>  - fix whitespace errors
>>  - Clearer implementation of AArch64 dilithium generator
>
> It all looks great to me, the comments are good (a little too verbose for my taste, but they explain well what is happening in the code).

@ferakocz Thanks. I'll integrate this now.
I'll also propose a restructuring of the ML_KEM implementation using VSeq as part of a review of that PR.

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

PR Comment: https://git.openjdk.org/jdk/pull/24026#issuecomment-2737470277


More information about the hotspot-compiler-dev mailing list