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: [PATCH] Fix PR debug/19345

From: Mark Mitchell <mark at codesourcery dot com>
To: Jason Merrill <jason at redhat dot com>
Cc: Daniel Berlin <dberlin at dberlin dot org>, gcc-patches at gcc dot gnu dot org
Date: Thu, 03 Feb 2005 22:04:44 -0800
Subject: Re: [PATCH] Fix PR debug/19345
Organization: CodeSourcery, LLC
References: <Pine.LNX.4.60.0501262220570.5167@dberlin.org> <41F89195.6050301@codesourcery.com> <Pine.LNX.4.60.0501270910080.6792@dberlin.org> <Pine.LNX.4.60.0501270920190.6792@dberlin.org> <Pine.LNX.4.60.0501281031381.2766@dberlin.org> <41FAA5E8.5050501@codesourcery.com> <xypsm4dbc9j.fsf@miranda.boston.redhat.com>

Jason Merrill wrote:

I'm not sure.  Every time I try to understand TYPE_STUB_DECL I get
confused.  Only Jason Merrill seems to have a firm grasp on that.

I'm not sure I do, either.  I thought all class types should have
TYPE_STUB_DECL set, but I've never been clear on why it's necessary in
addition to putting the decl in TYPE_NAME.

OK, that makes two of us.

I suspect, from a quick look at the code, that it serves no purpose at all, other than to waste a slot in TYPE_DECLs. Certainly, in C++, the canonical DECL for a class type should be TYPE_NAME (TYPE_MAIN_VARIANT (type)).

I suspect that in 4.1 we could just remove TYPE_STUB_DECL.

--
Mark Mitchell
CodeSourcery, LLC
mark@codesourcery.com
(916) 791-8304

Follow-Ups:
- Re: [PATCH] Fix PR debug/19345
  - From: Daniel Berlin

References:
- Re: [PATCH] Fix PR debug/19345
  - From: Jason Merrill

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