From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: by sourceware.org (Postfix, from userid 1105) id BDB1C3858D35; Thu, 5 Jan 2023 00:03:11 +0000 (GMT) DKIM-Filter: OpenDKIM Filter v2.11.0 sourceware.org BDB1C3858D35 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=sourceware.org; s=default; t=1672876991; bh=jJXou9JqsYrNyVMfxj/5liJzicwbpnwyX0ieOpVm8k8=; h=From:To:Subject:Date:From; b=fSglhDTVuenAbGKU4V9Jn84wgbJsMqOdKaodqdatgy9XCFvteFRSDdeVyNmv+27lL eB7OZJt22ZS8P6P+nutNmrQqFGE/UW4KTAXTiWd3JGbD0UYdeVrFVlpW1TQ7Qpmprv xQg7cjwC3S8xU9Har6EEjGRSGkGOFEvBRXmzh2E8= Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit From: Joseph Myers To: glibc-cvs@sourceware.org Subject: [glibc] Fix ldbl-128 built-in function use X-Act-Checkin: glibc X-Git-Author: Joseph Myers X-Git-Refname: refs/heads/master X-Git-Oldrev: 48b74865c63840b288bd85b4d8743533b73b339b X-Git-Newrev: 8f27dc1af5238adbc14000f073178ee4a2c2b3cf Message-Id: <20230105000311.BDB1C3858D35@sourceware.org> Date: Thu, 5 Jan 2023 00:03:11 +0000 (GMT) List-Id: https://sourceware.org/git/gitweb.cgi?p=glibc.git;h=8f27dc1af5238adbc14000f073178ee4a2c2b3cf commit 8f27dc1af5238adbc14000f073178ee4a2c2b3cf Author: Joseph Myers Date: Thu Jan 5 00:02:54 2023 +0000 Fix ldbl-128 built-in function use Fix the following issues with built-in function use in sysdeps/ieee754/ldbl-128 and sysdeps/ieee754/float128: * fabsl used __builtin_fabsf128 unconditionally, breaking the build with GCC 6 for several architectures; it should use __builtin_fabsl with an appropriate redirection in float128_private.h. (I'm not particularly concerned with building glibc with GCC 6; rather, I want to be able to run the tgmath.h tests with GCC 6, which is a significantly different case for tgmath.h compared to GCC 7 and later because of the lack of _FloatN / _FloatNx support in the compiler, and at present running the tests with a compiler means building glibc with that compiler.) * Some (conditional) uses of built-in functions had been added to ldbl-128 without appropriate float128_private.h remapping (there was remapping for the macros controlling whether the built-in functions are used, just not for the functions themselves). * s_llrintl.c called __builtin_round not __builtin_llrintl, which is obviously wrong. Tested with build-many-glibcs.py for aarch64-linux-gnu, GCC 6 (where it fixes the glibc build) and GCC 12, and with the glibc testsuite for x86_64. Diff: --- sysdeps/ieee754/float128/float128_private.h | 4 ++++ sysdeps/ieee754/ldbl-128/s_fabsl.c | 2 +- sysdeps/ieee754/ldbl-128/s_llrintl.c | 2 +- 3 files changed, 6 insertions(+), 2 deletions(-) diff --git a/sysdeps/ieee754/float128/float128_private.h b/sysdeps/ieee754/float128/float128_private.h index 78107b1525..b786488389 100644 --- a/sysdeps/ieee754/float128/float128_private.h +++ b/sysdeps/ieee754/float128/float128_private.h @@ -406,7 +406,11 @@ #define __builtin_ceill __builtin_ceilf128 #define __builtin_truncl __builtin_truncf128 #define __builtin_roundl __builtin_roundf128 +#define __builtin_roundevenl __builtin_roundevenf128 +#define __builtin_lrintl __builtin_lrintf128 +#define __builtin_llrintl __builtin_llrintf128 #define __builtin_copysignl __builtin_copysignf128 +#define __builtin_fabsl __builtin_fabsf128 /* Get the constant suffix from bits/floatn-compat.h. */ #define L(x) __f128 (x) diff --git a/sysdeps/ieee754/ldbl-128/s_fabsl.c b/sysdeps/ieee754/ldbl-128/s_fabsl.c index f22515fec6..2e5ea3f413 100644 --- a/sysdeps/ieee754/ldbl-128/s_fabsl.c +++ b/sysdeps/ieee754/ldbl-128/s_fabsl.c @@ -27,6 +27,6 @@ static char rcsid[] = "$NetBSD: $"; _Float128 __fabsl (_Float128 x) { - return __builtin_fabsf128 (x); + return __builtin_fabsl (x); } libm_alias_ldouble (__fabs, fabs) diff --git a/sysdeps/ieee754/ldbl-128/s_llrintl.c b/sysdeps/ieee754/ldbl-128/s_llrintl.c index 3a5635d509..22e6e5f812 100644 --- a/sysdeps/ieee754/ldbl-128/s_llrintl.c +++ b/sysdeps/ieee754/ldbl-128/s_llrintl.c @@ -31,7 +31,7 @@ long long int __llrintl (_Float128 x) { #if USE_LLRINTL_BUILTIN - return __builtin_round (x); + return __builtin_llrintl (x); #else /* Use generic implementation. */ static const _Float128 two112[2] =