RFR: JDK-8277175 : Add a parallel multiply method to BigInteger [v7]
kabutz
duke at openjdk.java.net
Thu Jan 20 10:10:52 UTC 2022
On Sat, 15 Jan 2022 18:03:33 GMT, Paul Sandoz <psandoz at openjdk.org> wrote:
>> kabutz has updated the pull request incrementally with one additional commit since the last revision:
>>
>> Changed depth type to byte to save 8 bytes on each RecursiveSquare instance
>
> test/jdk/java/math/BigInteger/BigIntegerParallelMultiplyTest.java line 64:
>
>> 62: BigInteger fib = fibonacci(n, BigInteger::multiply);
>> 63: System.out.printf("fibonacci(%d) = %d%n", n, fib);
>> 64: }
>
> I think we can remove this and the loop block at #70-80, since we have the performance test. After that we are good.
Done.
-------------
PR: https://git.openjdk.java.net/jdk/pull/6409
More information about the core-libs-dev
mailing list