> On November 22, 2018 5:30:14 PM GMT+01:00, Jan Hubicka <hubi...@ucw.cz> wrote: > >> * g++.dg/lto/odr-2_0.C: Remove extra brace > >> > >> diff --git a/gcc/testsuite/g++.dg/lto/odr-2_0.C > >b/gcc/testsuite/g++.dg/lto/odr-2_0.C > >> index 222fa2c1db..3ebb49efa2 100644 > >> --- a/gcc/testsuite/g++.dg/lto/odr-2_0.C > >> +++ b/gcc/testsuite/g++.dg/lto/odr-2_0.C > >> @@ -1,5 +1,5 @@ > >> // { dg-lto-do link } > >> -// { dg-lto-options { { -O0 -flto } } > >> +// { dg-lto-options { -O0 -flto } } > > > >Doesn't this make the testcase to be run twice, once with -O0 and > >second > >time with -flto rather than running it once with -O0 -flto? > > Yes.
Actually it would be useful to have ODR tested with optimization on because streaming is somewhat optimization level specific. I will rework the testcases today so they do not need dg-lto-options Honza