Mirror of the gdb mailing list
 help / color / mirror / Atom feed
From: Daniel Jacobowitz <drow@mvista.com>
To: Andrew Cagney <ac131313@redhat.com>, gdb@sources.redhat.com
Subject: Re: Enable frame-base before frame-unwind?
Date: Thu, 26 Jun 2003 04:56:00 -0000	[thread overview]
Message-ID: <20030626021443.GA12416@nevyn.them.org> (raw)
In-Reply-To: <20030625170253.GA23496@nevyn.them.org>

On Wed, Jun 25, 2003 at 01:02:53PM -0400, Daniel Jacobowitz wrote:
> On Wed, Jun 25, 2003 at 11:42:41AM -0400, Andrew Cagney wrote:
> > Here's a new theory on how to migrate to the unwind code:
> > 
> > 	Enable frame-base before frame-unwind.
> > 
> > Why?  The frame-unwind code can't be enabled until frame-base is working 
> > anyway.  Since frame-base is an almost direct replacement for 
> > FRAME_LOCALS_ADDRESS and FRAME_ARGS_ADDRESS, and the return values for 
> > the old/new methods are the same, I think getting frame-base working is 
> > going to be much easier than getting frame-unwind working.  Since 
> > frame-base and frame-unwind share the prologue analysis code, debugging 
> > it with the less harmful frame-base should make life easier.
> > 
> > Any one want to prove the theory?
> 
> ... sure.

> I think you've answered this once already, but is there a list of
> routines that have needed to change, so far?  Or do I need to put one
> together as I go along?

I don't think the theory holds.

I picked ARM, because I've been meaning to work on framifying ARM for
some time now.  Just like Richard, I've been running over and over into
places where it's just not clear what the new code is supposed to do,
or how it's supposed to do it.  So take this with a grain of salt.

ARM defines neither FRAME_LOCALS_ADDRESS or FRAME_ARGS_ADDRESS.  Both
default to the value of get_frame_base.  In order to frame-base-ify it,
I would have to move all the code which unwinds the frame base to the
new structure, and I can't figure out how.

I'm looking over the AVR patch now...

-- 
Daniel Jacobowitz
MontaVista Software                         Debian GNU/Linux Developer


  reply	other threads:[~2003-06-26  4:56 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2003-06-25 16:51 Andrew Cagney
2003-06-25 18:28 ` Daniel Jacobowitz
2003-06-26  4:56   ` Daniel Jacobowitz [this message]
2003-06-26 14:07     ` Andrew Cagney

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20030626021443.GA12416@nevyn.them.org \
    --to=drow@mvista.com \
    --cc=ac131313@redhat.com \
    --cc=gdb@sources.redhat.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox