On Thu, 10 Nov 2022 22:41:31 GMT, Volodymyr Paprotski <d...@openjdk.org> wrote:

>> src/java.base/share/classes/com/sun/crypto/provider/Poly1305.java line 249:
>> 
>>> 247:     @ForceInline
>>> 248:     @IntrinsicCandidate
>>> 249:     private void processMultipleBlocks(byte[] input, int offset, int 
>>> length, long[] aLimbs, long[] rLimbs) {
>> 
>> A comment here to indicate aLimbs and rLimbs are part of a and r and used in 
>> intrinsic.
>
> done

Overall, it looks weird to see aLimbs/rLimbs being unused, but I see why it is 
so. If security folks are fine with that, I'm OK with it as well.

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

PR: https://git.openjdk.org/jdk/pull/10582

Reply via email to