On Sat, 15 Jan 2022 18:03:33 GMT, Paul Sandoz <psan...@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

Reply via email to