[cfe-commits] proposed fix to bug Bug 7910

Aaron Ballman aaron at aaronballman.com
Tue Feb 7 05:50:57 PST 2012


Thanks for the patch!  It's been committed in r149984.  You can mark
the bug as being fixed now, too.

Thanks again!

~Aaron

On Mon, Feb 6, 2012 at 9:17 PM, Eitan Adler <lists at eitanadler.com> wrote:
> On Sun, Feb 5, 2012 at 6:00 PM, Aaron Ballman <aaron at aaronballman.com> wrote:
>> On Mon, Jan 16, 2012 at 9:54 PM, Eitan Adler <lists at eitanadler.com> wrote:
>>> I was asked to resend with an attached patch instead
>>
>> Patch looks reasonable, but does have tabs in it.  Also, the error
>> isn't quite as clear as it could be.  Perhaps "a significand" or
>> "significant digits" instead of "a number?"
>
> I went with "a significand" and believe I fixed the spaces issues.
> Attached is the updated patch and I've attached it to the bug as well:
> http://llvm.org/bugs/show_bug.cgi?id=7910
>
> --
> Eitan Adler




More information about the cfe-commits mailing list