Message ID | 2530234b-3446-d3e0-7f32-356c362f0a7b@linux.vnet.ibm.com |
---|---|
State | New |
Headers | show |
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;
Index: gcc/testsuite/gcc.target/powerpc/pr78691-ppc.c =================================================================== --- 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;