From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 22462 invoked by alias); 4 Mar 2005 15:06:37 -0000 Mailing-List: contact gdb-patches-help@sources.redhat.com; run by ezmlm Precedence: bulk List-Subscribe: List-Archive: List-Post: List-Help: , Sender: gdb-patches-owner@sources.redhat.com Received: (qmail 22341 invoked from network); 4 Mar 2005 15:06:30 -0000 Received: from unknown (HELO nevyn.them.org) (66.93.172.17) by sourceware.org with SMTP; 4 Mar 2005 15:06:30 -0000 Received: from drow by nevyn.them.org with local (Exim 4.44 #1 (Debian)) id 1D7ENp-0008Ng-GW for ; Fri, 04 Mar 2005 10:06:29 -0500 Date: Fri, 04 Mar 2005 15:06:00 -0000 From: Daniel Jacobowitz To: gdb-patches@sources.redhat.com Subject: Re: [RFA] New GDB target iq2000 Message-ID: <20050304150629.GA32143@nevyn.them.org> Mail-Followup-To: gdb-patches@sources.redhat.com References: <20050222114141.GA18314@cygbert.vinschen.de> <20050303173443.GD18681@nevyn.them.org> <20050304094605.GU2839@cygbert.vinschen.de> <20050304141439.GA30249@nevyn.them.org> <20050304150129.GF2839@cygbert.vinschen.de> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20050304150129.GF2839@cygbert.vinschen.de> User-Agent: Mutt/1.5.6+20040907i X-SW-Source: 2005-03/txt/msg00044.txt.bz2 On Fri, Mar 04, 2005 at 04:01:29PM +0100, Corinna Vinschen wrote: > On Mar 4 09:14, Daniel Jacobowitz wrote: > > On Fri, Mar 04, 2005 at 10:46:05AM +0100, Corinna Vinschen wrote: > > > I'm sorry, but the reason for getting rid of linetable-aware code is > > > somewhat beyond me. > > > > Because _there is nothing architecture specific about what you are > > doing_. Therefore, most likely, it is either right for all platforms > > or wrong for this one. I want to understand which. If it's right for > > all platforms, I'd like it to live in common code so that we can > > maintain it for all platforms. > > The platform specific part is to call iq2000_scan_prologue if the > line number information is bogus. > > > > I'll happily do something else, as far as it's > > > available and works, but using skip_prologue_using_sal is really no > > > option here. > > > > Why? Is it the same problem Kevin described? As I wrote, I have > > successfully used this function on other architectures. > > I haven't exactly analyzed the situation so far, but using > skip_prologue_using_sal results in three more FAILs in the testsuite: > > FAIL: gdb.base/break.exp: breakpoint small function, optimized file > FAIL: gdb.base/break.exp: run until breakpoint set at small function, optimized file > FAIL: gdb.base/nodebug.exp: running to inner in runto > > All three cases don't look like simple coincidence. In all three cases > we suffer from either optimized code or unavailable debug information. > The target specific "knowledge", which is represented by the call to > iq2000_scan_prologue helps to master this situation. Would you mind posting disassembly, line table (readelf -wl), and GDB transcript for these failures? I'm not trying to block the inclusion of the iq2000 port, but I would like to make sure we understand this problem before we include a workaround. -- Daniel Jacobowitz CodeSourcery, LLC