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: 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


  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