From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 23597 invoked by alias); 8 Jun 2006 23:25:50 -0000 Received: (qmail 23579 invoked by uid 22791); 8 Jun 2006 23:25:47 -0000 X-Spam-Check-By: sourceware.org Received: from intranet.codesourcery.com (HELO mail.codesourcery.com) (65.74.133.6) by sourceware.org (qpsmtpd/0.31) with ESMTP; Thu, 08 Jun 2006 23:25:45 +0000 Received: (qmail 30726 invoked from network); 8 Jun 2006 23:25:43 -0000 Received: from unknown (HELO localhost) (jimb@127.0.0.2) by mail.codesourcery.com with ESMTPA; 8 Jun 2006 23:25:43 -0000 To: Julian Brown Cc: gdb-patches@sourceware.org, Daniel Jacobowitz Subject: Re: [PATCH] Accept DWARF 3-format debug info References: <4488A920.9090705@codesourcery.com> From: Jim Blandy Date: Thu, 08 Jun 2006 23:25:00 -0000 In-Reply-To: <4488A920.9090705@codesourcery.com> (Julian Brown's message of "Thu, 08 Jun 2006 23:48:00 +0100") Message-ID: User-Agent: Gnus/5.11 (Gnus v5.11) Emacs/22.0.50 (gnu/linux) MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii X-IsSubscribed: yes Mailing-List: contact gdb-patches-help@sourceware.org; run by ezmlm Precedence: bulk List-Subscribe: List-Archive: List-Post: List-Help: , Sender: gdb-patches-owner@sourceware.org X-SW-Source: 2006-06/txt/msg00083.txt.bz2 Julian Brown writes: > This is part of a series of patches from a CodeSourcery branch which > enable the output of ARM's RVCT 2.2 compiler to be debugged with gdb. > > ARMCC's support for DWARF 3 is superior to its DWARF 2 support, and > GDB apparently supports much of DWARF 3 already. This patch allows > DWARF 3 format debug info to be accepted by gdb. > > (Some things are notably missing from the DWARF 3 support -- one thing > I've noticed so far is lack of support for DW_TAG_partial_unit, but > RVDS 2.2 doesn't seem to need that. A couple of other minor things > will be addressed in followup patches.) I think this patch is right. It's true that GDB doesn't support all of DWARF 3, but GDB's general philosophy is to garner as much information as it can, and punt the rest, which works well with DWARF's philosophy. So GDB shouldn't just punt entire compilation units because they're marked as DWARF 3. First, please add yourself to the Write After Approval list in gdb/MAINTAINERS, with an appropriate ChangeLog entry. Then, go ahead and put in the patch as a separate commit. Thanks!