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] |
This patch tries to implement comdat group to properly fix the second testcase in PR 16276. There are still a few issues: 1. The latest binutils is needed. The current binutils in CVS doesn't support mixing comdat and linkonce: http://sources.redhat.com/bugzilla/show_bug.cgi?id=246 I do have a patch to fix it. It will be tricky to check if binutils is OK. 2. It uses string functions to convert the linkonce section name into comdat group. The interface isn't very elegant. But its advantages are a. Minimum change for gcc. Gcc supports more than ELF while comdat group is ELF specific. b. Section name is very significant in gcc. It is used for code generation among others. Also we may have to keep the same section naming scheme to support __attribute__ ((section (".foo"))) Introducing a parallel section naming scheme for comdat group may be problematic. c. It is easy to put a section into a group by giving it the right name. H.J.
Attachment:
gcc-3.4-comdat-7.patch
Description: Text document
Index Nav: | [Date Index] [Subject Index] [Author Index] [Thread Index] | |
---|---|---|
Message Nav: | [Date Prev] [Date Next] | [Thread Prev] [Thread Next] |