[binutils-gdb] binutils/testsuite: Enable `strip-13' test for `hppa*-*'

Maciej W.Rozycki macro@sourceware.org
Thu Apr 5 12:10:00 GMT 2018


https://sourceware.org/git/gitweb.cgi?p=binutils-gdb.git;h=aad7be454bb3c8e2dd44c10cc16c80bacf8f04c0

commit aad7be454bb3c8e2dd44c10cc16c80bacf8f04c0
Author: Maciej W. Rozycki <macro@mips.com>
Date:   Thu Apr 5 13:08:35 2018 +0100

    binutils/testsuite: Enable `strip-13' test for `hppa*-*'
    
    Based on relocations defined in include/elf/*.h files we have relocation
    numbers: 143, 159, 214 and 215 currently not used by any of our ELF
    targets.  Use 143 then instead of 241 to enable the `strip-13' test for
    `hppa*-*' targets.  It has a side effect with some targets of verifying
    that unused relocations whose numbers are below the respective R_*_max
    value are handled correctly.
    
    	binutils/
    	* testsuite/binutils-all/strip-13.s: Use 143 (0x8f) rather than
    	241 (0xf1) for the relocation number and RELA addend.
    	* testsuite/binutils-all/strip-13.d: Remove `hppa*-*' from the
    	`not-target' list.

Diff:
---
 binutils/ChangeLog                         | 7 +++++++
 binutils/testsuite/binutils-all/strip-13.d | 3 +--
 binutils/testsuite/binutils-all/strip-13.s | 4 ++--
 3 files changed, 10 insertions(+), 4 deletions(-)

diff --git a/binutils/ChangeLog b/binutils/ChangeLog
index 0f8776c..23598a0 100644
--- a/binutils/ChangeLog
+++ b/binutils/ChangeLog
@@ -1,3 +1,10 @@
+2018-04-05  Maciej W. Rozycki  <macro@mips.com>
+
+	* testsuite/binutils-all/strip-13.s: Use 143 (0x8f) rather than
+	241 (0xf1) for the relocation number and RELA addend.
+	* testsuite/binutils-all/strip-13.d: Remove `hppa*-*' from the
+	`not-target' list.
+
 2018-04-04  John David Anglin  <danglin@gcc.gnu.org>
 
 	* MAINTAINERS: Update my e-mail address.
diff --git a/binutils/testsuite/binutils-all/strip-13.d b/binutils/testsuite/binutils-all/strip-13.d
index a34e1eb..48549c7 100644
--- a/binutils/testsuite/binutils-all/strip-13.d
+++ b/binutils/testsuite/binutils-all/strip-13.d
@@ -1,9 +1,8 @@
 #PROG: strip
 #strip: -g
 #error: .* bad value
-#not-target: arm-* d10v-* dlx-* h8300-* hppa*-* ip2k-* rx-* xgate-*
+#not-target: arm-* d10v-* dlx-* h8300-* ip2k-* rx-* xgate-*
 # The D10V, DLX and XGATE targets only support REL relocations but this test uses RELA relocations.
 # The ARM target does support both types, but defaults to REL.
-# The HPPA targets use reloc 241, which is the value this test uses as its "unknown" reloc.
 # The H8300-*, IP2K and RX targets do not complain about unrecognised relocs, unless they are actually used
 #  (which is what should really happen with the other targets...)
diff --git a/binutils/testsuite/binutils-all/strip-13.s b/binutils/testsuite/binutils-all/strip-13.s
index 5da2426..886c0ee 100644
--- a/binutils/testsuite/binutils-all/strip-13.s
+++ b/binutils/testsuite/binutils-all/strip-13.s
@@ -4,8 +4,8 @@ foo:
 
 	.section .rela.text
 	.dc.a	 0
-	.dc.a    0x000000f1
-	.dc.a	 0x000000f1
+	.dc.a    0x0000008f
+	.dc.a	 0x0000008f
 
 	.dc.a	 0
 	.dc.a    0



More information about the Binutils-cvs mailing list