public inbox for gdb-patches@sourceware.org
 help / color / mirror / Atom feed
From: Pedro Alves <pedro@codesourcery.com>
To: Joel Brobecker <brobecker@adacore.com>
Cc: Tom Tromey <tromey@redhat.com>, gdb-patches@sourceware.org
Subject: Re: [PATCH] gdb.ada/catch_ex.exp, gdb.ada/mi_catch_ex.exp and unsupported catchpoints
Date: Fri, 09 Dec 2011 17:20:00 -0000	[thread overview]
Message-ID: <201112091717.12970.pedro@codesourcery.com> (raw)
In-Reply-To: <20111207220119.GD21915@adacore.com>

On Wednesday 07 December 2011 22:01:19, Joel Brobecker wrote:
> (to answer Pedro's questions, the files in question are libgnat.a and
> libgnat.so)
> 
> > Fedora strips everything to a separate debuginfo file in an automated
> > way.  There might be an exception for one library, but last time I asked
> > about this, I was told that exceptions are a pain and could we please do
> > something else.  Also, the granularity of debuginfo packages is quite
> > large.
> 
> I can see the reason for this, but it is kind of a pain for Ada.
> I believe the makefiles already build the runtime without debugging
> info except for the few files that we need (I think it's basically
> a-tags.adb and a-except.adb)...
> 
> > If you can make it so this can work without debuginfo somehow, that
> > would be much better for users.
> 
> The problem is that we need to get to the argument of function we are
> breaking on in order to determine which exception has been raised.
> That's where we really need the debug info. I think I even added
> a comment about that in the code...

GDB knows the target's function call ABI (for infcalls), and supposedly
this function's prototype is cast in stone as part of the ABI too.
GDB could just know where to get the arguments from?  You'd still need
minimal symbols in libgnat, but that doesn't look much different
from needing minimal symbols for libpthread, from the packager's
perspective -- it should be simpler than caring about stripping the library
except a couple objects, due to size considerations.

-- 
Pedro Alves

  parent reply	other threads:[~2011-12-09 17:17 UTC|newest]

Thread overview: 18+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-12-06 17:33 Pedro Alves
2011-12-07 10:07 ` Joel Brobecker
2011-12-07 15:29   ` Pedro Alves
2011-12-07 16:10     ` Pedro Alves
2011-12-07 22:01   ` Tom Tromey
2011-12-07 23:18     ` Joel Brobecker
2011-12-09  3:50       ` Tom Tromey
2011-12-09 17:20       ` Pedro Alves [this message]
2011-12-09 18:00         ` Tom Tromey
2011-12-09 18:13           ` Pedro Alves
2011-12-09 18:40             ` Tom Tromey
2011-12-10 22:53               ` Pedro Alves
2011-12-11 20:33                 ` Joel Brobecker
2011-12-20 14:53                 ` Tom Tromey
2012-01-10 20:26                   ` Pedro Alves
2011-12-11 17:42   ` [commit] Ada exception catchpoint support cleanup Joel Brobecker
2011-12-11 17:44   ` [commit] Warn if missing debug info for Ada exception catchpoints Joel Brobecker
2011-12-11 18:04   ` [commit/Ada] improve message when cannot insert Ada exception catchpoint Joel Brobecker

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=201112091717.12970.pedro@codesourcery.com \
    --to=pedro@codesourcery.com \
    --cc=brobecker@adacore.com \
    --cc=gdb-patches@sourceware.org \
    --cc=tromey@redhat.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;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).