* [PATCH], PR 77924, Fix PowerPC breakage on AIX
@ 2016-10-10 20:47 Michael Meissner
2016-10-11 17:14 ` Segher Boessenkool
0 siblings, 1 reply; 2+ messages in thread
From: Michael Meissner @ 2016-10-10 20:47 UTC (permalink / raw)
To: gcc-patches, Segher Boessenkool, David Edelsohn, Bill Schmidt
I accidently broke AIX with my patch on October 6th. That patch split
-mfloat128 into -mfloat128-type and -mfloat128 under PowerPC Linux. This patch
fixes that issue. I bootstrapped it on PowerPC Linux with no regressions, and
David Edelsohn reports that it fixes the problem on AIX. Is it ok to apply the
patch?
2016-10-10 Michael Meissner <meissner@linux.vnet.ibm.com>
PR target/77924
* config/rs6000/rs6000.c (rs6000_init_builtins): Only create the
distinct __ibm128 IBM extended double type if long doubles are
128-bits and the default format for long double is IEEE 128-bit.
Index: gcc/config/rs6000/rs6000.c
===================================================================
--- gcc/config/rs6000/rs6000.c (revision 240941)
+++ gcc/config/rs6000/rs6000.c (working copy)
@@ -16572,10 +16572,10 @@ rs6000_init_builtins (void)
floating point, we need make sure the type is non-zero or else self-test
fails during bootstrap.
- We don't register a built-in type for __ibm128 or __float128 if the type
- is the same as long double. Instead we add a #define for __ibm128 or
- __float128 in rs6000_cpu_cpp_builtins to long double. */
- if (TARGET_IEEEQUAD || !TARGET_LONG_DOUBLE_128)
+ We don't register a built-in type for __ibm128 if the type is the same as
+ long double. Instead we add a #define for __ibm128 in
+ rs6000_cpu_cpp_builtins to long double. */
+ if (TARGET_LONG_DOUBLE_128 && FLOAT128_IEEE_P (TFmode))
{
ibm128_float_type_node = make_node (REAL_TYPE);
TYPE_PRECISION (ibm128_float_type_node) = 128;
--
Michael Meissner, IBM
IBM, M/S 2506R, 550 King Street, Littleton, MA 01460-6245, USA
email: meissner@linux.vnet.ibm.com, phone: +1 (978) 899-4797
^ permalink raw reply [flat|nested] 2+ messages in thread
* Re: [PATCH], PR 77924, Fix PowerPC breakage on AIX
2016-10-10 20:47 [PATCH], PR 77924, Fix PowerPC breakage on AIX Michael Meissner
@ 2016-10-11 17:14 ` Segher Boessenkool
0 siblings, 0 replies; 2+ messages in thread
From: Segher Boessenkool @ 2016-10-11 17:14 UTC (permalink / raw)
To: Michael Meissner, gcc-patches, David Edelsohn, Bill Schmidt
On Mon, Oct 10, 2016 at 04:46:53PM -0400, Michael Meissner wrote:
> I accidently broke AIX with my patch on October 6th. That patch split
> -mfloat128 into -mfloat128-type and -mfloat128 under PowerPC Linux. This patch
> fixes that issue. I bootstrapped it on PowerPC Linux with no regressions, and
> David Edelsohn reports that it fixes the problem on AIX. Is it ok to apply the
> patch?
>
> 2016-10-10 Michael Meissner <meissner@linux.vnet.ibm.com>
>
> PR target/77924
> * config/rs6000/rs6000.c (rs6000_init_builtins): Only create the
> distinct __ibm128 IBM extended double type if long doubles are
> 128-bits and the default format for long double is IEEE 128-bit.
Okay, thanks!
Segher
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2016-10-11 17:14 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-10-10 20:47 [PATCH], PR 77924, Fix PowerPC breakage on AIX Michael Meissner
2016-10-11 17:14 ` Segher Boessenkool
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).