diff mbox

Fix gcc.dg/torture/pr47968.c testcase on i?86

Message ID 20110304231020.GN30899@tyan-ft48-01.lab.bos.redhat.com
State New
Headers show

Commit Message

Jakub Jelinek March 4, 2011, 11:10 p.m. UTC
Hi!

This testcase fails on i686-linux, because of:
gcc.dg/torture/pr47968.c:6:7: note: The ABI for passing parameters with 16-byte alignment has changed in GCC 4.6
gcc.dg/torture/pr47968.c:6:7: warning: SSE vector argument without SSE enabled changes the ABI [enabled by default]

This patch fixes it by disabling warnings.  Ok?

2011-03-04  Jakub Jelinek  <jakub@redhat.com>

	* gcc.dg/torture/pr47968.c: Ignore warnings.


	Jakub

Comments

Richard Biener March 4, 2011, 11:47 p.m. UTC | #1
On Sat, Mar 5, 2011 at 12:10 AM, Jakub Jelinek <jakub@redhat.com> wrote:
> Hi!
>
> This testcase fails on i686-linux, because of:
> gcc.dg/torture/pr47968.c:6:7: note: The ABI for passing parameters with 16-byte alignment has changed in GCC 4.6
> gcc.dg/torture/pr47968.c:6:7: warning: SSE vector argument without SSE enabled changes the ABI [enabled by default]
>
> This patch fixes it by disabling warnings.  Ok?

We accept all -Wno- flags on all targets, right?  (why does -w not suffice?)

Ok.

Thanks,
Richard.

> 2011-03-04  Jakub Jelinek  <jakub@redhat.com>
>
>        * gcc.dg/torture/pr47968.c: Ignore warnings.
>
> --- gcc/testsuite/gcc.dg/torture/pr47968.c.jj   2011-03-04 19:39:16.092421074 +0100
> +++ gcc/testsuite/gcc.dg/torture/pr47968.c      2011-03-04 22:02:14.980388635 +0100
> @@ -1,4 +1,5 @@
>  /* { dg-do compile } */
> +/* { dg-options "-w -Wno-psabi" } */
>
>  typedef __attribute__ ((vector_size (16))) float float4;
>  typedef __attribute__ ((vector_size (16))) double double2;
> @@ -8,4 +9,3 @@ float foo (double2 d2)
>   float4 f4 = (float4) d2;
>   return *(float *) &f4;
>  }
> -
>
>        Jakub
>
Jakub Jelinek March 5, 2011, midnight UTC | #2
On Sat, Mar 05, 2011 at 12:47:18AM +0100, Richard Guenther wrote:
> On Sat, Mar 5, 2011 at 12:10 AM, Jakub Jelinek <jakub@redhat.com> wrote:
> > This testcase fails on i686-linux, because of:
> > gcc.dg/torture/pr47968.c:6:7: note: The ABI for passing parameters with 16-byte alignment has changed in GCC 4.6
> > gcc.dg/torture/pr47968.c:6:7: warning: SSE vector argument without SSE enabled changes the ABI [enabled by default]
> >
> > This patch fixes it by disabling warnings.  Ok?
> 
> We accept all -Wno- flags on all targets, right?  (why does -w not suffice?)

-w disables warnings, -Wpsabi stuff is inform().
Some other tests also use -w -Wno-psabi (and, -Wpsabi is in c-family/c.opt,
so available everywhere).

	Jakub
diff mbox

Patch

--- gcc/testsuite/gcc.dg/torture/pr47968.c.jj	2011-03-04 19:39:16.092421074 +0100
+++ gcc/testsuite/gcc.dg/torture/pr47968.c	2011-03-04 22:02:14.980388635 +0100
@@ -1,4 +1,5 @@ 
 /* { dg-do compile } */
+/* { dg-options "-w -Wno-psabi" } */
 
 typedef __attribute__ ((vector_size (16))) float float4;
 typedef __attribute__ ((vector_size (16))) double double2;
@@ -8,4 +9,3 @@  float foo (double2 d2)
   float4 f4 = (float4) d2;
   return *(float *) &f4;
 }
-