From patchwork Fri Dec 13 22:24:21 2019 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: "Gabriel F. T. Gomes" X-Patchwork-Id: 36862 Received: (qmail 93425 invoked by alias); 13 Dec 2019 22:24:56 -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 Delivered-To: mailing list libc-alpha@sourceware.org Received: (qmail 93372 invoked by uid 89); 13 Dec 2019 22:24:56 -0000 Authentication-Results: sourceware.org; auth=none X-Spam-SWARE-Status: No, score=-16.2 required=5.0 tests=AWL, BAYES_00, GIT_PATCH_0, GIT_PATCH_1, GIT_PATCH_2, GIT_PATCH_3, KAM_SHORT, RCVD_IN_DNSWL_NONE autolearn=ham version=3.3.1 spammy= X-HELO: smtpout1.mo803.mail-out.ovh.net From: "Gabriel F. T. Gomes" To: Subject: [PATCH v4 3/9] ldbl-128ibm-compat: Add *cvt functions Date: Fri, 13 Dec 2019 19:24:21 -0300 Message-ID: <20191213222427.14278-4-gabriel@inconstante.net.br> In-Reply-To: <20191213222427.14278-1-gabriel@inconstante.net.br> References: <20191213222427.14278-1-gabriel@inconstante.net.br> MIME-Version: 1.0 X-Ovh-Tracer-Id: 1130966458565643971 X-VR-SPAMSTATE: OK X-VR-SPAMSCORE: 0 X-VR-SPAMCAUSE: gggruggvucftvghtrhhoucdtuddrgedufedrudelledgudeitdcutefuodetggdotefrodftvfcurfhrohhfihhlvgemucfqggfjpdevjffgvefmvefgnecuuegrihhlohhuthemucehtddtnecunecujfgurhephffvufffkffojghfggfgtghisehtkeertdertddtnecuhfhrohhmpedfifgrsghrihgvlhcuhfdrucfvrdcuifhomhgvshdfuceoghgrsghrihgvlhesihhntghonhhsthgrnhhtvgdrnhgvthdrsghrqeenucffohhmrghinhephhhtthhpuhhrlhhsrdhnohdpghhnuhdrohhrghenucfkpheptddrtddrtddrtddpudejledrudekvddrudehuddrkeeknecurfgrrhgrmhepmhhouggvpehsmhhtphdqohhuthdphhgvlhhopefgigegrdgvmhhprdhlohgtrghlpdhinhgvtheptddrtddrtddrtddpmhgrihhlfhhrohhmpehgrggsrhhivghlsehinhgtohhnshhtrghnthgvrdgvthhirdgsrhdprhgtphhtthhopehlihgstgdqrghlphhhrgesshhouhhrtggvfigrrhgvrdhorhhgnecuvehluhhsthgvrhfuihiivgeptd From: "Gabriel F. T. Gomes" No changes since v3. - Reviewed-by Paul Murphy. Waiting for the preceding patches. Changes since v2: - Fix http URLs. No changes since v1. -- 8< -- This patch adds IEEE long double versions of q*cvt* functions for powerpc64le. Unlike all other long double to/from string conversion functions, these do not rely on internal functions that can take floating-point numbers with different formats and act on them accordingly, instead, the related files are rebuilt with the -mabi=ieeelongdouble compiler flag set. Having -mabi=ieeelongdouble passed to the compiler causes the object files to be marked with a .gnu_attribute that is incompatible with the .gnu_attribute in files built with -mabi=ibmlongdouble (the default). The difference causes error messages similar to the following: ld: libc_pic.a(s_isinfl.os) uses IBM long double, libc_pic.a(ieee128-qefgcvt_r.os) uses IEEE long double. collect2: error: ld returned 1 exit status make[2]: *** [../Makerules:649: libc_pic.os] Error 1 Although this warning is useful in other situations, the library actually needs to have functions with different long double formats, so .gnu_attribute generation is explicitly disabled for these files with the use of -mno-gnu-attribute. Tested for powerpc64le on the branch that actually enables the sysdeps/ieee754/ldbl-128ibm-compat for powerpc64le. Reviewed-by: Paul E. Murphy --- misc/efgcvt-template.c | 6 ++- misc/efgcvt_r-template.c | 6 ++- sysdeps/ieee754/ldbl-128ibm-compat/Makefile | 12 ++++- sysdeps/ieee754/ldbl-128ibm-compat/Versions | 6 +++ .../ldbl-128ibm-compat/ieee128-qefgcvt.c | 52 +++++++++++++++++++ .../ldbl-128ibm-compat/ieee128-qefgcvt_r.c | 34 ++++++++++++ 6 files changed, 113 insertions(+), 3 deletions(-) create mode 100644 sysdeps/ieee754/ldbl-128ibm-compat/ieee128-qefgcvt.c create mode 100644 sysdeps/ieee754/ldbl-128ibm-compat/ieee128-qefgcvt_r.c diff --git a/misc/efgcvt-template.c b/misc/efgcvt-template.c index 7fabdf264d..fe4c28bf0f 100644 --- a/misc/efgcvt-template.c +++ b/misc/efgcvt-template.c @@ -23,6 +23,10 @@ #include #include +#ifndef SPRINTF +# define SPRINTF sprintf +#endif + #define APPEND(a, b) APPEND2 (a, b) #define APPEND2(a, b) a##b @@ -66,6 +70,6 @@ __ECVT (FLOAT_TYPE value, int ndigit, int *decpt, int *sign) char * __GCVT (FLOAT_TYPE value, int ndigit, char *buf) { - sprintf (buf, "%.*" FLOAT_FMT_FLAG "g", MIN (ndigit, NDIGIT_MAX), value); + SPRINTF (buf, "%.*" FLOAT_FMT_FLAG "g", MIN (ndigit, NDIGIT_MAX), value); return buf; } diff --git a/misc/efgcvt_r-template.c b/misc/efgcvt_r-template.c index 977750118c..bae2d30a77 100644 --- a/misc/efgcvt_r-template.c +++ b/misc/efgcvt_r-template.c @@ -25,6 +25,10 @@ #include #include +#ifndef SNPRINTF +# define SNPRINTF __snprintf +#endif + #define APPEND(a, b) APPEND2 (a, b) #define APPEND2(a, b) a##b @@ -73,7 +77,7 @@ __FCVT_R (FLOAT_TYPE value, int ndigit, int *decpt, int *sign, /* Value is Inf or NaN. */ *sign = 0; - n = __snprintf (buf, len, "%.*" FLOAT_FMT_FLAG "f", MIN (ndigit, NDIGIT_MAX), + n = SNPRINTF (buf, len, "%.*" FLOAT_FMT_FLAG "f", MIN (ndigit, NDIGIT_MAX), value); /* Check for a too small buffer. */ if (n >= (ssize_t) len) diff --git a/sysdeps/ieee754/ldbl-128ibm-compat/Makefile b/sysdeps/ieee754/ldbl-128ibm-compat/Makefile index 6d5414c54a..49d9af9fea 100644 --- a/sysdeps/ieee754/ldbl-128ibm-compat/Makefile +++ b/sysdeps/ieee754/ldbl-128ibm-compat/Makefile @@ -251,10 +251,18 @@ endif ifeq ($(subdir),misc) ldbl-extra-routines += err \ error \ - syslog + syslog \ + qefgcvt \ + qefgcvt_r + +CFLAGS-ieee128-qefgcvt.c += -mabi=ieeelongdouble -Wno-psabi -mno-gnu-attribute +CFLAGS-ieee128-qefgcvt_r.c += -mabi=ieeelongdouble -Wno-psabi -mno-gnu-attribute tests-internal += tst-ibm128-warn tst-ieee128-warn tests-internal += tst-ibm128-error tst-ieee128-error +tests-internal += tst-ibm128-efgcvt tst-ieee128-efgcvt + +$(objpfx)tst-ieee128-efgcvt: gnulib-tests += $(f128-loader-link) $(objpfx)tst-ibm128-%.c: tst-ldbl-%.c cp $< $@ @@ -264,9 +272,11 @@ $(objpfx)tst-ieee128-%.c: tst-ldbl-%.c CFLAGS-tst-ibm128-warn.c += -mabi=ibmlongdouble -Wno-psabi CFLAGS-tst-ibm128-error.c += -mabi=ibmlongdouble -Wno-psabi +CFLAGS-tst-ibm128-efgcvt.c += -mabi=ibmlongdouble -Wno-psabi CFLAGS-tst-ieee128-warn.c += -mfloat128 -mabi=ieeelongdouble -Wno-psabi CFLAGS-tst-ieee128-error.c += -mfloat128 -mabi=ieeelongdouble -Wno-psabi +CFLAGS-tst-ieee128-efgcvt.c += -mfloat128 -mabi=ieeelongdouble -Wno-psabi tests-container += test-syslog-ieee128 test-syslog-ibm128 CFLAGS-test-syslog-ieee128.c += -mfloat128 -mabi=ieeelongdouble -Wno-psabi diff --git a/sysdeps/ieee754/ldbl-128ibm-compat/Versions b/sysdeps/ieee754/ldbl-128ibm-compat/Versions index da8e0ce7ee..b4609c99f7 100644 --- a/sysdeps/ieee754/ldbl-128ibm-compat/Versions +++ b/sysdeps/ieee754/ldbl-128ibm-compat/Versions @@ -224,5 +224,11 @@ libc { __strfmonieee128; __strfmon_lieee128; + + __qecvtieee128; + __qfcvtieee128; + __qgcvtieee128; + __qecvtieee128_r; + __qfcvtieee128_r; } } diff --git a/sysdeps/ieee754/ldbl-128ibm-compat/ieee128-qefgcvt.c b/sysdeps/ieee754/ldbl-128ibm-compat/ieee128-qefgcvt.c new file mode 100644 index 0000000000..1e9fd16db2 --- /dev/null +++ b/sysdeps/ieee754/ldbl-128ibm-compat/ieee128-qefgcvt.c @@ -0,0 +1,52 @@ +/* IEEE binary128 versions of *cvt functions. + Copyright (C) 2018 Free Software Foundation, Inc. + This file is part of the GNU C Library. + + The GNU C Library is free software; you can redistribute it and/or + modify it under the terms of the GNU Lesser General Public + License as published by the Free Software Foundation; either + version 2.1 of the License, or (at your option) any later version. + + The GNU C Library is distributed in the hope that it will be useful, + but WITHOUT ANY WARRANTY; without even the implied warranty of + MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU + Lesser General Public License for more details. + + You should have received a copy of the GNU Lesser General Public + License along with the GNU C Library; if not, see + . */ + +/* When in IEEE long double mode, call ___ieee128_sprintf. */ +#include +typeof (sprintf) ___ieee128_sprintf attribute_hidden; +#define SPRINTF ___ieee128_sprintf + +/* Declare internal functions: ___qecvtieee128_r and ___qfcvtieee128_r, + built from a different compiling unit, and called from here. */ +#include +typeof (qecvt_r) ___qecvtieee128_r; +typeof (qfcvt_r) ___qfcvtieee128_r; + +/* Rename the static buffers and pointer, otherwise the IEEE long double + variants of qecvt and qfcvt would reuse the same buffers and pointer + as their non-IEEE long double counterparts. */ +#define qecvt_buffer qecvtieee128_buffer +#define qfcvt_buffer qfcvtieee128_buffer +#define qfcvt_bufptr qfcvtieee128_bufptr + +#define ECVT __qecvtieee128 +#define FCVT __qfcvtieee128 +#define GCVT __qgcvtieee128 +#define __ECVT ___qecvtieee128 +#define __FCVT ___qfcvtieee128 +#define __GCVT ___qgcvtieee128 +#define __ECVT_R ___qecvtieee128_r +#define __FCVT_R ___qfcvtieee128_r +#include +#include + +#define cvt_symbol(local, symbol) \ + strong_alias (local, symbol) +cvt_symbol (___qfcvtieee128, __qfcvtieee128); +cvt_symbol (___qecvtieee128, __qecvtieee128); +cvt_symbol (___qgcvtieee128, __qgcvtieee128); diff --git a/sysdeps/ieee754/ldbl-128ibm-compat/ieee128-qefgcvt_r.c b/sysdeps/ieee754/ldbl-128ibm-compat/ieee128-qefgcvt_r.c new file mode 100644 index 0000000000..1a5e43972f --- /dev/null +++ b/sysdeps/ieee754/ldbl-128ibm-compat/ieee128-qefgcvt_r.c @@ -0,0 +1,34 @@ +/* IEEE binary128 versions of reentrant *cvt_r functions. + Copyright (C) 2018 Free Software Foundation, Inc. + This file is part of the GNU C Library. + + The GNU C Library is free software; you can redistribute it and/or + modify it under the terms of the GNU Lesser General Public + License as published by the Free Software Foundation; either + version 2.1 of the License, or (at your option) any later version. + + The GNU C Library is distributed in the hope that it will be useful, + but WITHOUT ANY WARRANTY; without even the implied warranty of + MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU + Lesser General Public License for more details. + + You should have received a copy of the GNU Lesser General Public + License along with the GNU C Library; if not, see + . */ + +/* When in IEEE long double mode, call ___ieee128_snprintf. */ +#include +typeof (snprintf) ___ieee128_snprintf attribute_hidden; +#define SNPRINTF ___ieee128_snprintf + +#define ECVT_R __qecvtieee128_r +#define FCVT_R __qfcvtieee128_r +#define __ECVT_R ___qecvtieee128_r +#define __FCVT_R ___qfcvtieee128_r +#include +#include + +#define cvt_symbol(local, symbol) \ + strong_alias (local, symbol) +cvt_symbol (___qfcvtieee128_r, __qfcvtieee128_r); +cvt_symbol (___qecvtieee128_r, __qecvtieee128_r);