From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 95232 invoked by alias); 21 Feb 2019 19:22:19 -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 95210 invoked by uid 89); 21 Feb 2019 19:22:18 -0000 Authentication-Results: sourceware.org; auth=none X-Spam-SWARE-Status: No, score=-1.9 required=5.0 tests=BAYES_00,SPF_PASS autolearn=ham version=3.3.2 spammy= X-HELO: mx1.suse.de Received: from mx2.suse.de (HELO mx1.suse.de) (195.135.220.15) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with ESMTP; Thu, 21 Feb 2019 19:22:17 +0000 Received: from relay2.suse.de (unknown [195.135.220.254]) by mx1.suse.de (Postfix) with ESMTP id 7D012B15D; Thu, 21 Feb 2019 19:22:15 +0000 (UTC) Date: Thu, 21 Feb 2019 19:50:00 -0000 User-Agent: K-9 Mail for Android In-Reply-To: <1043f46b-20ff-0d20-2cc3-27f4300f465d@redhat.com> References: <1043f46b-20ff-0d20-2cc3-27f4300f465d@redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable Subject: Re: [PATCH] Remove unused extern variables from debug (PR debug/86964) To: Jeff Law ,Johan Carlsson ,"gcc-patches@gcc.gnu.org" From: Richard Biener Message-ID: X-SW-Source: 2019-02/txt/msg01760.txt.bz2 On February 21, 2019 8:06:05 PM GMT+01:00, Jeff Law wrote: >On 2/19/19 5:51 AM, Johan Carlsson wrote: >> Adding so that the DIEs of unused external variables can be removed >from the output when >> using -feliminate-unused-debug-symbols. >>=20 >> Tested on x86_64-elf, aarch64-elf, powerpc-eabi, powerpc64-elf and >arm-eabi. >>=20 >>=20 >> 2019-02-18 Johan Karlsson >>=20 >> PR debug/86964 >> * dwarf2out.c (premark_used_variables): New function. >> (prune_unused_types_walk): Do not mark not premarked external >> variables. >> (prune_unused_types): Call premark_used_variables. >>=20 >> * gcc.dg/debug/dwarf2/pr86964.c: New testcase. >Just a note, we're addressing regression bugfixes only at this point in >GCC development. I've queued this for gcc-10. Note that technically this is a regression from pre-early-debug.=20 Richard.=20 >jeff