Mirror of the gdb-patches mailing list
 help / color / mirror / Atom feed
From: "Maciej W. Rozycki" <macro@codesourcery.com>
To: Richard Sandiford <rdsandiford@googlemail.com>
Cc: <gdb-patches@sourceware.org>
Subject: Re: [RFA] MIPS/GDB: Fix the handling of MIPS16 thunks
Date: Fri, 13 Apr 2012 20:46:00 -0000	[thread overview]
Message-ID: <alpine.DEB.1.10.1204132055370.19835@tp.orcam.me.uk> (raw)
In-Reply-To: <87lim0vjr9.fsf@talisman.home>

On Thu, 12 Apr 2012, Richard Sandiford wrote:

> >  Did you actually add this unwind information for both the mips16.S pieces 
> > and the compiler generated bits
> 
> Yes.  But like I say, the fix was only (suppoed to be) for what you
> called call/return stubs, i.e. those in which the stub JALs to the target,
> fiddles with the return value afterwards, then JRs back to the caller.

 OK, just wanted to be sure.

 BTW, do you also find the choice of "__call_stub_" and "__call_stub_fp_" 
for function prefixes unfortunate?  You can't really tell them apart in 
all cases except by interpreting code as you can have a valid user 
function starting with "fp_" -- this is not a reserved namespace, unlike 
anything starting with "_".  These should really be "__call_stub_" and 
"__call_fp_stub_" or whatever, one just shouldn't be a substring of the 
other, sigh...

> It wasn't intended to touch pure "run this code first" stubs like...
> 
> > (how about the PIC stubs produced by LD?)?  
> 
> ...these.

 Ack.  They're mostly handled by GDB already anyway.

> > Does that happen to address the problem I reported here:
> >
> > http://gcc.gnu.org/ml/gcc-patches/2011-12/msg01067.html
> 
> 'Fraid not.  It was a separate problem.

 Any chance you'll be able to look into it anytime soon?  I can cope with 
the patch I proposed there, but people may prefer an in-tree solution and 
it doesn't look like I'll have time in the near future to work on anything 
like what you've outlined instead of my simple solution.

  Maciej


  reply	other threads:[~2012-04-13 20:27 UTC|newest]

Thread overview: 16+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-04-10 22:36 Maciej W. Rozycki
2012-04-11 19:16 ` Richard Sandiford
2012-04-12 13:09   ` Maciej W. Rozycki
2012-04-12 19:39     ` Richard Sandiford
2012-04-13 20:46       ` Maciej W. Rozycki [this message]
2012-04-20 14:54 ` Pedro Alves
2012-04-20 16:01   ` Maciej W. Rozycki
2012-04-21 18:55     ` Daniel Jacobowitz
2012-04-26  0:10     ` Maciej W. Rozycki
2012-04-26 13:10       ` Pedro Alves
2012-04-26 17:27         ` Maciej W. Rozycki
2012-04-26 13:23       ` gdb_test_multiple and empty $message Pedro Alves
2012-04-26 13:34         ` Pedro Alves
2012-04-26 14:31         ` Tom Tromey
2012-04-26 14:42           ` Joel Brobecker
2012-04-27 20:23           ` Maciej W. Rozycki

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=alpine.DEB.1.10.1204132055370.19835@tp.orcam.me.uk \
    --to=macro@codesourcery.com \
    --cc=gdb-patches@sourceware.org \
    --cc=rdsandiford@googlemail.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