From owner-freebsd-standards@FreeBSD.ORG Fri Jul 12 17:13:59 2013 Return-Path: Delivered-To: freebsd-standards@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [8.8.178.115]) by hub.freebsd.org (Postfix) with ESMTP id 673E1AFA; Fri, 12 Jul 2013 17:13:59 +0000 (UTC) (envelope-from swhetzel@gmail.com) Received: from mail-ie0-x232.google.com (mail-ie0-x232.google.com [IPv6:2607:f8b0:4001:c03::232]) by mx1.freebsd.org (Postfix) with ESMTP id 10CBE1118; Fri, 12 Jul 2013 17:13:59 +0000 (UTC) Received: by mail-ie0-f178.google.com with SMTP id u16so21061158iet.37 for ; Fri, 12 Jul 2013 10:13:58 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20120113; h=mime-version:in-reply-to:references:date:message-id:subject:from:to :cc:content-type; bh=QiisUPirFKoVMBGknphIrRZc1nDMAFRG3tSIkRd6f4A=; b=vxMm7x7CcKMfsBiyJzfiOWrjLS025G2jU91bPy+W96nfW9qdCgS9Xw2/DSv+X0SCDb goeFjw/mOmVXX3usiEu1nAVePgNC1I6223OMcOpKWPqRzWc5q7PVNFI0+QYFGlg3kUZC ggLyEeaIXiPbojCUchcVEYkwn7v5YnqwoP7UN+H/MgQgcHRkgt2m0J+I5DYPwVVHkVcN Ct6HMwonTcyBHk8P4lQYoYdW41t6zF9dc10dOCjRECmfC1YjXNVjM6BND8Jsioc5WHRz Qv1suFDojub+xZ9OagB/FmqcP18QB1oBMnyH3u2o2XKuXpKxm6YymlcvdN1sUjCXdNRd mSzA== MIME-Version: 1.0 X-Received: by 10.50.67.43 with SMTP id k11mr1242994igt.26.1373649238734; Fri, 12 Jul 2013 10:13:58 -0700 (PDT) Received: by 10.50.221.179 with HTTP; Fri, 12 Jul 2013 10:13:58 -0700 (PDT) In-Reply-To: References: <20130710155809.0f589c22@thor.walstatt.dyndns.org> <20130710183315.725dfde0@thor.walstatt.dyndns.org> <20130710203200.5359fd18@thor.walstatt.dyndns.org> <51DDC04B.6040209@FreeBSD.org> <20957.49978.73666.392417@khavrinen.csail.mit.edu> <20130711130043.R920@besplex.bde.org> <20130711202908.L84170@besplex.bde.org> Date: Fri, 12 Jul 2013 12:13:58 -0500 Message-ID: Subject: Re: CURRENT: CLANG 3.3 and -stad=c++11 and -stdlib=libc++: isnan()/isninf() oddity From: Scot Hetzel To: David Chisnall Content-Type: text/plain; charset=ISO-8859-1 Cc: FreeBSD CURRENT , Garrett Wollman , "freebsd-toolchain@FreeBSD.org" , Tijl Coosemans , "freebsd-standards@FreeBSD.org" X-BeenThere: freebsd-standards@freebsd.org X-Mailman-Version: 2.1.14 Precedence: list List-Id: Standards compliance List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Fri, 12 Jul 2013 17:13:59 -0000 On Fri, Jul 12, 2013 at 11:16 AM, Scot Hetzel wrote: > On Thu, Jul 11, 2013 at 9:33 AM, David Chisnall wrote: >> On 11 Jul 2013, at 13:11, Bruce Evans 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.