RFR 7131192: BigInteger.doubleValue() is depressingly slow
David Chase
david.r.chase at oracle.com
Thu Jun 20 15:14:25 UTC 2013
Not a reviewer, but depressingly much experience hacking Java FP.
It all looks right to me (i.e., exponent extraction and rounding modes including the round-up carry to the next higher power of two) and I like the comments explaining the little hacks, and I trust the testing to smoke out fencepost errors.
David
On 2013-06-18, at 8:45 PM, Brian Burkhalter <brian.burkhalter at oracle.com> wrote:
> The updated comments have been propagated to the online webrev
>
> http://cr.openjdk.java.net/~bpb/7131192/
>
> On Jun 17, 2013, at 11:39 AM, Brian Burkhalter wrote:
>
>> I'll update this but without reposting the webrev until I can upload the HTML version.
>>
>> On Jun 17, 2013, at 11:37 AM, Louis Wasserman wrote:
>>
>>> You're right, it'd be WIDTH - 1, but since most of the comments refer to BITS + 1, that evens out nicely.
>>>
>>> On Mon, Jun 17, 2013 at 11:34 AM, Brian Burkhalter<brian.burkhalter at oracle.com> wrote:
>>> Would it be WIDTH or WIDTH - 1, i.e., with or without the "implied" bit?
>>>
>>> On Jun 17, 2013, at 11:32 AM, Louis Wasserman wrote:
>>>
>>>> The comments mention SIGNIFICAND_BITS, which I think should probably be SIGNIFICAND_WIDTH?
>>
>
More information about the core-libs-dev
mailing list