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 <[email protected]> wrote: > On Sun, Feb 5, 2012 at 6:00 PM, Aaron Ballman <[email protected]> wrote: >> On Mon, Jan 16, 2012 at 9:54 PM, Eitan Adler <[email protected]> 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 _______________________________________________ cfe-commits mailing list [email protected] http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
