From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.2 (2018-09-13) on dcvr.yhbt.net X-Spam-Level: X-Spam-ASN: AS31976 209.132.180.0/23 X-Spam-Status: No, score=-4.1 required=3.0 tests=AWL,BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_EF,HEADER_FROM_DIFFERENT_DOMAINS, MAILING_LIST_MULTI,RCVD_IN_DNSWL_MED,SPF_HELO_PASS,SPF_PASS shortcircuit=no autolearn=ham autolearn_force=no version=3.4.2 Received: from sourceware.org (server1.sourceware.org [209.132.180.131]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by dcvr.yhbt.net (Postfix) with ESMTPS id 02F631F462 for ; Wed, 5 Jun 2019 22:46:11 +0000 (UTC) DomainKey-Signature: a=rsa-sha1; c=nofws; d=sourceware.org; h=list-id :list-unsubscribe:list-subscribe:list-archive:list-post :list-help:sender:date:from:to:cc:subject:in-reply-to:references :mime-version:content-type:message-id; q=dns; s=default; b=YdXqF YCfsCSPgq6cAJ/Tz7lPaY8q6DWNfP7YIaFRtuYsi1IghHWBLZLNbh5HqZtAuMMeo ceQh+cQu/cKGo1hwwykXOQxzRGNhKngOhJz2NeO1fAE9ckHRU/UQD1JRS5EA0TJQ 7TdGyRQYeb4Iv7zqs8NFm8luDqxrzpKb8FOC74= DKIM-Signature: v=1; a=rsa-sha1; c=relaxed; d=sourceware.org; h=list-id :list-unsubscribe:list-subscribe:list-archive:list-post :list-help:sender:date:from:to:cc:subject:in-reply-to:references :mime-version:content-type:message-id; s=default; bh=oM4lCPfIq8u NIfzZN6mQDho6FPo=; b=S7d3ZTCmarSjAEuezXC0/HW0GhfRv5+TBza/9ARQrR6 w+gBDkivIkswQo3jM26x3uew3DBzGh09K8Hhk+3Y246Zhv0EFzfE5lMueN3IrakU kIDWadegQX+0vql/10AeiBRPsn01DGSANNKY9Zlu8XwXwLwlI3YrXJe2A3SjDB2o = Received: (qmail 86949 invoked by alias); 5 Jun 2019 22:46:10 -0000 Mailing-List: contact libc-alpha-help@sourceware.org; run by ezmlm Precedence: bulk List-Id: List-Unsubscribe: List-Subscribe: List-Archive: List-Post: List-Help: , Sender: libc-alpha-owner@sourceware.org Received: (qmail 86940 invoked by uid 89); 5 Jun 2019 22:46:09 -0000 Authentication-Results: sourceware.org; auth=none X-HELO: mx0a-001b2d01.pphosted.com Date: Wed, 5 Jun 2019 19:45:57 -0300 From: "Gabriel F. T. Gomes" To: Adhemerval Zanella Cc: Subject: Re: [PATCH 12/28] math: Use wordsize-64 version for isnan In-Reply-To: <20190329133529.22523-13-adhemerval.zanella@linaro.org> References: <20190329133529.22523-1-adhemerval.zanella@linaro.org> <20190329133529.22523-13-adhemerval.zanella@linaro.org> MIME-Version: 1.0 Content-Type: multipart/mixed; boundary="MP_/BKzAQ=bVP56r.5+ZA0bC.dj" x-cbid: 19060522-0020-0000-0000-00000EF4B89A X-IBM-SpamModules-Scores: X-IBM-SpamModules-Versions: BY=3.00011221; HX=3.00000242; KW=3.00000007; PH=3.00000004; SC=3.00000286; SDB=6.01213781; UDB=6.00637978; IPR=6.00994855; MB=3.00027199; MTD=3.00000008; XFM=3.00000015; UTC=2019-06-05 22:46:05 X-IBM-AV-DETECTION: SAVI=unused REMOTE=unused XFE=unused x-cbparentid: 19060522-0021-0000-0000-0000661E2E74 Message-Id: <20190605194557.1b6d97f8@tereshkova.br.ibm.com> --MP_/BKzAQ=bVP56r.5+ZA0bC.dj Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Content-Disposition: inline On Fri, Mar 29 2019, Adhemerval Zanella wrote: > > - The resulting binary difference on 32 bits architecture is minimum > for the non hotspot symbol. I don't actually understand what the impact on 32-bits platforms would be, so I can't comment on that, but, if this patch turns out to be problematic for 32-bits, we could patch sysdeps/ieee754/ldbl-opt files to be mindful of __WORDSIZE and selectively include files from sysdeps/ieee754/dbl-64 or sysdeps/ieee754/dbl-64/wordsize-64 (see attached patch for an explanation of what I mean). My point in raising this is that the subsequent patch [1] benefits from this current patch (or, alternatively, from the attached patch, since they have the same outcame, as expected). So, if this patch gets objections we could go with the alternative. [1] https://sourceware.org/ml/libc-alpha/2019-03/msg00674.html --MP_/BKzAQ=bVP56r.5+ZA0bC.dj Content-Type: text/plain; charset=us-ascii Content-Transfer-Encoding: 7bit Content-Disposition: attachment; filename=wordsize-test.patch diff --git a/sysdeps/ieee754/ldbl-opt/s_finite.c b/sysdeps/ieee754/ldbl-opt/s_finite.c index 7d3ab0068d..85ad5e625a 100644 --- a/sysdeps/ieee754/ldbl-opt/s_finite.c +++ b/sysdeps/ieee754/ldbl-opt/s_finite.c @@ -1,5 +1,10 @@ +#include #include +#if __WORDSIZE == 64 +#include +#else #include +#endif weak_alias (__finite, ___finite) #if IS_IN (libm) # if LONG_DOUBLE_COMPAT(libm, GLIBC_2_1) diff --git a/sysdeps/ieee754/ldbl-opt/s_isinf.c b/sysdeps/ieee754/ldbl-opt/s_isinf.c index 1f760a0320..da5dc01601 100644 --- a/sysdeps/ieee754/ldbl-opt/s_isinf.c +++ b/sysdeps/ieee754/ldbl-opt/s_isinf.c @@ -1,5 +1,10 @@ +#include #include +#if __WORDSIZE == 64 +#include +#else #include +#endif #if !IS_IN (libm) # if LONG_DOUBLE_COMPAT(libc, GLIBC_2_0) compat_symbol (libc, __isinf, __isinfl, GLIBC_2_0); diff --git a/sysdeps/ieee754/ldbl-opt/s_isnan.c b/sysdeps/ieee754/ldbl-opt/s_isnan.c index 33f57f1955..99ee75ce34 100644 --- a/sysdeps/ieee754/ldbl-opt/s_isnan.c +++ b/sysdeps/ieee754/ldbl-opt/s_isnan.c @@ -1,5 +1,10 @@ +#include #include +#if __WORDSIZE == 64 +#include +#else #include +#endif #if !IS_IN (libm) # if LONG_DOUBLE_COMPAT(libc, GLIBC_2_0) compat_symbol (libc, __isnan, __isnanl, GLIBC_2_0); --MP_/BKzAQ=bVP56r.5+ZA0bC.dj--