From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 124464 invoked by alias); 5 Sep 2019 16:13:15 -0000 Mailing-List: contact gcc-patches-help@gcc.gnu.org; run by ezmlm Precedence: bulk List-Id: List-Archive: List-Post: List-Help: Sender: gcc-patches-owner@gcc.gnu.org Received: (qmail 124455 invoked by uid 89); 5 Sep 2019 16:13:15 -0000 Authentication-Results: sourceware.org; auth=none X-Spam-SWARE-Status: No, score=-19.9 required=5.0 tests=AWL,BAYES_00,ENV_AND_HDR_SPF_MATCH,GIT_PATCH_2,GIT_PATCH_3,RCVD_IN_DNSWL_NONE,SPF_PASS,USER_IN_DEF_SPF_WL autolearn=ham version=3.3.1 spammy= X-HELO: mail-wm1-f47.google.com Received: from mail-wm1-f47.google.com (HELO mail-wm1-f47.google.com) (209.85.128.47) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with ESMTP; Thu, 05 Sep 2019 16:13:13 +0000 Received: by mail-wm1-f47.google.com with SMTP id t9so3756464wmi.5 for ; Thu, 05 Sep 2019 09:13:11 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=20161025; h=mime-version:from:date:message-id:subject:to; bh=vRXlXnMDO/kvGI6HgmCLrmwUHbOkhKecgYEJpWRAaZs=; b=Ho4ktocG2imksy0miAHGdlCNlgqzvTPGianeUHpmF70EBA6d9+0BGYEufm6bpo47rm S7sMmJa/EX3ansJhcmWeBws+5IbyLVqrhcbB7dlc+GwCyXtXs0cq6Kl9o+a1oyH3tzbj g20VXAg8cPoRsTRKjDFl9zZpCnOmSf4uEbkOrCg4qQl3tANK65K9XrGyWBfKLB/6HxaC tud7tDiYCbSezty1/eUOnFWou38VSwMhuzF2I7e+eUPUaTwcgVSd/iuZSyURDgHN+jik yDUnx5Vv/Qa2gs1tFTbuFvwmx4TX7wvis0+27W9VWPpdmmRVrxUWlnNOsyGg8Usnopsb RqsA== MIME-Version: 1.0 From: "Caroline Tice via gcc-patches" Reply-To: Caroline Tice Date: Thu, 05 Sep 2019 16:13:00 -0000 Message-ID: Subject: [PATCH] Fix testcase to not use vtable verification with LTO To: GCC Patches Content-Type: text/plain; charset="UTF-8" X-IsSubscribed: yes X-SW-Source: 2019-09/txt/msg00309.txt.bz2 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? -- Caroline cmtice@google.com ChangeLog entry (gcc/testsuite/ChangeLog): 2019-09-05 Caroline Tice 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)