From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 11275 invoked by alias); 10 Aug 2008 02:45:42 -0000 Received: (qmail 11265 invoked by uid 22791); 10 Aug 2008 02:45:42 -0000 X-Spam-Check-By: sourceware.org Received: from rock.gnat.com (HELO rock.gnat.com) (205.232.38.15) by sourceware.org (qpsmtpd/0.31) with ESMTP; Sun, 10 Aug 2008 02:45:05 +0000 Received: from localhost (localhost.localdomain [127.0.0.1]) by filtered-rock.gnat.com (Postfix) with ESMTP id 7B6FF2A9760; Sat, 9 Aug 2008 22:45:03 -0400 (EDT) Received: from rock.gnat.com ([127.0.0.1]) by localhost (rock.gnat.com [127.0.0.1]) (amavisd-new, port 10024) with LMTP id xbubokB0gwF9; Sat, 9 Aug 2008 22:45:03 -0400 (EDT) Received: from joel.gnat.com (localhost.localdomain [127.0.0.1]) by rock.gnat.com (Postfix) with ESMTP id 9B56B2A975B; Sat, 9 Aug 2008 22:45:02 -0400 (EDT) Received: by joel.gnat.com (Postfix, from userid 1000) id 381FCE7ACD; Sun, 10 Aug 2008 04:44:55 +0200 (CEST) Date: Sun, 10 Aug 2008 02:45:00 -0000 From: Joel Brobecker To: Aleksandar Ristovski , gdb-patches@sources.redhat.com, Jerome Guitton Subject: Re: Powerpc skip prologue Message-ID: <20080810024455.GF4936@adacore.com> References: <20080809141651.GB4936@adacore.com> <20080809161131.GA9444@caradoc.them.org> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20080809161131.GA9444@caradoc.them.org> User-Agent: Mutt/1.4.2.2i Mailing-List: contact gdb-patches-help@sourceware.org; run by ezmlm Precedence: bulk List-Id: List-Subscribe: List-Archive: List-Post: List-Help: , Sender: gdb-patches-owner@sourceware.org X-SW-Source: 2008-08/txt/msg00270.txt.bz2 > Could you test an alternative for me? The patch from my December > message, but without rs6000_force_frame_sniffer, the > rs6000_gdbarch_init change, or the new testcase. I believe that will > cover the same cases as your patch, plus several others. This is a > slightly larger portion of the patch than Aleksandar reposted earlier. Sure! > 2008-08-09 Daniel Jacobowitz > > * rs6000-tdep.c (struct rs6000_framedata): Add gpr_mask, used_bl, > lr_register. > (rs6000_in_function_epilogue_p): Check for bctr. > (skip_prologue): Initialize lr_register. Set lr_reg to a register > number. Set gpr_mask and used_bl. Continue scanning while some > expected registers are not saved. Set lr_register if LR is not > stored. > (rs6000_frame_cache): Handle gpr_mask and lr_register. > > * gdb.arch/powerpc-prologue.exp: Correct saved registers. This tested OK on ppc-linux :-). -- Joel