> -----Original Message-----
> From: gcc-patches-ow...@gcc.gnu.org [mailto:gcc-patches-
> ow...@gcc.gnu.org] On Behalf Of Rainer Orth
> Sent: Wednesday, February 5, 2014 4:22 PM
> To: Iyer, Balaji V
> Cc: Jakub Jelinek; Paolo Carlini; gcc-patches@gcc.gnu.org
> Subject: Re: regression test issue
> 
> "Iyer, Balaji V" <balaji.v.i...@intel.com> 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

Reply via email to