On Mon, 21 Nov 2022 07:41:12 GMT, Daniel Jeliński <djelin...@openjdk.org> wrote:
>> Xue-Lei Andrew Fan has updated the pull request incrementally with one
>> additional commit since the last revision:
>>
>> remove duplicated bench test
>
> src/java.base/share/classes/sun/security/util/math/intpoly/IntegerPolynomial.java
> line 561:
>
>> 559: // Reduce if required.
>> 560: // if (numAdds >= maxAdds) {
>> 561: if (numAdds > 32 - bitsPerLimb) {
>
> if we allow this number of additions, `setProduct(SmallValue)` might overflow
> in the future. Currently it's safe - we only create a limited set of
> SmallValues, and they are all small enough to avoid this risk.
> `getSmallValue` allows numbers up to `bitsPerLimb` bits. I think we can
> adjust that.
Sorry, I did not get the ideas. Did you meant to adjust the implementation of
getSmallValue to allow 32 bits int value?
-------------
PR: https://git.openjdk.org/jdk/pull/10624
