diff mbox

[rs6000] Fix gcc.target/powerpc/pr78691-ppc.c

Message ID 2530234b-3446-d3e0-7f32-356c362f0a7b@linux.vnet.ibm.com
State New
Headers show

Commit Message

Bill Schmidt Dec. 9, 2016, 3:39 p.m. UTC 
Hi,

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?

Thanks,
Bill


2016-12-09  Bill Schmidt  <wschmidt@linux.vnet.ibm.com>

	* gcc.target/powerpc/pr78691-ppc.c: Restrict test case to 32-bit
	targets.

Comments

Segher Boessenkool Dec. 9, 2016, 4:01 p.m. UTC | #1 
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;
diff mbox

Patch

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;