Hi Robert,

On Sep 15, 2014, at 2:09 AM, Robert Gibson <robbiexgib...@yahoo.com> wrote:

> How are the performance characteristics after the patch?

I have not tested that aspect as yet but clearly it should be evaluated. Given 
that JDK-8057793 causes a violation of the specification of BigDecimal I 
thought it was important to check in the fix as-is without further delay.

> I see that a lot of effort went in to tuning last December under 8029501.

Yes, that was a fair bit of work.

> By the way, as part of my investigations into this bug I noticed that 
> BigIntegerTest::divideLarge no longer tests the Burnikel-Ziegler part of the 
> division code, since the heuristic to decide Knuth or BZ diverged from the 
> algorithm to generate the test cases (as part of 8029501). 

I created https://bugs.openjdk.java.net/browse/JDK-8058505 for this problem.

Thanks,

Brian

Reply via email to