From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 10112 invoked by alias); 26 Apr 2002 01:33:21 -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 10105 invoked from network); 26 Apr 2002 01:33:21 -0000 Received: from unknown (HELO pizda.ninka.net) (216.101.162.242) by sources.redhat.com with SMTP; 26 Apr 2002 01:33:21 -0000 Received: from localhost (IDENT:davem@localhost.localdomain [127.0.0.1]) by pizda.ninka.net (8.9.3/8.9.3) with ESMTP id SAA12556; Thu, 25 Apr 2002 18:23:41 -0700 Date: Thu, 25 Apr 2002 18:33:00 -0000 Message-Id: <20020425.182340.84003879.davem@redhat.com> To: msnyder@redhat.com Cc: gdb-patches@sources.redhat.com Subject: Re: [RFA] Improve Sparc epilogue analysis From: "David S. Miller" In-Reply-To: <3CC86E14.F795FFE5@redhat.com> References: <3CC74F72.6B0F8C8B@redhat.com> <20020424.175238.00378500.davem@redhat.com> <3CC86E14.F795FFE5@redhat.com> Mime-Version: 1.0 Content-Type: Text/Plain; charset=us-ascii Content-Transfer-Encoding: 7bit X-SW-Source: 2002-04/txt/msg01058.txt.bz2 From: Michael Snyder Date: Thu, 25 Apr 2002 13:59:00 -0700 "David S. Miller" wrote: > > From: Michael Snyder > Date: Wed, 24 Apr 2002 17:36:02 -0700 > > "David S. Miller" wrote: > > I'd like to ask that you re-review this change, given this, please. > > The part that's mentioned in the changelog entry is OK, except > you can remove sparc_skip_prologue_frameless_p (and say so in > the changelog) rather than rewrite it. > > Wait a second, how can I remove sparc_skip_prologue_frameless_p > when it isn't even there anymore? :-) > > What exists is "sparc_prologue_frameless_p" which is a boolean. > That is what replaces the functionality for what I removed > (sparc_skip_prologue_frameless_p, SKIP_PROLOGUE_FRAMELESS_P, > et al.). > > I think you're still a little confused on the current state of > the tree. That comment was unnecessary, thank you. I think it was more than appropriate. Whether the function is in or out, it is wrong as written, or at the least you have to justify why you have changed the behavior. If it does not exist in the tree, it by definition can't be in any other state than "not there". This part of the thread centers around you telling me to remove "sparc_skip_prologue_frameless_p". And I'm trying to say "it isn't there to begin with, what function are you even talking about?" The existing function returns TRUE if and only if the input address is equal to the address returned by examine_prologue, which in this case will be the address of one of a small set of specific instructions (eg. a SAVE or an add sp). Your function, in the presence of symbols, will return TRUE if and only if the input address is equal to the last instruction in the line (presumably of the prologue). This is a change. I don't know whether it was intentional or not, but you can't make it without explaining it. Here you are making a point about my changes to sparc_prologue_frameless_p, and those are quite valid. And I have dealt with this in the followup patch I sent last night. But this has nothing to do with my what I am replying to, which is you telling me to remove a differently named function which no longer exists in the tree.