diff options
author | Alan Modra <amodra@gmail.com> | 2014-08-05 10:48:47 +0930 |
---|---|---|
committer | Alan Modra <amodra@gmail.com> | 2014-08-05 10:48:47 +0930 |
commit | 02eb0a49bceb35e4b0503e6ffc11e85151dbc571 (patch) | |
tree | c86b7958a0d272b86424ae7a6a38565472834d85 | |
parent | 13e570f80cbfb299a8858ce6830e91a6cb40ab7b (diff) | |
download | gdb-02eb0a49bceb35e4b0503e6ffc11e85151dbc571.zip gdb-02eb0a49bceb35e4b0503e6ffc11e85151dbc571.tar.gz gdb-02eb0a49bceb35e4b0503e6ffc11e85151dbc571.tar.bz2 |
Fix load of archive element with common def for -u sym
* linker.c (generic_link_check_archive_element): Move handling
of command link -u symbols with a common symbol def to the
code handling non-common symbols so that archive element symbols
are loaded. Use generic_link_add_object_symbols.
-rw-r--r-- | bfd/ChangeLog | 7 | ||||
-rw-r--r-- | bfd/linker.c | 42 |
2 files changed, 16 insertions, 33 deletions
diff --git a/bfd/ChangeLog b/bfd/ChangeLog index fa976c4..876de33 100644 --- a/bfd/ChangeLog +++ b/bfd/ChangeLog @@ -1,5 +1,12 @@ 2014-08-05 Alan Modra <amodra@gmail.com> + * linker.c (generic_link_check_archive_element): Move handling + of command link -u symbols with a common symbol def to the + code handling non-common symbols so that archive element symbols + are loaded. Use generic_link_add_object_symbols. + +2014-08-05 Alan Modra <amodra@gmail.com> + PR 13557 * linker.c (struct archive_list, struct archive_hash_entry, struct archive_hash_table, archive_hash_newfunc, diff --git a/bfd/linker.c b/bfd/linker.c index a31e473..1877e67 100644 --- a/bfd/linker.c +++ b/bfd/linker.c @@ -1120,29 +1120,21 @@ generic_link_check_archive_element (bfd *abfd, /* P is a symbol we are looking for. */ - if (! bfd_is_com_section (p->section)) + if (! bfd_is_com_section (p->section) + || (h->type == bfd_link_hash_undefined + && h->u.undef.abfd == NULL)) { - bfd_size_type symcount; - asymbol **symbols; - bfd *oldbfd = abfd; - - /* This object file defines this symbol, so pull it in. */ + /* P is not a common symbol, or an undefined reference was + created from outside BFD such as from a linker -u option. + This object file defines the symbol, so pull it in. */ + *pneeded = TRUE; if (!(*info->callbacks ->add_archive_element) (info, abfd, bfd_asymbol_name (p), &abfd)) return FALSE; /* Potentially, the add_archive_element hook may have set a substitute BFD for us. */ - if (abfd != oldbfd - && !bfd_generic_link_read_symbols (abfd)) - return FALSE; - symcount = _bfd_generic_link_get_symcount (abfd); - symbols = _bfd_generic_link_get_symbols (abfd); - if (! generic_link_add_symbol_list (abfd, info, symcount, - symbols, collect)) - return FALSE; - *pneeded = TRUE; - return TRUE; + return generic_link_add_object_symbols (abfd, info, collect); } /* P is a common symbol. */ @@ -1153,23 +1145,6 @@ generic_link_check_archive_element (bfd *abfd, bfd_vma size; unsigned int power; - symbfd = h->u.undef.abfd; - if (symbfd == NULL) - { - /* This symbol was created as undefined from outside - BFD. We assume that we should link in the object - file. This is for the -u option in the linker. */ - if (!(*info->callbacks - ->add_archive_element) (info, abfd, bfd_asymbol_name (p), - &abfd)) - return FALSE; - /* Potentially, the add_archive_element hook may have set a - substitute BFD for us. But no symbols are going to get - registered by anything we're returning to from here. */ - *pneeded = TRUE; - return TRUE; - } - /* Turn the symbol into a common symbol but do not link in the object file. This is how a.out works. Object formats that require different semantics must implement @@ -1177,6 +1152,7 @@ generic_link_check_archive_element (bfd *abfd, undefs list. We add the section to a common section attached to symbfd to ensure that it is in a BFD which will be linked in. */ + symbfd = h->u.undef.abfd; h->type = bfd_link_hash_common; h->u.c.p = (struct bfd_link_hash_common_entry *) bfd_hash_allocate (&info->hash->table, |