[testsuite] fix testsuite/41659, target or xfail with dg-lto-do

Richard Guenther richard.guenther@gmail.com
Mon Oct 12 20:03:00 GMT 2009


On Mon, Oct 12, 2009 at 7:19 PM, Janis Johnson <janis187@us.ibm.com> wrote:
> The test directive dg-lto-do does not support target and xfail causes
> and ignores them.  That behavior is documented in the section on LTO
> testing in the GCC internals manual, but there was no warning or error
> about it.  This patch fixes a test that tries to skip a test with a
> "target" clause and reports an error for using "target" or "xfail" with
> that directive.
>
> Tested on powerpc64-linux by running lto.exp for c,c++,fortran, checked
> in on trunk.
>
> 2009-10-12  Janis Johnson  <janis187@us.ibm.com>
>
>        PR testsuite/41659
>        * gcc.dg/lto/20090914-2.c: Use dg-skip-if to skip test.
>        * lib/lto.exp (lto-get-options-main): Report error for using "target"
>        or "xfail" with dg-lto-dg.
>
> Index: gcc/testsuite/gcc.dg/lto/20090914-2_0.c
> ===================================================================
> --- gcc/testsuite/gcc.dg/lto/20090914-2_0.c     (revision 152593)
> +++ gcc/testsuite/gcc.dg/lto/20090914-2_0.c     (working copy)
> @@ -1,4 +1,5 @@
> -/* { dg-lto-do run { target x86_64-*-* i?86-*-* } } */
> +/* { dg-lto-do run */

Hmm, I miss a } here and

> +/* { dg-skip-if "x86 only" { ! { x86_64-*-* i?86-*-* } } { "*" } { "" } }

a */ here.

But maybe you noticed and checked in sth different.

Thanks btw for fixing this.

Richard.



More information about the Gcc-patches mailing list