PATCH: Support mixing COMDAT and linkonce
H. J. Lu
hjl@lucon.org
Sat May 15 15:44:00 GMT 2004
On Sat, May 15, 2004 at 11:09:47AM +0930, Alan Modra wrote:
> 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
We can't guarantee everyone will use the same gcc.
> 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.
I will see what I can do. I want to check if 2 comdat/linkonce sections
define the same set of external symbols instead of comparing sizes.
H.J.
More information about the Binutils
mailing list