Re: CURRENT: CLANG 3.3 and -stad=c++11 and -stdlib=libc++: isnan()/isninf() oddity

From: Scot Hetzel <swhetzel_at_gmail.com>
Date: Fri, 12 Jul 2013 12:13:58 -0500
On Fri, Jul 12, 2013 at 11:16 AM, Scot Hetzel <swhetzel_at_gmail.com> wrote:
> On Thu, Jul 11, 2013 at 9:33 AM, David Chisnall <theraven_at_freebsd.org> wrote:
>> On 11 Jul 2013, at 13:11, Bruce Evans <brde_at_optusnet.com.au> wrote:
>>
>>> The error message for the __builtin_isnan() version is slightly better up
>>> to where it says more.
>>>
>>> The less-unportable macro can do more classification and detect problems
>>> at compile time using __typeof().
>>
>> The attached patch fixes the related test cases in the libc++ test suite.  Please review.
>>
>
>  #define        fpclassify(x) \
> -    ((sizeof (x) == sizeof (float)) ? __fpclassifyf(x) \
> -    : (sizeof (x) == sizeof (double)) ? __fpclassifyd(x) \
> -    : __fpclassifyl(x))
> +        __fp_type_select(x, __fpclassifyf, __fpclassifyd, __fpclassifyd)
>
> The last __fpclassifyd should be __fpclassifyl.
>
I see it has already been fixed.


-- 
DISCLAIMER:

No electrons were maimed while sending this message. Only slightly bruised.
Received on Fri Jul 12 2013 - 15:13:59 UTC

This archive was generated by hypermail 2.4.0 : Wed May 19 2021 - 11:40:39 UTC