MIPS/BFD: For n64 hold the number of internal relocs in `->reloc_count'
authorMaciej W. Rozycki <macro@imgtec.com>
Fri, 19 May 2017 14:11:31 +0000 (15:11 +0100)
committerMaciej W. Rozycki <macro@imgtec.com>
Fri, 19 May 2017 23:31:39 +0000 (00:31 +0100)
commite5713223cbc1025f484fa2548d58f74173646b8a
tree3bc243c6e43c0f5f0f1b7724a586181a15e7c66b
parentf0681695daa9d4fd493f3ca88f194675c9affbff
MIPS/BFD: For n64 hold the number of internal relocs in `->reloc_count'

Revert parts of commit fee24f1c5bfe ("objdump improvements for mips
elf64"), <https://sourceware.org/ml/binutils/2003-03/msg00108.html>, and
make the `->reloc_count' member of `struct bfd_section' hold the actual
number of internal relocations stored in its `->relocation' vector.  To
do so adjust `mips_elf64_slurp_one_reloc_table' to set `->reloc_count'
to the actual number of internal relocations retrieved and discard
`mips_elf64_canonicalize_reloc', `mips_elf64_canonicalize_dynamic_reloc'
and their corresponding target macros.  Contrary to the description of
`mips_elf64_slurp_one_reloc_table', adjusted appropriately, this makes
generic relocation processing code happy and satisfies the "merge notes
section" binutils test case.

Add extra binutils test cases to expand the coverage of the generic
"merge notes section" test case, now passing with the n64 ABI, across
the MIPS o32, n32 and n64 ABIs regardless of the default ABI selected in
target configuration, and also to verify correctness of the relocations
produced.  Conversely, do not provide any additional test cases for the
original issue addressed with the commit referred:

- objdump would display only 1/3 of the total number of relocations,
  because it used the external relocation count, but each external
  relocation is brought in as 3 internal relocations.

as n64 ABI relocation processing with `objdump -r' and `objdump -R' is
already widely covered across the GAS and LD test suites.

bfd/
* elf64-mips.c (mips_elf64_canonicalize_reloc): Remove prototype
and function.
(mips_elf64_canonicalize_dynamic_reloc): Likewise.
(mips_elf64_slurp_one_reloc_table): Set `reloc_count' to the
actual number of internal relocations retrieved.  Adjust
function description.
(bfd_elf64_canonicalize_reloc): Remove macro.
(bfd_elf64_canonicalize_dynamic_reloc): Likewise.

binutils/
* testsuite/binutils-all/mips/mips-note-2.d: New test.
* testsuite/binutils-all/mips/mips-note-2r.d: New test.
* testsuite/binutils-all/mips/mips-note-2-n32.d: New test.
* testsuite/binutils-all/mips/mips-note-2-n64.d: New test.
* testsuite/binutils-all/mips/mips-note-2r-n32.d: New test.
* testsuite/binutils-all/mips/mips-note-2r-n64.d: New test.
* testsuite/binutils-all/mips/mips.exp: Define `has_newabi'.
Run the new tests.
bfd/ChangeLog
bfd/elf64-mips.c
binutils/ChangeLog
binutils/testsuite/binutils-all/mips/mips-note-2-n32.d [new file with mode: 0644]
binutils/testsuite/binutils-all/mips/mips-note-2-n64.d [new file with mode: 0644]
binutils/testsuite/binutils-all/mips/mips-note-2.d [new file with mode: 0644]
binutils/testsuite/binutils-all/mips/mips-note-2r-n32.d [new file with mode: 0644]
binutils/testsuite/binutils-all/mips/mips-note-2r-n64.d [new file with mode: 0644]
binutils/testsuite/binutils-all/mips/mips-note-2r.d [new file with mode: 0644]
binutils/testsuite/binutils-all/mips/mips.exp