From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from smtp-out2.suse.de (smtp-out2.suse.de [195.135.220.29]) by sourceware.org (Postfix) with ESMTPS id 0F3E83858C39 for ; Fri, 10 Sep 2021 06:02:03 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.4.1 sourceware.org 0F3E83858C39 Received: from relay2.suse.de (relay2.suse.de [149.44.160.134]) by smtp-out2.suse.de (Postfix) with ESMTP id 028F21FE48; Fri, 10 Sep 2021 06:02:02 +0000 (UTC) Received: from murzim.suse.de (murzim.suse.de [10.160.4.192]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by relay2.suse.de (Postfix) with ESMTPS id 113A4A3B88; Fri, 10 Sep 2021 06:02:01 +0000 (UTC) Date: Fri, 10 Sep 2021 08:02:00 +0200 (CEST) From: Richard Biener To: Jeff Law cc: gcc-patches@gcc.gnu.org, dj@delorie.com Subject: Re: [PATCH] Remove dbx.h, do not set PREFERRED_DEBUGGING_TYPE from dbxcoff.h, lynx.h In-Reply-To: Message-ID: References: <49oqn7p0-n811-366q-p55q-1s9pq9ss8n3@fhfr.qr> User-Agent: Alpine 2.21 (LSU 202 2017-01-01) MIME-Version: 1.0 X-Spam-Status: No, score=-4.9 required=5.0 tests=BAYES_00, DKIM_SIGNED, DKIM_VALID, DKIM_VALID_AU, DKIM_VALID_EF, SPF_HELO_NONE, SPF_PASS, TXREP autolearn=ham autolearn_force=no version=3.4.4 X-Spam-Checker-Version: SpamAssassin 3.4.4 (2020-01-24) on server2.sourceware.org Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 8BIT X-Content-Filtered-By: Mailman/MimeDel 2.1.29 X-BeenThere: gcc-patches@gcc.gnu.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Gcc-patches mailing list List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Fri, 10 Sep 2021 06:02:04 -0000 On Thu, 9 Sep 2021, Jeff Law wrote: > > > On 9/9/2021 7:19 AM, Richard Biener via Gcc-patches wrote: > > The following removes the unused config/dbx.h file and removes the > > setting of PREFERRED_DEBUGGING_TYPE from dbxcoff.h which is > > overridden by all users (djgpp/mingw/cygwin) via either including > > config/i386/djgpp.h or config/i386/cygming.h > > > > There are still circumstances where mingw and cygwin default to > > STABS, namely when HAVE_GAS_PE_SECREL32_RELOC is not defined and > > the target defaults to 32bit code generation. > > > > The new style handling DBX_DEBUGGING_INFO is in line with > > dbxelf.h which does not define PREFERRED_DEBUGGING_TYPE either. > > > > The patch also removes the PREFERRED_DEBUGGING_TYPE define from > > lynx.h which always follows elfos.h already defaulting to DWARF, > > so the comment about STABS being the default is misleading and > > outdated. There's no listed maintainer for Lynx OS. > > > > I have not tested this in any ways but I also have no idea how > > to meaningfully do so. > > > > OK? > > > > Thanks, > > Richard. > > > > 2021-09-09 Richard Biener > > > > PR target/102255 > > * config/dbx.h: Remove. > > * config/dbxcoff.h: Do not define PREFERRED_DEBUGGING_TYPE. > > * config/lynx.h: Likewise. > I'd go ahead and install.  We're on a clear path to kill dbx/stabs and if this > breaks those ports, better to do so as early as possible to give folks a > chance to fix 'em. > > I can't really help on the testing side for this -- my tester doesn't try to > test djgpp, mingw or cygwin. I see. Mind the patch doesn't change anything (unless my analysis was flawed). It merely reduces the grep hit for DBX_DEBUGGING_INFO and PREFERRED_DEBUGGING_TYPE ;) My immediate goal is to get rid of PREFERRED_DEBUGGING_TYPE (it will always be DWARF_DEBUGGING_INFO) and _default_ all ports to DWARF so my original goal of being able to deprecate STABS for GCC 12 will work out (which would be sth odd if some port still defaults to it). I've pushed this change now. Richard.