diff mbox series

[rs6000,committed] Fix PR81143

Message ID 5f11d26b-de6e-e8fe-3d79-6793925cb090@vnet.ibm.com
State New
Headers show
Series [rs6000,committed] Fix PR81143 | expand

Commit Message

Peter Bergner Feb. 8, 2018, 8:44 p.m. UTC
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__.

Comments

Peter Bergner April 2, 2018, 3:26 p.m. UTC | #1
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
diff mbox series

Patch

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