This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: Ping2: [PATCH] PR debug/16063. Add DW_AT_type to DW_TAG_enumeration.
- From: Jakub Jelinek <jakub at redhat dot com>
- To: Mark Wielaard <mjw 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>, Jason Merrill <jason at redhat dot com>
- Date: Mon, 28 Apr 2014 14:23:13 +0200
- 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>
- Reply-to: Jakub Jelinek <jakub at redhat dot com>
On Mon, Apr 28, 2014 at 01:17:32PM +0200, Mark Wielaard wrote:
> On Tue, 2014-04-22 at 12:31 +0200, Mark Wielaard wrote:
> > On Mon, 2014-04-14 at 23:19 +0200, Mark Wielaard wrote:
> > > On Fri, 2014-04-11 at 11:03 -0700, Cary Coutant wrote:
> > > > >> The DWARF bits are fine with me.
> > > > >
> > > > > Thanks. Who can approve the other bits?
> > > >
> > > > You should probably get C and C++ front end approval. I'm not really
> > > > sure who needs to review patches in c-family/. Since the part in c/ is
> > > > so tiny, maybe all you need is a C++ front end maintainer. Both
> > > > Richard Henderson and Jason Merrill are global reviewers, so either of
> > > > them could approve the whole thing.
> > >
> > > Thanks, I added them to the CC.
> > >
> > > > > When approved should I wait till stage 1 opens before committing?
> > > >
> > > > Yes. The PR you're fixing is an enhancement request, not a regression,
> > > > so it needs to wait.
> > >
> > > Since stage one just opened up again this seems a good time to re-ask
> > > for approval then :) Rebased patch against current trunk attached.
> >
> > Ping. Tom already pushed his patches to GDB that take advantage of the
> > new information if available.
>
> Ping2. Please let me know if I should ping/cc other people to get this
> reviewed.
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).
Jakub