public inbox for glibc-cvs@sourceware.org
help / color / mirror / Atom feed
* [glibc/google/grte/v5-2.27/master] Do not use ppc-specific long double pack/unpack when compiling with clang
@ 2021-08-28  0:37 Fangrui Song
  0 siblings, 0 replies; 2+ messages in thread
From: Fangrui Song @ 2021-08-28  0:37 UTC (permalink / raw)
  To: glibc-cvs

https://sourceware.org/git/gitweb.cgi?p=glibc.git;h=2c9e5207e426910c6c61c8bf3d31844b6f9aa0b1

commit 2c9e5207e426910c6c61c8bf3d31844b6f9aa0b1
Author: Stan Shebs <stanshebs@google.com>
Date:   Tue Jun 9 10:05:56 2020 -0700

    Do not use ppc-specific long double pack/unpack when compiling with clang

Diff:
---
 sysdeps/powerpc/fpu/math_ldbl.h | 5 +++++
 1 file changed, 5 insertions(+)

diff --git a/sysdeps/powerpc/fpu/math_ldbl.h b/sysdeps/powerpc/fpu/math_ldbl.h
index 124391b94b..0867403d94 100644
--- a/sysdeps/powerpc/fpu/math_ldbl.h
+++ b/sysdeps/powerpc/fpu/math_ldbl.h
@@ -47,8 +47,13 @@ ldbl_unpack_ppc (long double l, double *a, double *aa)
   *aa = xl;
 }
 
+/* These inline functions do not work with clang at all; drop back to the
+   default versions.  (The defaults are not inlined by clang either, but they
+   return correct values at least.)*/
+#if !defined __clang__
 #define ldbl_pack   ldbl_pack_ppc
 #define ldbl_unpack ldbl_unpack_ppc
+#endif /* !__clang__ */
 
 #include <sysdeps/ieee754/ldbl-128ibm/math_ldbl.h>


^ permalink raw reply	[flat|nested] 2+ messages in thread
* [glibc/google/grte/v5-2.27/master] Do not use ppc-specific long double pack/unpack when compiling with clang
@ 2020-06-09 17:06 Stan Shebs
  0 siblings, 0 replies; 2+ messages in thread
From: Stan Shebs @ 2020-06-09 17:06 UTC (permalink / raw)
  To: glibc-cvs

https://sourceware.org/git/gitweb.cgi?p=glibc.git;h=7e9285612d997d7cface8141ad4db191758864c4

commit 7e9285612d997d7cface8141ad4db191758864c4
Author: Stan Shebs <stanshebs@google.com>
Date:   Tue Jun 9 10:05:56 2020 -0700

    Do not use ppc-specific long double pack/unpack when compiling with clang

Diff:
---
 sysdeps/powerpc/fpu/math_ldbl.h | 5 +++++
 1 file changed, 5 insertions(+)

diff --git a/sysdeps/powerpc/fpu/math_ldbl.h b/sysdeps/powerpc/fpu/math_ldbl.h
index 124391b94b..0867403d94 100644
--- a/sysdeps/powerpc/fpu/math_ldbl.h
+++ b/sysdeps/powerpc/fpu/math_ldbl.h
@@ -47,8 +47,13 @@ ldbl_unpack_ppc (long double l, double *a, double *aa)
   *aa = xl;
 }
 
+/* These inline functions do not work with clang at all; drop back to the
+   default versions.  (The defaults are not inlined by clang either, but they
+   return correct values at least.)*/
+#if !defined __clang__
 #define ldbl_pack   ldbl_pack_ppc
 #define ldbl_unpack ldbl_unpack_ppc
+#endif /* !__clang__ */
 
 #include <sysdeps/ieee754/ldbl-128ibm/math_ldbl.h>


^ permalink raw reply	[flat|nested] 2+ messages in thread
end of thread, other threads:[~2021-08-28  0:37 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-08-28  0:37 [glibc/google/grte/v5-2.27/master] Do not use ppc-specific long double pack/unpack when compiling with clang Fangrui Song
  -- strict thread matches above, loose matches on Subject: below --
2020-06-09 17:06 Stan Shebs

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).