<div dir="ltr"><div><div>I see no immediate issue with this patch, but I am not one of the usual reviewers for this part of the code base.<br></div><br></div>-- HT<br><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Feb 9, 2016 at 2:56 PM, Jorge Teixeira <span dir="ltr"><<a href="mailto:j.lopes.teixeira@gmail.com" target="_blank">j.lopes.teixeira@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Thanks Hubert. Somehow I omitted that prefix when typing the macros,<br>
and I did not noticed it when I was testing because on my arch<br>
DECIMAL_DIG is defined to be the LDBL version...<br>
<br>
Updated patch is attached.<br>
<span class="HOEnZb"><font color="#888888"><br>
JT<br>
</font></span><div class="HOEnZb"><div class="h5"><br>
On Tue, Feb 9, 2016 at 1:41 PM, Hubert Tong<br>
<<a href="mailto:hubert.reinterpretcast@gmail.com">hubert.reinterpretcast@gmail.com</a>> wrote:<br>
> There is a __LDBL_DECIMAL_DIG__ predefined macro. __DECIMAL_DIG__ will not<br>
> always be the same as __LDBL_DECIMAL_DIG__.<br>
><br>
> -- HT<br>
><br>
> On Mon, Feb 8, 2016 at 11:26 PM, Jorge Teixeira via cfe-commits<br>
> <<a href="mailto:cfe-commits@lists.llvm.org">cfe-commits@lists.llvm.org</a>> wrote:<br>
>><br>
>> Hi, I filed the bug (<a href="https://llvm.org/bugs/show_bug.cgi?id=26283" rel="noreferrer" target="_blank">https://llvm.org/bugs/show_bug.cgi?id=26283</a>) some<br>
>> time ago and nobody picked it up, so here is a trivial patch exposing<br>
>> the missing macros, that to the best of my ability were already<br>
>> present as the internal underscored versions.<br>
>><br>
>> Perhaps a more general bug about C11 floating point (lack of)<br>
>> conformance should be filed, so that some form of unit test/macro<br>
>> validation could be worked on, but this patch does scratch my current<br>
>> itch.<br>
>><br>
>> Successfully tested on x86-64 Xubuntu 14.04 with clang 3.8 from the<br>
>> ppa, patched with the attached diff.<br>
>><br>
>> First contribution, so feel free to suggest improvements or point to<br>
>> more detailed step-by-step instructions/guidelines.<br>
>><br>
>> Cheers,<br>
>><br>
>> JT<br>
>><br>
>> _______________________________________________<br>
>> cfe-commits mailing list<br>
>> <a href="mailto:cfe-commits@lists.llvm.org">cfe-commits@lists.llvm.org</a><br>
>> <a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits</a><br>
>><br>
><br>
</div></div></blockquote></div><br></div></div>