On Wed, 9 Feb 2022 22:42:50 GMT, Paul Sandoz <psan...@openjdk.org> wrote:

>> src/java.base/share/classes/java/math/BigInteger.java line 1603:
>> 
>>> 1601:      * parallel multiplication algorithm will use more CPU resources
>>> 1602:      * to compute the result faster, with no increase in memory
>>> 1603:      * consumption.
>> 
>> The implNote should cover a space of possible parallel multiply 
>> implementations so it doesn't have to be updated as often as the 
>> implementation is tuned or adjusted. So I'd prefer to have a statement like 
>> "may use more memory" even if the current implementation doesn't actually 
>> use more memory. If there are any "contraindications" on when to use the 
>> method, they could be listed here too.
>
> @kabutz I approved, but can you address Joe's comment, then i will update the 
> CSR.

Done.

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

PR: https://git.openjdk.java.net/jdk/pull/6409

Reply via email to