This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: PATCH: Use COMDAT section group instead of gnu.linkonce
- From: Zack Weinberg <zack at codesourcery dot com>
- To: "H. J. Lu" <hjl at lucon dot org>
- Cc: Mike Stump <mrs at apple dot com>, gcc-patches at gcc dot gnu dot org, Jim Wilson <wilson at specifixinc dot com>, gcc at gcc dot gnu dot org
- Date: Wed, 05 May 2004 11:39:25 -0700
- Subject: Re: PATCH: Use COMDAT section group instead of gnu.linkonce
- References: <20040504054619.GA5965@lucon.org><4075E063-9DFC-11D8-B137-003065A77310@apple.com><20040505182649.GA8969@lucon.org>
"H. J. Lu" <hjl@lucon.org> writes:
> On Tue, May 04, 2004 at 11:52:54AM -0700, Mike Stump wrote:
>> On Monday, May 3, 2004, at 10:46 PM, H. J. Lu wrote:
>> >Should we start moving toward COMDAT section group? It will help fix
>> >DWARF2 problems.
>>
>> We are in favor of this. Our platform doesn't yet have group COMDAT,
>> but, that is the way we know we need to go. Thanks.
>
> This patch enables COMDAT group. It even works with
>
> const int foo __attribute__((section(".gnu.linkonce.r.foo")));
>
> There are no regressions on ia32, ia64 and x86-64. You just need to
> apply
>
> http://sources.redhat.com/ml/binutils/2004-05/msg00022.html
>
> on top of the current binutils in CVS.
Uh, surely there is a better way to do this than a long chain of
strcmp() operations on the linkonce section name?
zw