From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 32941 invoked by alias); 21 Nov 2017 02:46:28 -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 32926 invoked by uid 89); 21 Nov 2017 02:46:28 -0000 Authentication-Results: sourceware.org; auth=none X-Virus-Found: No X-Spam-SWARE-Status: No, score=-3.2 required=5.0 tests=AWL,BAYES_00,KAM_SHORT,KB_WAM_FROM_NAME_SINGLEWORD,RCVD_IN_DNSWL_NONE,SPF_PASS autolearn=no version=3.3.2 spammy=HContent-Transfer-Encoding:8bit X-HELO: mail-pf0-f182.google.com Received: from mail-pf0-f182.google.com (HELO mail-pf0-f182.google.com) (209.85.192.182) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with ESMTP; Tue, 21 Nov 2017 02:46:26 +0000 Received: by mail-pf0-f182.google.com with SMTP id r14so2264562pfl.11 for ; Mon, 20 Nov 2017 18:46:26 -0800 (PST) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:subject:from:to:cc:references:message-id:date :user-agent:mime-version:in-reply-to:content-language :content-transfer-encoding; bh=xN0r6bU1y+2xxTcnoC8CYfrpY6+M9NRgRlL5f8/2dpg=; b=WQh81kqKQneu5yo5riFb1Gv9qQhpV7a/g85QNgYNLR4cccKLLHXVAPDK+dukqxIyMP yJigJ8wjyFS8Jz0y+eJQ+hMaOTfnxTiuEWJN2pXleL4jrQH34Lma6HaujfCBZtaA+Fbd 2uT4fOfWTN7QnBAusTdlmehi0FbvQO4FIWv4CY/dSuXmav/qGOXQQiRp1QYzsroeyb7x /+WkP7Z/8kuKtlgm0eYJFqyezZmxalGJD05n/Oj1Ta3gv0r6JSRsaAGVrLeaU6mTh3vX 1VRWoNUU425i9Eb6xlel89hNL4aW+7vB2PUJB/NBQmPNDXSL51jq33DTC/qnX8ZMuAxF Qucw== X-Gm-Message-State: AJaThX7IEvevyjO5nPHQbDpZvb3JvmXJQB9xrV/xR48jwas0duYpIx6l GnJbf3rJIY6E8wr2M/J1Odt8SIqQ3cc= X-Google-Smtp-Source: AGs4zMYeiX9cOlDe3xVQDuaM9/zixiHo2mvcZsdTpk6iOFPs+HWSGRLhdUGF1GqBp29SEQh2hWZEfA== X-Received: by 10.98.46.7 with SMTP id u7mr13524713pfu.37.1511232384588; Mon, 20 Nov 2017 18:46:24 -0800 (PST) Received: from [10.17.0.120] ([12.206.222.5]) by smtp.gmail.com with ESMTPSA id y1sm5796127pgq.41.2017.11.20.18.46.23 (version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Mon, 20 Nov 2017 18:46:24 -0800 (PST) Subject: Re: Patch ping From: Jim Wilson To: Jakub Jelinek , Jason Merrill , Nathan Sidwell Cc: gcc-patches@gcc.gnu.org References: <20171120075512.GE14653@tucnak> <7faf2079-f3d2-2b9c-5d08-107cad3eb859@sifive.com> Message-ID: Date: Tue, 21 Nov 2017 03:01:00 -0000 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.4.0 MIME-Version: 1.0 In-Reply-To: <7faf2079-f3d2-2b9c-5d08-107cad3eb859@sifive.com> Content-Type: text/plain; charset=utf-8; format=flowed Content-Transfer-Encoding: 8bit X-SW-Source: 2017-11/txt/msg01846.txt.bz2 On 11/20/2017 02:58 PM, Jim Wilson wrote: > The dwarf2out.c patch looks good to me, though the testcase does not > fail on unpatched mainline anymore.  I had to go back to the 2017-10-22 > snapshot to see the failure.  There was a followup from Mark Wielaard > mentioning that elfutils still fails on mainline, so maybe we can get a > testcase from there. The testcase was "broken" by Jan's big patch on Nov 3 https://gcc.gnu.org/ml/gcc-patches/2017-11/msg00104.html It didn't seem useful to look into that, so I tried running multidelta/creduce on the elfutils sources, and managed to get a small testcase that still fails with mainline. I then fixed it to get rid of the warnings. #include extern int c (void); int a(int b) { if (c()) abort(); } There is no guarantee that this testcase will still show the problem in the future, but it seems good enough for now. OK with this testcase. And with your testcase too if you think that still has value. Jim