1999-09-12 Donn Terry <donn@interix.com>

	* ldlang.c (section_already_linked): Use comdat information if it
	is available.

diff --git a/ld/ChangeLog b/ld/ChangeLog
index 66bf46ef053791d5feb42ba22e2300cd02989216..ddd70feea17c5810ac7050cc182f4e7ead1edfbd 100644 (file)
--- a/ld/ChangeLog
+++ b/ld/ChangeLog
@@ -23,6 +23,9 @@
 
 1999-09-12  Donn Terry  <donn@interix.com>
 
+       * ldlang.c (section_already_linked): Use comdat information if it
+       is available.
+
        * emultempl/pe.em (PE_DEF_SECTION_ALIGNMENT): If
        OVERRIDE_SECTION_ALIGNMENT is defined, change to that.
        (gld_${EMULATION_NAME}_before_parse): Use EXECUTABLE_NAME if it is

diff --git a/ld/ldlang.c b/ld/ldlang.c
index 102ece9bc19da77327b230958debbb5725a5ff26..56a00540103c06d0fa7af70e12243c49a64712d6 100644 (file)
--- a/ld/ldlang.c
+++ b/ld/ldlang.c
@@ -862,9 +862,11 @@ exp_init_os (exp)
 /* Sections marked with the SEC_LINK_ONCE flag should only be linked
    once into the output.  This routine checks each sections, and
    arranges to discard it if a section of the same name has already
-   been linked.  This code assumes that all relevant sections have the
-   SEC_LINK_ONCE flag set; that is, it does not depend solely upon the
-   section name.  This is called via bfd_map_over_sections.  */
+   been linked.  If the section has COMDAT information, then it uses
+   that to decide whether the section should be included.  This code
+   assumes that all relevant sections have the SEC_LINK_ONCE flag set;
+   that is, it does not depend solely upon the section name.  This is
+   called via bfd_map_over_sections.  */
 
 /*ARGSUSED*/
 static void
@@ -919,7 +921,10 @@ section_already_linked (abfd, sec, data)
 
   for (l = sec_link_once_list; l != NULL; l = l->next)
     {
-      if (strcmp (name, bfd_get_section_name (l->sec->owner, l->sec)) == 0)
+      if (strcmp (name, bfd_get_section_name (l->sec->owner, l->sec)) == 0
+         && (sec->comdat == NULL
+             || l->sec->comdat == NULL
+             || strcmp (sec->comdat->name, l->sec->comdat->name) == 0))
        {
          /* The section has already been linked.  See if we should
              issue a warning.  */
@@ -932,8 +937,12 @@ section_already_linked (abfd, sec, data)
              break;
 
            case SEC_LINK_DUPLICATES_ONE_ONLY:
-             einfo (_("%P: %B: warning: ignoring duplicate section `%s'\n"),
-                    abfd, name);
+             if (sec->comdat == NULL)
+               einfo (_("%P: %B: warning: ignoring duplicate section `%s'\n"),
+                      abfd, name);
+             else
+               einfo (_("%P: %B: warning: ignoring duplicate `%s' section symbol `%s'\n"),
+                      abfd, name, sec->comdat->name);
              break;
 
            case SEC_LINK_DUPLICATES_SAME_CONTENTS:
@@ -952,8 +961,13 @@ section_already_linked (abfd, sec, data)
            }
 
          /* Set the output_section field so that wild_doit does not
-            create a lang_input_section structure for this section.  */
+            create a lang_input_section structure for this section.
+            Since there might be a symbol in the section being
+            discarded, we must retain a pointer to the section which
+            we are really going to use.  */
          sec->output_section = bfd_abs_section_ptr;
+         if (sec->comdat != NULL)
+           sec->comdat->sec = l->sec;
 
          return;
        }