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