Create PLT eh_frame section if there is .eh_frame section
authorH.J. Lu <hjl.tools@gmail.com>
Tue, 22 May 2012 14:46:44 +0000 (14:46 +0000)
committerH.J. Lu <hjl.tools@gmail.com>
Tue, 22 May 2012 14:46:44 +0000 (14:46 +0000)
bfd/

PR ld/13909
* elf32-i386.c (elf_i386_create_dynamic_sections): Create PLT
eh_frame section if there is an input .eh_frame section.
* elf64-x86-64.c (elf_x86_64_create_dynamic_sections): Likewise.

ld/testsuite/

PR ld/13909
* ld-i386/i386.exp: Run pr13909.
* ld-x86-64/x86-64.exp: Likewise.

* ld-i386/dummy.s: New file.
* ld-i386/pr13909.d: Likewise.
* ld-x86-64/pr13909.d: Likewise.

bfd/ChangeLog
bfd/elf32-i386.c
bfd/elf64-x86-64.c
ld/testsuite/ChangeLog
ld/testsuite/ld-i386/dummy.s [new file with mode: 0644]
ld/testsuite/ld-i386/i386.exp
ld/testsuite/ld-i386/pr13909.d [new file with mode: 0644]
ld/testsuite/ld-x86-64/pr13909.d [new file with mode: 0644]
ld/testsuite/ld-x86-64/x86-64.exp

index cfdcaa3..6271710 100644 (file)
@@ -1,5 +1,12 @@
 2012-05-22  H.J. Lu  <hongjiu.lu@intel.com>
 
+       PR ld/13909
+       * elf32-i386.c (elf_i386_create_dynamic_sections): Create PLT
+       eh_frame section if there is an input .eh_frame section.
+       * elf64-x86-64.c (elf_x86_64_create_dynamic_sections): Likewise.
+
+2012-05-22  H.J. Lu  <hongjiu.lu@intel.com>
+
        PR ld/14105
        * elf32-i386.c (elf_i386_create_dynamic_sections): Always
        create PLT eh_frame section with SEC_LINKER_CREATED.
index 84299ce..e0d7fff 100644 (file)
@@ -1016,7 +1016,8 @@ elf_i386_create_dynamic_sections (bfd *dynobj, struct bfd_link_info *info)
 
   if (!info->no_ld_generated_unwind_info
       && htab->plt_eh_frame == NULL
-      && htab->elf.splt != NULL)
+      && htab->elf.splt != NULL
+      && bfd_get_section_by_name (dynobj, ".eh_frame") != NULL)
     {
       flagword flags = get_elf_backend_data (dynobj)->dynamic_sec_flags;
       htab->plt_eh_frame
index 1f490d4..27354df 100644 (file)
@@ -980,7 +980,8 @@ elf_x86_64_create_dynamic_sections (bfd *dynobj,
 
   if (!info->no_ld_generated_unwind_info
       && htab->plt_eh_frame == NULL
-      && htab->elf.splt != NULL)
+      && htab->elf.splt != NULL
+      && bfd_get_section_by_name (dynobj, ".eh_frame") != NULL)
     {
       const struct elf_x86_64_backend_data *const abed
        = get_elf_x86_64_backend_data (dynobj);
index e0d072f..b05b8d7 100644 (file)
@@ -1,4 +1,14 @@
-2012-05-20  H.J. Lu  <hongjiu.lu@intel.com>
+2012-05-22  H.J. Lu  <hongjiu.lu@intel.com>
+
+       PR ld/13909
+       * ld-i386/i386.exp: Run pr13909.
+       * ld-x86-64/x86-64.exp: Likewise.
+
+       * ld-i386/dummy.s: New file.
+       * ld-i386/pr13909.d: Likewise.
+       * ld-x86-64/pr13909.d: Likewise.
+
+2012-05-22  H.J. Lu  <hongjiu.lu@intel.com>
 
        PR ld/14105
        * ld-elf/eh4.d: Add PLT eh_frame.
diff --git a/ld/testsuite/ld-i386/dummy.s b/ld/testsuite/ld-i386/dummy.s
new file mode 100644 (file)
index 0000000..403f980
--- /dev/null
@@ -0,0 +1 @@
+# Dummy
index 47f918f..982e09f 100644 (file)
@@ -233,6 +233,7 @@ run_dump_test "pr12718"
 run_dump_test "pr12921"
 run_dump_test "pr12570a"
 run_dump_test "pr12570b"
+run_dump_test "pr13909"
 
 if { !([istarget "i?86-*-linux*"]
        || [istarget "i?86-*-gnu*"]
diff --git a/ld/testsuite/ld-i386/pr13909.d b/ld/testsuite/ld-i386/pr13909.d
new file mode 100644 (file)
index 0000000..388fd56
--- /dev/null
@@ -0,0 +1,10 @@
+#name: PR ld/13909
+#source: dummy.s
+#as: --32
+#ld: -melf_i386 -shared
+#readelf: -S --wide
+
+#failif
+#...
+ +\[ [0-9]\] .eh_frame +PROGBITS +[0-9a-f]+ +[0-9a-f]+ 0+ +.*
+#...
diff --git a/ld/testsuite/ld-x86-64/pr13909.d b/ld/testsuite/ld-x86-64/pr13909.d
new file mode 100644 (file)
index 0000000..137c61f
--- /dev/null
@@ -0,0 +1,10 @@
+#name: PR ld/13909
+#source: dummy.s
+#as: --64
+#ld: -melf_x86_64 -shared
+#readelf: -S --wide
+
+#failif
+#...
+ +\[ [0-9]\] .eh_frame +PROGBITS +[0-9a-f]+ +[0-9a-f]+ 0+ +.*
+#...
index 7d2934f..42203bb 100644 (file)
@@ -208,6 +208,7 @@ run_dump_test "pr12921"
 run_dump_test "pr13947"
 run_dump_test "pr12570a"
 run_dump_test "pr12570b"
+run_dump_test "pr13909"
 
 if { ![istarget "x86_64-*-linux*"] && ![istarget "x86_64-*-nacl*"]} {
     return