]> git.ipfire.org Git - thirdparty/binutils-gdb.git/commit
binutils/testsuite: Support REL and MIPS64 reloc formats with `strip-13'
authorMaciej W. Rozycki <macro@mips.com>
Thu, 5 Apr 2018 12:08:35 +0000 (13:08 +0100)
committerMaciej W. Rozycki <macro@mips.com>
Thu, 5 Apr 2018 12:08:35 +0000 (13:08 +0100)
commit2f8ceb38991ea2ee595afbdd5da51ba1a53cbec7
tree3f6f5a6acedcffab41eeeb0e078f440b2c8cafbc
parentaad7be454bb3c8e2dd44c10cc16c80bacf8f04c0
binutils/testsuite: Support REL and MIPS64 reloc formats with `strip-13'

Add source variants for the `strip-13' test that produce relocations in
the REL and MIPS64 formats, fixing a failure for the `mips64el-openbsd'
target.  This also corrects output for `i*86-*', `i960-*', `m6812-*' and
`m68hc12-*', o32 `mips*-*', and `score*-*' targets, which however does
not show up as a test result change due to lax error message matching
causing `bad value' previously produced by `strip' as a result of input
file rejection to be accepted as a test pass.

For `m6811-*' aka `m68hc11-*' targets this causes a phantom regression,
because they use 16-bit addressing and therefore `.dc.a' emits 16-bit
quantities causing relocation data constructed in assembly not to be as
expected.  Previously input was rejected by `strip' with a `bad value'
message and now it is accepted, however due to the relocation data error
the relocation number is not one of the unsupported ones and the tool
completes successfully, which scores as a test failure.

Disable the test case for `m6811-*' and `m68hc11-*' targets then, as it
is a test case bug rather than a problem with the relevant backend.  A
separate change to the test case is required to correct this problem, at
which point the test case can be enabled for the affected targets.

binutils/
* testsuite/binutils-all/strip-13.s: Rename to...
* testsuite/binutils-all/strip-13rela.s: ... this.
* testsuite/binutils-all/strip-13rel.s: New test source.
* testsuite/binutils-all/strip-13mips64.s: New test source.
* testsuite/binutils-all/strip-13.d: Remove `arm-*', `d10v-*',
`dlx-*' and `xgate-*' from `not-target' list.  Add `m6811-*' and
`m68hc11-*' to `not-target' list.
* testsuite/binutils-all/objcopy.exp: Switch between sources for
`strip-13'.
binutils/ChangeLog
binutils/testsuite/binutils-all/objcopy.exp
binutils/testsuite/binutils-all/strip-13.d
binutils/testsuite/binutils-all/strip-13mips64.s [new file with mode: 0644]
binutils/testsuite/binutils-all/strip-13rel.s [new file with mode: 0644]
binutils/testsuite/binutils-all/strip-13rela.s [moved from binutils/testsuite/binutils-all/strip-13.s with 100% similarity]