http://gcc.gnu.org/bugzilla/show_bug.cgi?id=56933

Jakub Jelinek <jakub at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |jakub at gcc dot gnu.org

--- Comment #5 from Jakub Jelinek <jakub at gcc dot gnu.org> ---
(In reply to Bernd Edlinger from comment #4)
> The test case gcc.dg/vect/pr56933.c fails on a pentium II,
> because of invalid instruction.
> 
> A fairly obvious fix (which works for me) would be:

There is nothing obvious on it (maybe obviously broken).
The vect.exp tests aren't i?86/x86_64 only, so you definitely can't add
unguarded requirements for specific CPUs, it would cause the test not to be run
on ppcs/arm/etc.

The bug in this test is that it specifies { dg-do run }, either it shouldn't
(then the default will be run if CPU supports the -msse2 flags in this case, or
compile otherwise), or it should use the tree-vect.h + check_vect () framework
like many other tests.

> --- gcc/testsuite/gcc.dg/vect/pr56933.c.jj      2013-04-15
> 16:08:41.000000000 +0200
> +++ gcc/testsuite/gcc.dg/vect/pr56933.c 2013-08-22 21:12:19.000000000 +0200
> @@ -1,4 +1,5 @@
>  /* { dg-do run } */
> +/* { dg-require-effective-target sse2_runtime } */
> 
>  extern void abort (void);
>  void __attribute__((noinline,noclone))

Reply via email to