On Wed, 12 Aug 2026 07:48:38 GMT, Andrew Dinn <[email protected]> wrote:

>> src/hotspot/cpu/x86/stubGenerator_x86_64_kyber.cpp line 357:
>> 
>>> 355: //          xmm16: Barrett multiplier
>>> 356: //          xmm17: q
>>> 357: // Output:  xmm0-xmm7 (reduced coefficients each in [0, q])
>> 
>> Since you're describing set notation, this would be:
>> 
>>> // Output:  xmm0-xmm7 (reduced coefficients each in [0, q))
>> 
>> Note the exclusion of `q`.
>
> Ahem ...  set notation --> integer intervals ;-)

Isn't the inclusive range permitted? 
https://github.com/openjdk/jdk/blob/e1502b52f099c37c9c2fd2aad147dec95ff685b0/src/java.base/share/classes/com/sun/crypto/provider/ML_KEM.java#L1553

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

PR Review Comment: https://git.openjdk.org/jdk/pull/32295#discussion_r3765283422

Reply via email to