Mirror of the gdb-patches mailing list
 help / color / mirror / Atom feed
From: "Jim Blandy" <jimb@red-bean.com>
To: "Mark Kettenis" <mark.kettenis@xs4all.nl>,
	jimb@red-bean.com,  	brobecker@adacore.com,
	gdb-patches@sources.redhat.com
Subject: Re: [RFC/RFA/i386] pb reading insns if breakpoints still inserted
Date: Fri, 28 Apr 2006 21:42:00 -0000	[thread overview]
Message-ID: <8f2776cb0604281442m2f514240i9f86b4bfa38b1c33@mail.gmail.com> (raw)
In-Reply-To: <20060428211258.GA6713@nevyn.them.org>

On 4/28/06, Daniel Jacobowitz <drow@false.org> wrote:
> On Fri, Apr 28, 2006 at 11:09:19PM +0200, Mark Kettenis wrote:
> > But if we don't have debug information, what's the point in trying to
> > skip the prologue in order to put a breakpoint on ... eh what exactly?
> > Isn't it better to just punt prologue skipping in that case and place
> > the breakpoint on the first instruction of the code?
>
> Yes, I agree.

So you guys would like to push the deprecation boundary further the
other way, and deprecate SKIP_PROLOGUE itself?  That's an idea.

> I'd like to note, though, that the way we skip prologues based on debug
> information is completely wrong.  It only works with GCC and with other
> compilers that have chosen to be bug-compatible with GCC.  DWARF does
> support a "this is the end of the prologue" flag, but I don't know if
> GCC emits it, and I'm pretty positive GDB doesn't know how to consume
> it.

I think the argument was that, in the long run, GCC should be emitting
location lists so that the debugging information accurately describes
the location of the arguments even within the prologues.  Setting a
breakpoint on a function would actually set the breakpoint ... at the
function's entry point.  We could delete SKIP_PROLOGUE altogether. 
Then, there'd be no point in producing or consuming an "end of
prologue" marker whose meaning is kind of vague anyway.


  reply	other threads:[~2006-04-28 21:42 UTC|newest]

Thread overview: 20+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2006-04-28 17:12 Joel Brobecker
2006-04-28 17:54 ` Jim Blandy
2006-04-28 18:40   ` Mark Kettenis
2006-04-28 18:48     ` Joel Brobecker
2006-04-28 20:58     ` Jim Blandy
2006-04-28 21:09       ` Mark Kettenis
2006-04-28 21:13         ` Daniel Jacobowitz
2006-04-28 21:42           ` Jim Blandy [this message]
2006-04-28 21:54             ` Joel Brobecker
2006-04-28 21:49           ` Joel Brobecker
2006-04-28 22:00             ` Jim Blandy
2006-04-29 14:20 ` Eli Zaretskii
2006-04-29 14:28   ` Daniel Jacobowitz
2006-04-29 14:51     ` Eli Zaretskii
2006-04-29 15:06       ` Daniel Jacobowitz
2006-04-29 16:57         ` Eli Zaretskii
2006-04-29 23:14           ` Daniel Jacobowitz
2006-05-01 16:36 ` Mark Kettenis
2006-05-01 17:06   ` Joel Brobecker
2006-05-05 18:16   ` Daniel Jacobowitz

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=8f2776cb0604281442m2f514240i9f86b4bfa38b1c33@mail.gmail.com \
    --to=jimb@red-bean.com \
    --cc=brobecker@adacore.com \
    --cc=gdb-patches@sources.redhat.com \
    --cc=mark.kettenis@xs4all.nl \
    /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