On Sat, 1 Sep 2012, Marc Glisse wrote:

I have some issues with the vector-compare-2.c torture test. It passes a vector by value (argument and return type), which is likely to warn (although for some reason it doesn't for me, with today's compiler). And it takes -Wno-psabi through a .x file, but those are not read in c-c++-common, so I put it in dg-options. I would have changed the function to use pointers, but I don't know if it specifically wants to test passing by value...

Hello,

Pat Haugen just explained it to me: there was a default -w option in the old location of the testcase. However, there isn't in the new one. I am thus proposing the following, which restores the old behavior. Note that -Wno-psabi is still needed as it is not implied by -w.

2012-09-20  Marc Glisse  <marc.gli...@inria.fr>

        PR c++/54427
        * c-c++-common/torture/vector-compare-2.c: Add -w.

--
Marc Glisse
Index: c-c++-common/torture/vector-compare-2.c
===================================================================
--- c-c++-common/torture/vector-compare-2.c     (revision 191585)
+++ c-c++-common/torture/vector-compare-2.c     (working copy)
@@ -1,12 +1,12 @@
 /* { dg-do run } */
-/* { dg-options "-Wno-psabi" } */
+/* { dg-options "-Wno-psabi -w" } */
 #define vector(elcount, type)  \
 __attribute__((vector_size((elcount)*sizeof(type)))) type
 
 /* Check that constant folding in 
    these simple cases works.  */
 vector (4, int)
 foo (vector (4, int) x)
 {
   return   (x == x) + (x != x) + (x >  x) 
         + (x <  x) + (x >= x) + (x <= x);

Reply via email to