RFR: JDK-8277175 : Add a parallel multiply method to BigInteger [v10]

kabutz duke at openjdk.java.net
Thu Feb 10 07:07:48 UTC 2022


On Wed, 9 Feb 2022 22:42:50 GMT, Paul Sandoz <psandoz at 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


More information about the core-libs-dev mailing list