Mirror of the gdb mailing list
 help / color / mirror / Atom feed
From: Michael Eager <eager@eagercon.com>
To: Wenbo Yang <wenbo.yang@simplnano.com>
Cc: Mark Kettenis <mark.kettenis@xs4all.nl>,  gdb@sources.redhat.com
Subject: Re: frame cache
Date: Mon, 30 Jul 2007 22:01:00 -0000	[thread overview]
Message-ID: <46AE5570.7080202@eagercon.com> (raw)
In-Reply-To: <46A98934.5040902@simplnano.com>

Wenbo Yang wrote:
> Michael Eager wrote:
> 
>> Perhaps so, I don't see where other targets check for debug
>> info before calling analyze_prologue().  For example, on i386,
>> i386_analyze_prologue() is called each and every time that
>> i386_skip_prologue() is called.
> 
> It depends on target . If you register dwarf2 frame sniffers to gdbarch, 
> and your compiler can emit proper debugging information, gdb will not 
> call analyze_prologue(). At least for our target(I did the porting), it 
> works as Mark Kettenis said:
> "prologue analysis should only be done as a last resort, i.e. when 
> proper debug information is not available. "

Which target?

>> I don't see where there is any test in symtab.c or infrun.c
>> which tests for debug info before calling SKIP_PROLOGUE which
>> calls <target>_skip_prologue.
> 
> Because if you want to use debugging information to skip prologue, you 
> write the code in this function. So, test should be place here in this 
> function.

As I mentioned, in x86 or PowerPC there is no code to bypass analyzing
the prologue, even if there is debug info.  If prologue analysis is
truly to be done "only as a last resort", and I agree that this should
be the case, then this seems not to be represented in the gdb code.

If I've missed where this is done, feel free to point me at the correct
code.

-- 
Michael Eager	 eager@eagercon.com
1960 Park Blvd., Palo Alto, CA 94306  650-325-8077


  reply	other threads:[~2007-07-30 21:17 UTC|newest]

Thread overview: 17+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-07-24 17:08 Michael Eager
2007-07-24 17:14 ` Michael Eager
2007-07-24 17:36   ` Daniel Jacobowitz
2007-07-24 18:19     ` Michael Eager
2007-07-24 18:34       ` Daniel Jacobowitz
2007-07-24 18:34       ` Mark Kettenis
2007-07-24 17:17 ` Daniel Jacobowitz
2007-07-24 18:10   ` Michael Eager
2007-07-24 18:22     ` Daniel Jacobowitz
2007-07-24 18:45       ` Michael Eager
2007-07-24 18:45     ` Mark Kettenis
2007-07-24 19:08       ` Michael Eager
2007-07-25  2:18         ` Paul Koning
2007-07-27  9:18         ` Wenbo Yang
2007-07-30 22:01           ` Michael Eager [this message]
2007-07-30 23:07             ` Mark Kettenis
2007-07-31  3:51             ` Wenbo Yang

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=46AE5570.7080202@eagercon.com \
    --to=eager@eagercon.com \
    --cc=gdb@sources.redhat.com \
    --cc=mark.kettenis@xs4all.nl \
    --cc=wenbo.yang@simplnano.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