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