On Thu, Jan 26, 2017 at 09:44:14PM +0100, Thomas Schwinge wrote:
> > The following patch should fix that, ok for trunk?
> 
> Looks good to me (but I can't approve it, as you know).

Sure, I know.
> 
> One additional comment:
> 
> > --- gcc/testsuite/g++.dg/asan/pr69276.C.jj  2016-02-04 23:14:18.000000000 
> > +0100
> > +++ gcc/testsuite/g++.dg/asan/pr69276.C     2017-01-26 11:40:10.490771046 
> > +0100
> > @@ -1,6 +1,7 @@
> >  /* { dg-do run } */
> >  /* { dg-shouldfail "asan" } */
> > -/* { dg-additional-options "-O0 -fno-lto" } */
> > +/* { dg-skip-if "" { *-*-* }  { "*" } { "-O0" } } */
> > +/* { dg-additional-options "-fno-lto" } */
> 
> Probably can get rid of that "-fno-lto", too, as "-flto" is not used
> together with "-O0"?

Yeah, or:
/* { dg-skip-if "" { *-*-* } { "*" } { "-O0" } } */
/* { dg-skip-if "" { *-*-* } { "-flto" } { "" } } */

        Jakub

Reply via email to