From 48dfb43001b06090a7634c33bb2b8abd41fd77b6 Mon Sep 17 00:00:00 2001 From: Nick Clifton Date: Tue, 14 Dec 1999 23:35:43 +0000 Subject: [PATCH] Fix detection of common symbols --- bfd/ChangeLog | 10 ++++++++++ bfd/elflink.h | 49 ++++++++++++++++++++++++++++++++++++++++++++----- 2 files changed, 54 insertions(+), 5 deletions(-) diff --git a/bfd/ChangeLog b/bfd/ChangeLog index 8a8271e..fe3fd5e 100644 --- a/bfd/ChangeLog +++ b/bfd/ChangeLog @@ -1,3 +1,13 @@ +1999-12-14 Nick Clifton + + * 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 * config.bfd: Add support for sparc-*-netbsdelf* and diff --git a/bfd/elflink.h b/bfd/elflink.h index 9a4974f..01cb6d4 100644 --- a/bfd/elflink.h +++ b/bfd/elflink.h @@ -80,6 +80,42 @@ elf_bfd_link_add_symbols (abfd, info) } } +/* 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; } } -- 2.7.4