Mirror of the gdb-patches mailing list
 help / color / mirror / Atom feed
From: Tom Tromey <tromey@redhat.com>
To: Keith Seitz <keiths@redhat.com>
Cc: gdb-patches@sourceware.org
Subject: Re: The future of dwarf2_physname
Date: Fri, 20 May 2011 19:26:00 -0000	[thread overview]
Message-ID: <m3k4dl5g3m.fsf@fleche.redhat.com> (raw)
In-Reply-To: <20110519205943.GA7479@caradoc.them.org> (Daniel Jacobowitz's	message of "Thu, 19 May 2011 16:59:43 -0400")

>>>>> "Daniel" == Daniel Jacobowitz <drow@false.org> writes:

Daniel> In general, the sort of cross-check Jan has implemented could never
Daniel> reliably pass without extensions to DWARF.  Once you get up into
Daniel> templates, especially, there are template arguments that are hard
Daniel> or impossible to represent in template parameter DIEs.

DWARF 4 added some stuff to help with this, but I think g++ hasn't yet
caught up.  What problems do you know about?

I know of this:

    http://gcc.gnu.org/bugzilla/show_bug.cgi?id=41736

... namely, that g++ doesn't emit a value for a pointer-to-member
template parameter.  But this is a g++ bug, not a DWARF omission.

Also there is:

    http://gcc.gnu.org/bugzilla/show_bug.cgi?id=33861

I don't think I totally understand the issues with this one, but the
DWARF just has an address, so presumably in some scenarios we can wind
up with an odd canonical name (if we can't find the name corresponding
to that address).


It seems to me that if the DWARF spec is incomplete, then that is an
argument in favor of either (1) fixing DWARF and g++ or (2) keeping
DW_AT_linkage_name in g++ -- but not just dropping linkage-name, as that
would cause user-visible regressions.

Tom


  reply	other threads:[~2011-05-20 19:26 UTC|newest]

Thread overview: 24+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-05-18 22:35 Keith Seitz
2011-05-19 19:23 ` Jan Kratochvil
2011-05-20 19:53   ` Keith Seitz
2011-05-20 20:38     ` Jan Kratochvil
2011-05-20 20:39     ` Tom Tromey
2011-05-21 20:37       ` Daniel Jacobowitz
2011-05-19 21:00 ` Daniel Jacobowitz
2011-05-20 19:26   ` Tom Tromey [this message]
2011-05-21 20:34     ` Daniel Jacobowitz
2011-05-20 19:10 ` Tom Tromey
2011-05-23 13:17 ` Jan Kratochvil
2011-05-23 19:52   ` Tom Tromey
2011-05-23 19:57     ` Keith Seitz
2011-05-24 21:12   ` [rfc 1/2] libiberty/ options code reshuffle [Re: The future of dwarf2_physname] Jan Kratochvil
2011-06-02 15:36     ` obsolete: " Jan Kratochvil
2011-05-24 21:21   ` [rfc 2/2] Follow DW_AT_linkage_name for methods " Jan Kratochvil
2011-05-25 19:40     ` Keith Seitz
2011-05-25 20:42       ` Tom Tromey
2011-05-25 20:40     ` Tom Tromey
2011-06-01 18:35       ` Keith Seitz
2011-06-02 16:26         ` Jan Kratochvil
2011-06-02 18:20           ` Tom Tromey
2011-06-02 18:28             ` Jan Kratochvil
2011-06-02 19:03     ` obsolete: " Jan Kratochvil

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=m3k4dl5g3m.fsf@fleche.redhat.com \
    --to=tromey@redhat.com \
    --cc=gdb-patches@sourceware.org \
    --cc=keiths@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