[binutils-gdb] bfd: xtensa: fix shrink_dynamic_reloc_sections for export-dynamic

[Max Filippov]

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

commit 2487ef07c28b961c6e2b8c51161f88f93b181d07
Author: Max Filippov <jcmvbkbc@gmail.com>
Date:   Thu Mar 28 17:03:57 2019 -0700

    bfd: xtensa: fix shrink_dynamic_reloc_sections for export-dynamic
    
    shrink_dynamic_reloc_sections must remove PLT entry that was created for
    an undefined weak symbol in the presence of --export-dynamic option when
    relaxation coalesces literals pointing to that symbol. This fixes the
    following assertion:
    
      ld: BFD (GNU Binutils) 2.31.1 internal error, aborting at
      elf32-xtensa.c:3292 in elf_xtensa_finish_dynamic_sections
    
    2019-03-29  Max Filippov  <jcmvbkbc@gmail.com>
    bfd/
    	* elf32-xtensa.c (shrink_dynamic_reloc_sections): Add
    	info->export_dynamic to the conditional.
    
    ld/
    	* testsuite/ld-xtensa/relax-undef-weak-pie-export-dynamic.d: New
    	test definition.
    	* testsuite/ld-xtensa/xtensa.exp
    	(relax-undef-weak-pie-export-dynamic): Add new test.

Diff:
---
 bfd/ChangeLog                                              |  5 +++++
 bfd/elf32-xtensa.c                                         |  3 ++-
 ld/ChangeLog                                               |  7 +++++++
 .../ld-xtensa/relax-undef-weak-pie-export-dynamic.d        | 14 ++++++++++++++
 ld/testsuite/ld-xtensa/xtensa.exp                          |  1 +
 5 files changed, 29 insertions(+), 1 deletion(-)

diff --git a/bfd/ChangeLog b/bfd/ChangeLog
index 4dd3f27..6662b70 100644
--- a/bfd/ChangeLog
+++ b/bfd/ChangeLog
@@ -1,3 +1,8 @@
+2019-03-29  Max Filippov  <jcmvbkbc@gmail.com>
+
+	* elf32-xtensa.c (shrink_dynamic_reloc_sections): Add
+	info->export_dynamic to the conditional.
+
 2019-03-28  Alan Modra  <amodra@gmail.com>
 
 	PR 24392
diff --git a/bfd/elf32-xtensa.c b/bfd/elf32-xtensa.c
index c3df3d6..37ea200 100644
--- a/bfd/elf32-xtensa.c
+++ b/bfd/elf32-xtensa.c
@@ -10083,7 +10083,8 @@ shrink_dynamic_reloc_sections (struct bfd_link_info *info,
       && (input_section->flags & SEC_ALLOC) != 0
       && (dynamic_symbol || bfd_link_pic (info))
       && (!h || h->root.type != bfd_link_hash_undefweak
-	  || (dynamic_symbol && bfd_link_dll (info))))
+	  || (dynamic_symbol
+	      && (bfd_link_dll (info) || info->export_dynamic))))
     {
       asection *srel;
       bfd_boolean is_plt = FALSE;
diff --git a/ld/ChangeLog b/ld/ChangeLog
index da1e8e1..b987a13 100644
--- a/ld/ChangeLog
+++ b/ld/ChangeLog
@@ -1,3 +1,10 @@
+2019-03-29  Max Filippov  <jcmvbkbc@gmail.com>
+
+	* testsuite/ld-xtensa/relax-undef-weak-pie-export-dynamic.d: New
+	test definition.
+	* testsuite/ld-xtensa/xtensa.exp
+	(relax-undef-weak-pie-export-dynamic): Add new test.
+
 2019-03-26  Martin Liska  <mliska@suse.cz>
 
 	* plugin.c (get_symbols): Add lto_kind_str, lto_resolution_str,
diff --git a/ld/testsuite/ld-xtensa/relax-undef-weak-pie-export-dynamic.d b/ld/testsuite/ld-xtensa/relax-undef-weak-pie-export-dynamic.d
new file mode 100644
index 0000000..e2ff601
--- /dev/null
+++ b/ld/testsuite/ld-xtensa/relax-undef-weak-pie-export-dynamic.d
@@ -0,0 +1,14 @@
+#source: relax-undef-weak.s
+#ld: -pie --export-dynamic
+#readelf: -r
+#...
+Relocation section '\.rela\.dyn' .* 3 .*
+#...
+.*R_XTENSA_RTLD.*
+.*R_XTENSA_RTLD.*
+.*R_XTENSA_GLOB_DAT.*fd.*
+#...
+Relocation section '\.rela\.plt' .* 1 .*
+#...
+.*R_XTENSA_JMP_SLOT.*fd.*
+#...
diff --git a/ld/testsuite/ld-xtensa/xtensa.exp b/ld/testsuite/ld-xtensa/xtensa.exp
index 91b2314..d800c64 100644
--- a/ld/testsuite/ld-xtensa/xtensa.exp
+++ b/ld/testsuite/ld-xtensa/xtensa.exp
@@ -34,6 +34,7 @@ run_dump_test "relax-static-shared"
 run_dump_test "relax-static-local-shared"
 
 run_dump_test "relax-undef-weak-pie"
+run_dump_test "relax-undef-weak-pie-export-dynamic"
 run_dump_test "relax-undef-weak-local-pie"
 run_dump_test "relax-undef-weak-shared"
 run_dump_test "relax-undef-weak-local-shared"