This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]
Other format:	[Raw text]

Re: Ping2: [PATCH] PR debug/16063. Add DW_AT_type to DW_TAG_enumeration.

From: Jason Merrill <jason at redhat dot com>
To: Mark Wielaard <mjw at redhat dot com>, Jakub Jelinek <jakub at redhat dot com>
Cc: Cary Coutant <ccoutant at google dot com>, Tom Tromey <tromey at redhat dot com>, gcc-patches <gcc-patches at gcc dot gnu dot org>, Richard Henderson <rth at redhat dot com>
Date: Mon, 12 May 2014 23:22:11 -0400
Subject: Re: Ping2: [PATCH] PR debug/16063. Add DW_AT_type to DW_TAG_enumeration.
Authentication-results: sourceware.org; auth=none
References: <CAHACq4qas-aVxGf9ViChOcxAXR81TBBYQU5_qy5ZB5awSqts8g at mail dot gmail dot com> <1395744618 dot 5704 dot 98 dot camel at bordewijk dot wildebeest dot org> <1395859013 dot 5704 dot 170 dot camel at bordewijk dot wildebeest dot org> <87ha61qiug dot fsf at fleche dot redhat dot com> <CAHACq4rxfktMVOmocuGNB3z8uQWo-igMe25Eaakfew=y7bq9bQ at mail dot gmail dot com> <1397202952 dot 8924 dot 1 dot camel at bordewijk dot wildebeest dot org> <CAHACq4oQXU1SvLKKZJoJLzSy5xJX7PnSsKjO_duW90HwHu975Q at mail dot gmail dot com> <1397510343 dot 5633 dot 44 dot camel at bordewijk dot wildebeest dot org> <1398162670 dot 29199 dot 127 dot camel at bordewijk dot wildebeest dot org> <1398683852 dot 4016 dot 30 dot camel at bordewijk dot wildebeest dot org> <20140428122313 dot GE1817 at tucnak dot redhat dot com> <1398688641 dot 4016 dot 35 dot camel at bordewijk dot wildebeest dot org>

On 04/28/2014 08:37 AM, Mark Wielaard wrote:

On Mon, 2014-04-28 at 14:23 +0200, Jakub Jelinek wrote:

Do you want to add DW_AT_type to DW_TAG_enumeration only if it has explicit
underlying type (enum class foo: char { ... };) or even when the underlying
type is computed emplicitly (then you'd just use TREE_TYPE of the
ENUMERAL_TYPE if non-NULL).


The debugger cares about the actual underlying type used if the language
can use multiple. Either explicitly assigned by the user or implicitly
as derived by the language/compile flags used. So the lang hook should
provide one in both cases, if appropriate.

Why do you want to do this for C? It seems to me that this is onlyinteresting in C++ because of overload resolution.


Jason

Follow-Ups:
- Re: Ping2: [PATCH] PR debug/16063. Add DW_AT_type to DW_TAG_enumeration.
  - From: Mark Wielaard

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]