On Sun, Nov 23, 2014 at 07:58:30PM +0100, Jakub Jelinek wrote:
> On Sun, Nov 23, 2014 at 01:32:27PM -0500, tsaund...@mozilla.com wrote:
> > bug was already fixed, so just add the test case.
> >
> > tested this only passes with r217909 where it is fixed, ok?
> >
> > Trev
> >
> >
> > diff --git a/gcc/testsuite/gcc.dg/pr63856.c b/gcc/testsuite/gcc.dg/pr63856.c
> > new file mode 100644
> > index 0000000..8fb65c6
> > --- /dev/null
> > +++ b/gcc/testsuite/gcc.dg/pr63856.c
> > @@ -0,0 +1,14 @@
> > +/* { dg-do compile } */
> > +/* { dg-options "-O2 -fPIC" } */
> > +typedef int v2si __attribute__ ((vector_size (8)));
> > +typedef short v4hi __attribute__ ((vector_size (8)));
> > +
> > +int __attribute__ ((noinline, noclone)) f (v2si A, int N)
>
> Vector arguments or return values often result in -Wpsabi warnings,
> so you certainly want -Wno-psabi in dg-options.
> On the other side, -fPIC should be only used for { target pic },
> so better put that into dg-additional-options.
ok, thanks how about this then?
Trev
>
> > +{
> > + return ((v4hi) A)[N];
> > +}
> > +
> > +int __attribute__ ((noinline, noclone)) g (v2si A, int N)
> > +{
> > + return ((v4hi) A)[N];
> > +}
> > --
> > 2.1.3
>
> Jakub
From 1b9dc2c209614d598658579ad6faba112c64ca0f Mon Sep 17 00:00:00 2001
From: Trevor Saunders <tsaund...@mozilla.com>
Date: Sun, 23 Nov 2014 01:11:22 -0500
Subject: [PATCH] pr63856 - test case
---
gcc/testsuite/gcc.dg/pr63856.c | 15 +++++++++++++++
1 file changed, 15 insertions(+)
create mode 100644 gcc/testsuite/gcc.dg/pr63856.c
diff --git a/gcc/testsuite/gcc.dg/pr63856.c b/gcc/testsuite/gcc.dg/pr63856.c
new file mode 100644
index 0000000..6a987e2
--- /dev/null
+++ b/gcc/testsuite/gcc.dg/pr63856.c
@@ -0,0 +1,15 @@
+/* { dg-do compile } */
+/* { dg-options "-O2 -Wno-psabi" } */
+/* { dg-additional-options "-fPIC" { target fpic } } */
+typedef int v2si __attribute__ ((vector_size (8)));
+typedef short v4hi __attribute__ ((vector_size (8)));
+
+int __attribute__ ((noinline, noclone)) f (v2si A, int N)
+{
+ return ((v4hi) A)[N];
+}
+
+int __attribute__ ((noinline, noclone)) g (v2si A, int N)
+{
+ return ((v4hi) A)[N];
+}
--
2.1.3
