> -----Original Message-----
> From: [email protected] [mailto:gcc-patches-
> [email protected]] On Behalf Of Rainer Orth
> Sent: Wednesday, February 5, 2014 4:22 PM
> To: Iyer, Balaji V
> Cc: Jakub Jelinek; Paolo Carlini; [email protected]
> Subject: Re: regression test issue
>
> "Iyer, Balaji V" <[email protected]> writes:
>
> > Index: gcc/testsuite/g++.dg/cilk-plus/CK/catch_exc.cc
> >
> ==========================================================
> =========
> > --- gcc/testsuite/g++.dg/cilk-plus/CK/catch_exc.cc (revision 207437)
> > +++ gcc/testsuite/g++.dg/cilk-plus/CK/catch_exc.cc (working copy)
> > @@ -1,6 +1,7 @@
> > /* { dg-options "-fcilkplus" } */
> > /* { dg-do run { target i?86-*-* x86_64-*-* arm*-*-* } } */
> > /* { dg-options "-fcilkplus -lcilkrts" { target { i?86-*-* x86_64-*-*
> > arm*-*-* } } } */
> > +/* { dg-skip-if "" { *-*-* } { "-O1" } { "" } } */
> >
> > #include <assert.h>
> > #include <unistd.h>
> > Index: gcc/testsuite/c-c++-common/cilk-plus/CK/spawner_inline.c
> >
> ==========================================================
> =========
> > --- gcc/testsuite/c-c++-common/cilk-plus/CK/spawner_inline.c (revision
> 207437)
> > +++ gcc/testsuite/c-c++-common/cilk-plus/CK/spawner_inline.c (working
> copy)
> > @@ -1,6 +1,7 @@
> > /* { dg-do run { target { i?86-*-* x86_64-*-* } } } */
> > /* { dg-options "-fcilkplus" } */
> > /* { dg-additional-options "-lcilkrts" { target { i?86-*-* x86_64-*-*
> > } } } */
> > +/* { dg-skip-if "" { *-*-* } { "-O1" } { "" } } */
>
> You should put a comment (e.g. a PR reference) in the comment field to
> explain why you're skipping this test. Most likely, the last arg to
> dg-skip-if ({
> "" }) is optional; if so, please omit it.
There isn't a PR reference for this. I can add a comment in the testcase. Would
that be enough?
I will also remove the last {""} from the tests.
Thanks,
Balaji V. Iyer.
>
> Thanks.
> Rainer
>
> --
> -----------------------------------------------------------------------------
> Rainer Orth, Center for Biotechnology, Bielefeld University