* [PATCH, rs6000, committed] Fix PR81143
@ 2018-02-08 20:44 Peter Bergner
2018-04-02 15:26 ` Peter Bergner
0 siblings, 1 reply; 2+ messages in thread
From: Peter Bergner @ 2018-02-08 20:44 UTC (permalink / raw)
To: GCC Patches; +Cc: Segher Boessenkool
I have committed the following obvious testsuite patch to fix PR81143.
The "bug" is that __ORDER_LITTLE_ENDIAN__ is always defined for both
little and big endian compiles. I checked and this is the only use
of this in the gcc.target/powerpc/ directory.
Peter
PR target/81143
* gcc.target/powerpc/pr79799-2.c: Use __LITTLE_ENDIAN__.
Index: gcc/testsuite/gcc.target/powerpc/pr79799-2.c
===================================================================
--- gcc/testsuite/gcc.target/powerpc/pr79799-2.c (revision 257503)
+++ gcc/testsuite/gcc.target/powerpc/pr79799-2.c (revision 257504)
@@ -8,7 +8,7 @@
/* Optimize x = vec_insert (vec_extract (v2, N), v1, M) for SFmode if N is the default
scalar position. */
-#if __ORDER_LITTLE_ENDIAN__
+#if __LITTLE_ENDIAN__
#define ELE 2
#else
#define ELE 1
^ permalink raw reply [flat|nested] 2+ messages in thread
* Re: [PATCH, rs6000, committed] Fix PR81143
2018-02-08 20:44 [PATCH, rs6000, committed] Fix PR81143 Peter Bergner
@ 2018-04-02 15:26 ` Peter Bergner
0 siblings, 0 replies; 2+ messages in thread
From: Peter Bergner @ 2018-04-02 15:26 UTC (permalink / raw)
To: GCC Patches; +Cc: Segher Boessenkool
On 2/8/18 2:44 PM, Peter Bergner wrote:
> I have committed the following obvious testsuite patch to fix PR81143.
> The "bug" is that __ORDER_LITTLE_ENDIAN__ is always defined for both
> little and big endian compiles. I checked and this is the only use
> of this in the gcc.target/powerpc/ directory.
>
> Peter
>
> PR target/81143
> * gcc.target/powerpc/pr79799-2.c: Use __LITTLE_ENDIAN__.
>
> Index: gcc/testsuite/gcc.target/powerpc/pr79799-2.c
> ===================================================================
> --- gcc/testsuite/gcc.target/powerpc/pr79799-2.c (revision 257503)
> +++ gcc/testsuite/gcc.target/powerpc/pr79799-2.c (revision 257504)
> @@ -8,7 +8,7 @@
> /* Optimize x = vec_insert (vec_extract (v2, N), v1, M) for SFmode if N is the default
> scalar position. */
>
> -#if __ORDER_LITTLE_ENDIAN__
> +#if __LITTLE_ENDIAN__
> #define ELE 2
> #else
> #define ELE 1
I seem to have missed backporting this to GCC 7, where the same bug exists, so
I just committed the fix there too.
Peter
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2018-04-02 15:26 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-02-08 20:44 [PATCH, rs6000, committed] Fix PR81143 Peter Bergner
2018-04-02 15:26 ` Peter Bergner
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).