From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 19837 invoked by alias); 28 Jan 2003 13:14:47 -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 19829 invoked from network); 28 Jan 2003 13:14:46 -0000 Received: from unknown (HELO localhost.redhat.com) (66.30.197.194) by 172.16.49.205 with SMTP; 28 Jan 2003 13:14:46 -0000 Received: from redhat.com (localhost [127.0.0.1]) by localhost.redhat.com (Postfix) with ESMTP id B8A093D2C; Tue, 28 Jan 2003 08:14:44 -0500 (EST) Message-ID: <3E368244.3020300@redhat.com> Date: Tue, 28 Jan 2003 13:14:00 -0000 From: Andrew Cagney User-Agent: Mozilla/5.0 (X11; U; NetBSD macppc; en-US; rv:1.0.1) Gecko/20021211 X-Accept-Language: en-us, en MIME-Version: 1.0 To: Kevin Buettner Cc: gdb-patches@sources.redhat.com Subject: Re: [RFA] mips-tdep.c: Fix inferior function call breakage References: <1030123072642.ZM24777@localhost.localdomain> <3E3057F0.5000906@redhat.com> <1030124035453.ZM29667@localhost.localdomain> <3E3586B9.9080100@redhat.com> <1030127195746.ZM23807@localhost.localdomain> <3E35B9B2.5060104@redhat.com> <1030127235721.ZM2609@localhost.localdomain> Content-Type: text/plain; charset=us-ascii; format=flowed Content-Transfer-Encoding: 7bit X-SW-Source: 2003-01/txt/msg00756.txt.bz2 > On Jan 27, 5:58pm, Andrew Cagney wrote: > > >> I can see two options: >> >> - Modify get_prev_frame() to not call init extra info when in a dummy >> frame. Not to sure about this. > > > This might be okay, but it needs to be studied. Yep. Study it more when most of the mainstream targets have been overhauled. The PPC (which uses an unusual frame arangement) is especially troubling. >> - Modify mips_init_extra_frame_info() to return immediatly when a dummy >> frame. > > > I'm more confident in this solution. See below. Okay? Yes. Andrew > * mips-tdep.c (mips_init_extra_frame_info): Return early for > dummy frames. > > Index: mips-tdep.c > =================================================================== > RCS file: /cvs/src/src/gdb/mips-tdep.c,v > retrieving revision 1.160 > diff -u -p -r1.160 mips-tdep.c > --- mips-tdep.c 19 Jan 2003 04:06:46 -0000 1.160 > +++ mips-tdep.c 27 Jan 2003 23:51:21 -0000 > @@ -2468,9 +2468,13 @@ static void > mips_init_extra_frame_info (int fromleaf, struct frame_info *fci) > { > int regnum; > + mips_extra_func_info_t proc_desc; > + > + if (get_frame_type (fci) == DUMMY_FRAME) > + return; > > /* Use proc_desc calculated in frame_chain */ > - mips_extra_func_info_t proc_desc = > + proc_desc = > get_next_frame (fci) > ? cached_proc_desc > : find_proc_desc (get_frame_pc (fci), get_next_frame (fci), 1); > >