diff options
author | Alan Modra <amodra@gmail.com> | 2004-10-13 02:28:01 +0000 |
---|---|---|
committer | Alan Modra <amodra@gmail.com> | 2004-10-13 02:28:01 +0000 |
commit | 09c112daa3a9b33a03b9272da7e9e19e8d63dc91 (patch) | |
tree | 0eab074a82c35e142b376442815e3e1201970ca4 /ld | |
parent | 05d073081f8c36980df941fb74dcd0dc6b0fa428 (diff) | |
download | binutils-09c112daa3a9b33a03b9272da7e9e19e8d63dc91.zip binutils-09c112daa3a9b33a03b9272da7e9e19e8d63dc91.tar.gz binutils-09c112daa3a9b33a03b9272da7e9e19e8d63dc91.tar.bz2 |
PR 44
* ldcref.c (check_section_sym_xref): Delete.
(check_local_sym_xref): New function.
(check_nocrossrefs): Adjust call.
Diffstat (limited to 'ld')
-rw-r--r-- | ld/ChangeLog | 3 | ||||
-rw-r--r-- | ld/ldcref.c | 56 |
2 files changed, 46 insertions, 13 deletions
diff --git a/ld/ChangeLog b/ld/ChangeLog index 0b6aa1d..e03f60b 100644 --- a/ld/ChangeLog +++ b/ld/ChangeLog @@ -1,6 +1,9 @@ 2004-10-13 Alan Modra <amodra@bigpond.net.au> PR 44 + * ldcref.c (check_section_sym_xref): Delete. + (check_local_sym_xref): New function. + (check_nocrossrefs): Adjust call. * emultempl/pe.em (struct orphan_save): Add os_tail. (place_orphan): Backport assorted fixes from elf32.em. diff --git a/ld/ldcref.c b/ld/ldcref.c index 3094735..10f32ca 100644 --- a/ld/ldcref.c +++ b/ld/ldcref.c @@ -69,7 +69,7 @@ struct cref_hash_table { /* Forward declarations. */ static void output_one_cref (FILE *, struct cref_hash_entry *); -static void check_section_sym_xref (lang_input_statement_type *); +static void check_local_sym_xref (lang_input_statement_type *); static bfd_boolean check_nocrossref (struct cref_hash_entry *, void *); static void check_refs (const char *, asection *, bfd *, struct lang_nocrossrefs *); @@ -136,7 +136,7 @@ cref_hash_newfunc (struct bfd_hash_entry *entry, } /* Add a symbol to the cref hash table. This is called for every - symbol that is seen during the link. */ + global symbol that is seen during the link. */ void add_cref (const char *name, @@ -330,39 +330,69 @@ check_nocrossrefs (void) cref_hash_traverse (&cref_table, check_nocrossref, NULL); - lang_for_each_file (check_section_sym_xref); + lang_for_each_file (check_local_sym_xref); } -/* Checks for prohibited cross references to section symbols. */ +/* Check for prohibited cross references to local and section symbols. */ static void -check_section_sym_xref (lang_input_statement_type *statement) +check_local_sym_xref (lang_input_statement_type *statement) { bfd *abfd; - asection *sec; + lang_input_statement_type *li; + asymbol **asymbols, **syms; abfd = statement->the_bfd; if (abfd == NULL) return; - for (sec = abfd->sections; sec != NULL; sec = sec->next) + li = abfd->usrdata; + if (li != NULL && li->asymbols != NULL) + asymbols = li->asymbols; + else { - asection *outsec; + long symsize; + long symbol_count; - outsec = sec->output_section; - if (outsec != NULL) + symsize = bfd_get_symtab_upper_bound (abfd); + if (symsize < 0) + einfo (_("%B%F: could not read symbols; %E\n"), abfd); + asymbols = xmalloc (symsize); + symbol_count = bfd_canonicalize_symtab (abfd, asymbols); + if (symbol_count < 0) + einfo (_("%B%F: could not read symbols: %E\n"), abfd); + if (li != NULL) + { + li->asymbols = asymbols; + li->symbol_count = symbol_count; + } + } + + for (syms = asymbols; *syms; ++syms) + { + asymbol *sym = *syms; + if (sym->flags & (BSF_GLOBAL | BSF_WARNING | BSF_INDIRECT | BSF_FILE)) + continue; + if ((sym->flags & (BSF_LOCAL | BSF_SECTION_SYM)) != 0 + && sym->section->output_section != NULL) { - const char *outsecname; + const char *outsecname, *symname; struct lang_nocrossrefs *ncrs; struct lang_nocrossref *ncr; - outsecname = outsec->name; + outsecname = sym->section->output_section->name; + symname = NULL; + if ((sym->flags & BSF_SECTION_SYM) == 0) + symname = sym->name; for (ncrs = nocrossref_list; ncrs != NULL; ncrs = ncrs->next) for (ncr = ncrs->list; ncr != NULL; ncr = ncr->next) if (strcmp (ncr->name, outsecname) == 0) - check_refs (NULL, sec, abfd, ncrs); + check_refs (symname, sym->section, abfd, ncrs); } } + + if (li == NULL) + free (asymbols); } /* Check one symbol to see if it is a prohibited cross reference. */ |