From 645ea6a951e7e1d48eff89c71a812ed5bc308b57 Mon Sep 17 00:00:00 2001 From: Alan Modra Date: Sat, 11 Dec 2004 03:56:43 +0000 Subject: [PATCH] * elf64-ppc.c (opd_entry_value): Don't use cached relocs if they have been adjusted for output. (ppc64_elf_relocate_section): Mark sections with reloc_done. --- bfd/ChangeLog | 6 ++++++ bfd/elf64-ppc.c | 32 ++++++++++++++++++++++++++------ 2 files changed, 32 insertions(+), 6 deletions(-) diff --git a/bfd/ChangeLog b/bfd/ChangeLog index fe1087411bb..e97ea6e3bca 100644 --- a/bfd/ChangeLog +++ b/bfd/ChangeLog @@ -1,3 +1,9 @@ +2004-12-11 Alan Modra + + * elf64-ppc.c (opd_entry_value): Don't use cached relocs if they + have been adjusted for output. + (ppc64_elf_relocate_section): Mark sections with reloc_done. + 2004-12-10 Alan Modra * elf.c (bfd_elf_sym_name): Rename from bfd_elf_local_sym_name and diff --git a/bfd/elf64-ppc.c b/bfd/elf64-ppc.c index 9c975c8a470..08cfdac480c 100644 --- a/bfd/elf64-ppc.c +++ b/bfd/elf64-ppc.c @@ -4696,7 +4696,9 @@ opd_entry_value (asection *opd_sec, bfd_vma *code_off) { bfd *opd_bfd = opd_sec->owner; + Elf_Internal_Rela *relocs, *save_relocs = NULL; Elf_Internal_Rela *lo, *hi, *look; + bfd_vma val; /* No relocs implies we are linking a --just-symbols object. */ if (opd_sec->reloc_count == 0) @@ -4724,11 +4726,20 @@ opd_entry_value (asection *opd_sec, return val; } + /* If the .opd relocs have been adjusted for output, then we need to + re-read the original relocs rather than use the cached ones. */ + if (opd_sec->reloc_done) + { + save_relocs = elf_section_data (opd_sec)->relocs; + elf_section_data (opd_sec)->relocs = NULL; + } + /* Go find the opd reloc at the sym address. */ - lo = _bfd_elf_link_read_relocs (opd_bfd, opd_sec, NULL, NULL, TRUE); + lo = relocs = _bfd_elf_link_read_relocs (opd_bfd, opd_sec, NULL, NULL, + !opd_sec->reloc_done); BFD_ASSERT (lo != NULL); hi = lo + opd_sec->reloc_count - 1; /* ignore last reloc */ - + val = (bfd_vma) -1; while (lo < hi) { look = lo + (hi - lo) / 2; @@ -4743,7 +4754,6 @@ opd_entry_value (asection *opd_sec, && ELF64_R_TYPE ((look + 1)->r_info) == R_PPC64_TOC) { unsigned long symndx = ELF64_R_SYM (look->r_info); - bfd_vma val; asection *sec; if (symndx < symtab_hdr->sh_info) @@ -4757,7 +4767,7 @@ opd_entry_value (asection *opd_sec, symtab_hdr->sh_info, 0, NULL, NULL, NULL); if (sym == NULL) - return (bfd_vma) -1; + break; symtab_hdr->contents = (bfd_byte *) sym; } @@ -4792,12 +4802,17 @@ opd_entry_value (asection *opd_sec, *code_sec = sec; if (sec != NULL && sec->output_section != NULL) val += sec->output_section->vma + sec->output_offset; - return val; } break; } } - return (bfd_vma) -1; + + if (opd_sec->reloc_done) + { + elf_section_data (opd_sec)->relocs = save_relocs; + free (relocs); + } + return val; } /* Return the section that should be marked against GC for a given @@ -10422,6 +10437,11 @@ ppc64_elf_relocate_section (bfd *output_bfd, } } + /* If we're emitting relocations, then shortly after this function + returns, reloc offsets and addends for this section will be + adjusted. Worse, reloc symbol indices will be for the output + file rather than the input. Let opd_entry_value know. */ + input_section->reloc_done = info->emitrelocations; return ret; } -- 2.30.2