This is the mail archive of the binutils@sources.redhat.com mailing list for the binutils 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: Support mixing COMDAT and linkonce


On Fri, May 14, 2004 at 02:07:37PM -0700, H. J. Lu wrote:
> When there are mixed COMDAT and linkonce inputs, linker doesn't handle
> them gracefully:

Ewww.  Should we even try?  I understand that such a patch might be
useful while gcc is emitting both comdat and linkonce, but once you've
completed the change to comdat it shouldn't be necessary.  Also, I'm not
really happy with where you have added this code.  At least, it is the
wrong place to be discarding duplicate sections.  That ought to happen
in ldlang.c:section_already_linked.

-- 
Alan Modra
IBM OzLabs - Linux Technology Centre


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