This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH, rs6000] Fix gcc.target/powerpc/pr78691-ppc.c
- From: Segher Boessenkool <segher at kernel dot crashing dot org>
- To: Bill Schmidt <wschmidt at linux dot vnet dot ibm dot com>
- Cc: GCC Patches <gcc-patches at gcc dot gnu dot org>, David Edelsohn <dje dot gcc at gmail dot com>
- Date: Fri, 9 Dec 2016 10:01:22 -0600
- Subject: Re: [PATCH, rs6000] Fix gcc.target/powerpc/pr78691-ppc.c
- Authentication-results: sourceware.org; auth=none
- References: <2530234b-3446-d3e0-7f32-356c362f0a7b@linux.vnet.ibm.com>
Hi Bill,
On Fri, Dec 09, 2016 at 09:39:27AM -0600, Bill Schmidt wrote:
> The fix for PR78691 added a new test case for 32-bit PowerPC. However, the
> test needs to be restricted to 32-bit targets to avoid a FAIL. This patch
> fixes that.
>
> Tested on powerpc64-unknown-linux-gnu. Ok for trunk?
Could you instead just remove the -m32? There is nothing in the testcase
that requires it, AFAICS. There are a few more testcases with this btw,
if you feel like it ;-)
Segher
> --- gcc/testsuite/gcc.target/powerpc/pr78691-ppc.c (revision 243488)
> +++ gcc/testsuite/gcc.target/powerpc/pr78691-ppc.c (working copy)
> @@ -1,4 +1,5 @@
> /* PR tree-optimization/78691 */
> +/* { dg-do compile { target { powerpc*-*-* && ilp32 } } } */
> /* { dg-options "-Os -m32" } */
>
> int *b;