[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