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