RFR: 8351412: Add AVX-512 intrinsics for ML-KEM [v5]

Sandhya Viswanathan sviswanathan at openjdk.org
Fri May 16 00:32:51 UTC 2025


On Thu, 15 May 2025 00:36:26 GMT, Sandhya Viswanathan <sviswanathan at openjdk.org> wrote:

>> Ferenc Rakoczi has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Responding to comments by Sandhya.
>
> Another minor comment. Rest of the PR looks good to me.

> @sviswa7, thanks a lot for the review! If you agree with my changes to load the constants using broadcasting instructions instead of full AVX register loads, would you be so kind as to approve the PR and sponsor my integration?

The broadcast instructions look good. I only have one query on montMul above that I have wondering about.

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

PR Comment: https://git.openjdk.org/jdk/pull/24953#issuecomment-2885339535


More information about the graal-dev mailing list