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
next prev parent 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