Fix detection of common symbols

diff --git a/bfd/ChangeLog b/bfd/ChangeLog
index 8a8271e..fe3fd5e 100644 (file)
--- a/bfd/ChangeLog
+++ b/bfd/ChangeLog
@@ -1,3 +1,13 @@
+1999-12-14  Nick Clifton  <nickc@cygnus.com>
+
+       * elflink.h (is_global_symbol_definition): New Function: Return
+       true iff the symbol is being given a global definition in this
+       bfd. 
+       (elf_link_is_defined_archive_symbol): Do not bother processing
+       symbols for an archive element that has already been included
+       in the link.
+       Use is_global_symbol_definition().
+       
 1999-12-09  Andrew Cagney  <cagney@cygnus.com>
 
        * config.bfd: Add support for sparc-*-netbsdelf* and

diff --git a/bfd/elflink.h b/bfd/elflink.h
index 9a4974f..01cb6d4 100644 (file)
--- a/bfd/elflink.h
+++ b/bfd/elflink.h
@@ -80,6 +80,42 @@ elf_bfd_link_add_symbols (abfd, info)
     }
 }
 \f
+/* Return true iff this is a non-common definition of a symbol.  */
+static boolean
+is_global_symbol_definition (abfd, sym)
+     bfd * abfd;
+     Elf_Internal_Sym * sym;
+{
+  /* Local symbols do not count, but target specific ones might.  */
+  if (ELF_ST_BIND (sym->st_info) != STB_GLOBAL
+      && ELF_ST_BIND (sym->st_info) < STB_LOOS)
+    return false;
+
+  /* If the section is undefined, then so is the symbol.  */
+  if (sym->st_shndx == SHN_UNDEF)
+    return false;
+  
+  /* If the symbol is defined in the common section, then
+     it is a common definition and so does not count.  */
+  if (sym->st_shndx == SHN_COMMON)
+    return false;
+
+  /* If the symbol is in a target specific section then we
+     must rely upon the backend to tell us what it is.  */
+  if (sym->st_shndx >= SHN_LORESERVE && sym->st_shndx < SHN_ABS)
+    /* FIXME - this function is not coded yet:
+       
+       return _bfd_is_global_symbol_definition (abfd, sym);
+       
+       Instead for now assume that the definition is not global,
+       Even if this is wrong, at least the linker will behave
+       in the same way that it used to do.  */
+    return false;
+      
+  return true;
+}
+
+
 /* Search the symbol table of the archive element of the archive ABFD
    whoes archove map contains a mention of SYMDEF, and determine if
    the symbol is defined in this element.  */
@@ -104,6 +140,13 @@ elf_link_is_defined_archive_symbol (abfd, symdef)
   if (! bfd_check_format (abfd, bfd_object))
     return false;
 
+  /* If we have already included the element containing this symbol in the
+     link then we do not need to include it again.  Just claim that any symbol
+     it contains is not a definition, so that our caller will not decide to
+     (re)include this element.  */
+  if (abfd->archive_pass)
+    return false;
+  
   /* Select the appropriate symbol table.  */
   if ((abfd->flags & DYNAMIC) == 0 || elf_dynsymtab (abfd) == 0)
     hdr = &elf_tdata (abfd)->symtab_hdr;
@@ -159,11 +202,7 @@ elf_link_is_defined_archive_symbol (abfd, symdef)
 
       if (strcmp (name, symdef->name) == 0)
        {
-         result =
-           (ELF_ST_BIND (sym.st_info) == STB_GLOBAL)
-           && (sym.st_shndx != SHN_UNDEF)
-           && (sym.st_shndx != SHN_COMMON)
-           ;
+         result = is_global_symbol_definition (abfd, & sym);
          break;
        }
     }