This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH, rs6000, committed] Fix PR81143
- From: Peter Bergner <bergner at vnet dot ibm dot com>
- To: GCC Patches <gcc-patches at gcc dot gnu dot org>
- Cc: Segher Boessenkool <segher at kernel dot crashing dot org>
- Date: Mon, 2 Apr 2018 10:26:00 -0500
- Subject: Re: [PATCH, rs6000, committed] Fix PR81143
- References: <5f11d26b-de6e-e8fe-3d79-6793925cb090@vnet.ibm.com>
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