diff options
| author | Maciej W. Rozycki <macro@imgtec.com> | 2017-05-19 15:11:31 +0100 |
|---|---|---|
| committer | Maciej W. Rozycki <macro@imgtec.com> | 2017-05-20 00:31:39 +0100 |
| commit | e5713223cbc1025f484fa2548d58f74173646b8a (patch) | |
| tree | 3bc243c6e43c0f5f0f1b7724a586181a15e7c66b /binutils/ChangeLog | |
| parent | f0681695daa9d4fd493f3ca88f194675c9affbff (diff) | |
| download | gdb-e5713223cbc1025f484fa2548d58f74173646b8a.zip gdb-e5713223cbc1025f484fa2548d58f74173646b8a.tar.gz gdb-e5713223cbc1025f484fa2548d58f74173646b8a.tar.bz2 | |
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.
Diffstat (limited to 'binutils/ChangeLog')
| -rw-r--r-- | binutils/ChangeLog | 11 |
1 files changed, 11 insertions, 0 deletions
diff --git a/binutils/ChangeLog b/binutils/ChangeLog index b8e94d2..59e3bb3 100644 --- a/binutils/ChangeLog +++ b/binutils/ChangeLog @@ -1,5 +1,16 @@ 2017-05-19 Maciej W. Rozycki <macro@imgtec.com> + * 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. + +2017-05-19 Maciej W. Rozycki <macro@imgtec.com> + * testsuite/lib/utils-lib.exp (run_dump_test): Handle the `dump' option. |