From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 40954 invoked by alias); 13 Dec 2019 23:46:22 -0000 Mailing-List: contact libc-alpha-help@sourceware.org; run by ezmlm Precedence: bulk List-Id: List-Subscribe: List-Archive: List-Post: List-Help: , Sender: libc-alpha-owner@sourceware.org Received: (qmail 40942 invoked by uid 89); 13 Dec 2019 23:46:22 -0000 Authentication-Results: sourceware.org; auth=none X-Spam-SWARE-Status: No, score=-12.7 required=5.0 tests=AWL,BAYES_00,GIT_PATCH_3,RCVD_IN_DNSWL_LOW,SPF_PASS autolearn=ham version=3.3.1 spammy=arrives X-HELO: mx0a-001b2d01.pphosted.com Subject: Re: [PATCH v3 11/11] RFC: powerpc64le: Enable support for IEEE long double To: "Gabriel F. T. Gomes" Cc: libc-alpha@sourceware.org References: <20191203170540.18428-1-gabriel@inconstante.net.br> <20191203170540.18428-12-gabriel@inconstante.net.br> <3097820c-5cb5-b834-8714-67108a1bc745@linux.ibm.com> <20191213194136.14608794@tereshkova> From: Paul E Murphy Message-ID: <3c697aa9-4aec-0d9b-b120-83e1b34e551e@linux.ibm.com> Date: Fri, 13 Dec 2019 23:46:00 -0000 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101 Thunderbird/68.1.0 MIME-Version: 1.0 In-Reply-To: <20191213194136.14608794@tereshkova> Content-Type: text/plain; charset=utf-8; format=flowed Content-Transfer-Encoding: 7bit X-SW-Source: 2019-12/txt/msg00521.txt.bz2 On 12/13/19 4:41 PM, Gabriel F. T. Gomes wrote: > On Mon, 09 Dec 2019, Paul E Murphy wrote: >> Ignoring the individual files for a moment, is it appropriate to split >> this into one or more patches to introduce the >> __LONG_DOUBLE_USE_FLOAT128 usage into the various user headers? > > Since __LONG_DOUBLE_USES_FLOAT128 is only defined to 1 in > ldbl-ibm128-compat's bits/long-double.h, I suppose this could work. It's > always defined to 0 by all other bits/long-double.h headers (as suggested > by Florian [1]), including what's currently installed for powerpc64le. > > Let me check if I got your suggestion right. Are you suggesting that all > the changes to installed headers in this patch be split into a separate > patch that can be integrated to master? Then, this patch (the last one) > would only touch the Implies, .abilist, and ldbl-128ibm-compat-abi.h > files... > > I.e.: > >>> sysdeps/powerpc/powerpc64/le/Implies-before | 1 + >>> .../powerpc64/le/ldbl-128ibm-compat-abi.h | 8 ++ >>> .../linux/powerpc/powerpc64/le/libc.abilist | 93 ++++++++++++++++ >>> .../linux/powerpc/powerpc64/le/libm.abilist | 104 ++++++++++++++++++ > > ^ these? Exactly. They can remain inert until the remaining support arrives and is enabled, while getting the benefit of any ancillary refactoring occuring while the remaining libm work is finalized. Though, that is my opinion and may not represent a consensus.