[PATCH] Fix up pr58508.c testcase

Richard Biener richard.guenther@gmail.com
Wed Oct 23 10:02:00 GMT 2013


On Tue, Oct 22, 2013 at 9:26 PM, Jakub Jelinek <jakub@redhat.com> wrote:
> Hi!
>
> I've noticed that this testcase fails on i686-linux.  The problem is
> that the vect.exp infrastructure is terrible and dg-options overrides
> all the required flags.  So, vect/ testcases either don't have to have
> explicit options at all (and the way to express options is through magic
> naming of the tests), or one can use dg-additional-options.  But, in this
> case you just want to pass the defaults.
>
> Regtested on x86_64-linux and i686-linux, ok for trunk?

Ok.  Btw, it handles dg-additional-options just fine I think.

Richard.

> 2013-10-22  Jakub Jelinek  <jakub@redhat.com>
>
>         * gcc.dg/vect/pr58508.c: Remove dg-options.
>
> --- gcc/testsuite/gcc.dg/vect/pr58508.c.jj      2013-10-21 09:00:49.000000000 +0200
> +++ gcc/testsuite/gcc.dg/vect/pr58508.c 2013-10-22 14:21:16.064261179 +0200
> @@ -1,5 +1,4 @@
>  /* { dg-do compile } */
> -/* { dg-options "-O2 -ftree-vectorize -fdump-tree-vect-details" } */
>
>
>  /* The GCC vectorizer generates loop versioning for the following loop
>
>         Jakub



More information about the Gcc-patches mailing list