Re: [PATCH] Fix testcase to not use vtable verification with LTO

[Richard Biener <richard.guenther@gmail.com>]

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
> cmtice@google.com
>
> ChangeLog entry  (gcc/testsuite/ChangeLog):
>
> 2019-09-05  Caroline Tice  <cmtice@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)