This is the mail archive of the libstdc++@gcc.gnu.org mailing list for the libstdc++ 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: PR target/37520: NO_DOLLAR_IN_LABEL should be defined on x86 targets

From: Jason Merrill <jason at redhat dot com>
To: Gerald Pfeifer <gerald at pfeifer dot com>
Cc: "H.J. Lu" <hjl dot tools at gmail dot com>, gcc-patches at gcc dot gnu dot org, libstdc++ at gcc dot gnu dot org
Date: Thu, 05 Mar 2009 09:41:18 -0500
Subject: Re: PATCH: PR target/37520: NO_DOLLAR_IN_LABEL should be defined on x86 targets
References: <20090301200357.GA3911@lucon.org> <20090302032006.GA11200@lucon.org> <alpine.LSU.1.99.0903021559070.9031@acrux.dbai.tuwien.ac.at> <alpine.LSU.1.99.0903041153060.9031@acrux.dbai.tuwien.ac.at> <alpine.LSU.1.99.0903051419360.9031@acrux.dbai.tuwien.ac.at>

Gerald Pfeifer wrote:

I don't see how, but according to my tests it seems it is indeed the
patch above which cases the libstdc++ bootstrap failure for me.

Because it changes ANON_AGGRNAME_FORMAT but ANON_AGGRNAME_P still expects an anonymous name to start with JOINER, which is still $.

H.J.'s earlier patch to just change the order of the dot/dollar sections is OK.

Jason

Follow-Ups:
- Re: PATCH: PR target/37520: NO_DOLLAR_IN_LABEL should be defined on x86 targets
  - From: H.J. Lu

References:
- Re: PATCH: PR target/37520: NO_DOLLAR_IN_LABEL should be defined on x86 targets
  - From: Gerald Pfeifer
- Re: PATCH: PR target/37520: NO_DOLLAR_IN_LABEL should be defined on x86 targets
  - From: Gerald Pfeifer

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