[binutils-gdb/binutils-2_38-branch] i386: Update I386_NEED_DYNAMIC_RELOC_TYPE_P for DT_TEXTREL

H.J. Lu hjl@sourceware.org
Tue Feb 15 23:16:22 GMT 2022


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

commit ae1cab7d3f39686d9c7ddadbac297e9ae76ffccf
Author: H.J. Lu <hjl.tools@gmail.com>
Date:   Tue Feb 15 15:03:02 2022 -0800

    i386: Update I386_NEED_DYNAMIC_RELOC_TYPE_P for DT_TEXTREL
    
    Update I386_NEED_DYNAMIC_RELOC_TYPE_P to allow R_386_TLS_IE for relocation
    in read-only section.
    
    bfd/
    
            PR ld/28894
            * elfxx-x86.h (I386_NEED_DYNAMIC_RELOC_TYPE_P): Allow
            R_386_TLS_IE.
    
    ld/
            PR ld/28894
            * testsuite/ld-i386/i386.exp: Run pr28894.
            * testsuite/ld-i386/pr28894.d: New file.
            * testsuite/ld-i386/pr28894.s: Likewise.
    
    (cherry picked from commit 3319ba7a8d6a42ba071d691789bc184632399331)

Diff:
---
 bfd/elfxx-x86.h                | 1 +
 ld/testsuite/ld-i386/i386.exp  | 1 +
 ld/testsuite/ld-i386/pr28894.d | 3 +++
 ld/testsuite/ld-i386/pr28894.s | 7 +++++++
 4 files changed, 12 insertions(+)

diff --git a/bfd/elfxx-x86.h b/bfd/elfxx-x86.h
index 3219f363a9a..77fb1ad72bc 100644
--- a/bfd/elfxx-x86.h
+++ b/bfd/elfxx-x86.h
@@ -80,6 +80,7 @@
 #define I386_NEED_DYNAMIC_RELOC_TYPE_P(TYPE) \
   (I386_SIZE_TYPE_P (TYPE) \
    || I386_RELATIVE_RELOC_TYPE_P (TYPE) \
+   || (TYPE) == R_386_TLS_IE \
    || (TYPE) == R_386_TLS_LE \
    || (TYPE) == R_386_TLS_LE_32)
 #define X86_NEED_DYNAMIC_RELOC_TYPE_P(IS_X86_64, TYPE) \
diff --git a/ld/testsuite/ld-i386/i386.exp b/ld/testsuite/ld-i386/i386.exp
index d997c757325..fadbd160420 100644
--- a/ld/testsuite/ld-i386/i386.exp
+++ b/ld/testsuite/ld-i386/i386.exp
@@ -510,6 +510,7 @@ run_dump_test "pr27491-4"
 run_dump_test "dt-relr-1a"
 run_dump_test "dt-relr-1b"
 run_dump_test "pr28870"
+run_dump_test "pr28894"
 
 if { !([istarget "i?86-*-linux*"]
        || [istarget "i?86-*-gnu*"]
diff --git a/ld/testsuite/ld-i386/pr28894.d b/ld/testsuite/ld-i386/pr28894.d
new file mode 100644
index 00000000000..a8d1111eb1f
--- /dev/null
+++ b/ld/testsuite/ld-i386/pr28894.d
@@ -0,0 +1,3 @@
+#as: --32
+#ld: -shared -melf_i386 --warn-shared-textrel --fatal-warnings
+#error: .*warning: creating DT_TEXTREL in a shared object
diff --git a/ld/testsuite/ld-i386/pr28894.s b/ld/testsuite/ld-i386/pr28894.s
new file mode 100644
index 00000000000..1d3e3f37f3e
--- /dev/null
+++ b/ld/testsuite/ld-i386/pr28894.s
@@ -0,0 +1,7 @@
+	.text
+	addl	foo@INDNTPOFF, %eax
+	.section .tbss,"awT",@nobits
+	.globl	foo
+	.hidden foo
+foo:
+	.byte 0


More information about the Binutils-cvs mailing list