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: Thu, 12 Apr 2012 13:09:00 -0000 [thread overview]
Message-ID: <alpine.DEB.1.10.1204120049120.19835@tp.orcam.me.uk> (raw)
In-Reply-To: <8762d6ccmk.fsf@talisman.home>
On Wed, 11 Apr 2012, Richard Sandiford wrote:
> > [Richard, I've cc-ed you as the MIPS port maintainer of GCC and binutils,
> > the producers of MIPS16 and some other thunks covered here, in case you
> > had anything to add, and just so that you know this issue is being
> > addressed now.]
>
> Sounds like great work, thanks. Hope it goes in.
Well, there's only this small issue of gdbarch_in_solib_return_trampoline
-- it shouldn't be too hard to overcome. :)
> I don't really have anything constructive to say, but just out of
> curiosity: we "fixed" call/return stubs to have unwind information for
> GCC 4.7. Do you happen to know whether the test passes with that change?
I am not completely sure as I haven't tried it/got to 4.7 yet (you may
remember I had troubles building the head of the tree; I saw you fixed
that at some point, thanks), but it looks to me the generic trampoline
stuff I rely on here is tangential to any particular frame unwinders (as
long as they get their details right, of course, which may not necessarily
be true in all cases, as it is with the heuristic unwinders), so it should
keep working. I'll see if I can get to verifying this soon.
Did you actually add this unwind information for both the mips16.S pieces
and the compiler generated bits (how about the PIC stubs produced by LD?)?
Does that happen to address the problem I reported here:
http://gcc.gnu.org/ml/gcc-patches/2011-12/msg01067.html
? Note that it's not a coincidence the function name there is the same as
one of those I used in the test case posted here -- this GDB test case
will fail without a fix for that GCC problem just like the test case I
posted there did.
Maciej
next prev parent reply other threads:[~2012-04-12 10:57 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 [this message]
2012-04-12 19:39 ` Richard Sandiford
2012-04-13 20:46 ` Maciej W. Rozycki
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.1204120049120.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