On Thu, Sep 5, 2019 at 6:13 PM Caroline Tice via gcc-patches <gcc-patches@gcc.gnu.org> wrote: > > Yesterday I submitted a patch that disallows using vtable verfication > with LTO (they don't work > properly together), but I missed fixing the flags for one testcase. > This patch fixes that omission. > > Testing: Tescase passes with this change. > > Is this ok to commit?
OK. Richard. > -- Caroline > cmt...@google.com > > ChangeLog entry (gcc/testsuite/ChangeLog): > > 2019-09-05 Caroline Tice <cmt...@google.com> > > PR testsuite/91670 > * g++.dg/ubsan/pr59415.C: Disable LTO, since test uses > -fvtable-verify, and the two options are no longer allowed > together. > > Index: gcc/testsuite/g++.dg/ubsan/pr59415.C > =================================================================== > --- gcc/testsuite/g++.dg/ubsan/pr59415.C (revision 275387) > +++ gcc/testsuite/g++.dg/ubsan/pr59415.C (working copy) > @@ -1,5 +1,5 @@ > /* { dg-do compile } */ > -/* { dg-options "-fsanitize=null -Wall -fvtable-verify=std" } */ > +/* { dg-options "-fsanitize=null -Wall -fvtable-verify=std -fno-lto" } */ > > void > foo (void)