bfd/

	* elf32-arm.c (elf32_arm_stub_long_branch_any_any_pic): Rename
	to elf32_arm_stub_long_branch_any_arm_pic.
	(elf32_arm_stub_long_branch_any_thumb_pic): New.
	(enum elf32_arm_stub_type, arm_type_of_stub)
	(arm_size_one_stub): Handle any to ARM PIC and any to Thumb PIC
	separately.

	ld/testsuite/
	* ld-arm/farcall-arm-thumb-blx-pic-veneer.d,
	ld-arm/farcall-arm-thumb-pic-veneer.d,
	ld-arm/farcall-thumb-thumb-blx-pic-veneer.d: Update for fixed
	Thumb PIC stub.

diff --git a/bfd/ChangeLog b/bfd/ChangeLog
index f40acec..e909aed 100644 (file)
--- a/bfd/ChangeLog
+++ b/bfd/ChangeLog
@@ -1,3 +1,12 @@
+2009-02-24  Daniel Jacobowitz  <dan@codesourcery.com>
+
+       * elf32-arm.c (elf32_arm_stub_long_branch_any_any_pic): Rename
+       to elf32_arm_stub_long_branch_any_arm_pic.
+       (elf32_arm_stub_long_branch_any_thumb_pic): New.
+       (enum elf32_arm_stub_type, arm_type_of_stub)
+       (arm_size_one_stub): Handle any to ARM PIC and any to Thumb PIC
+       separately.
+
 2009-02-24  Joseph Myers  <joseph@codesourcery.com>
 
        * elf32-arm.c (PREV_SEC): Update comment.

diff --git a/bfd/elf32-arm.c b/bfd/elf32-arm.c
index 8424322..7499f80 100644 (file)
--- a/bfd/elf32-arm.c
+++ b/bfd/elf32-arm.c
@@ -2089,15 +2089,27 @@ static const insn_sequence elf32_arm_stub_short_branch_v4t_thumb_arm[] =
     ARM_REL_INSN(0xea000000, -8),     /* b    (X-8) */
   };
 
-/* ARM/Thumb -> ARM/Thumb long branch stub, PIC. On V5T and above, use
+/* ARM/Thumb -> ARM long branch stub, PIC.  On V5T and above, use
    blx to reach the stub if necessary.  */
-static const insn_sequence elf32_arm_stub_long_branch_any_any_pic[] =
+static const insn_sequence elf32_arm_stub_long_branch_any_arm_pic[] =
   {
     ARM_INSN(0xe59fc000),             /* ldr   r12, [pc] */
     ARM_INSN(0xe08ff00c),             /* add   pc, pc, ip */
     DATA_WORD(0, R_ARM_REL32, -4),    /* dcd   R_ARM_REL32(X-4) */
   };
 
+/* ARM/Thumb -> Thumb long branch stub, PIC.  On V5T and above, use
+   blx to reach the stub if necessary.  We can not add into pc;
+   it is not guaranteed to mode switch (different in ARMv6 and
+   ARMv7).  */
+static const insn_sequence elf32_arm_stub_long_branch_any_thumb_pic[] =
+  {
+    ARM_INSN(0xe59fc004),             /* ldr   r12, [pc, #4] */
+    ARM_INSN(0xe08fc00c),             /* add   ip, pc, ip */
+    ARM_INSN(0xe12fff1c),             /* bx    ip */
+    DATA_WORD(0, R_ARM_REL32, 0),     /* dcd   R_ARM_REL32(X) */
+  };
+
 /* Section name for stubs is the associated section name plus this
    string.  */
 #define STUB_SUFFIX ".stub"
@@ -2110,7 +2122,8 @@ enum elf32_arm_stub_type
   arm_stub_long_branch_thumb_only,
   arm_stub_long_branch_v4t_thumb_arm,
   arm_stub_short_branch_v4t_thumb_arm,
-  arm_stub_long_branch_any_any_pic,
+  arm_stub_long_branch_any_arm_pic,
+  arm_stub_long_branch_any_thumb_pic,
 };
 
 struct elf32_arm_stub_hash_entry
@@ -2858,7 +2871,7 @@ arm_type_of_stub (struct bfd_link_info *info,
                    /* PIC stubs.  */
                    ? ((globals->use_blx)
                       /* V5T and above.  */
-                      ? arm_stub_long_branch_any_any_pic
+                      ? arm_stub_long_branch_any_thumb_pic
                       /* not yet supported on V4T.  */
                       : arm_stub_none)
 
@@ -2895,7 +2908,7 @@ arm_type_of_stub (struct bfd_link_info *info,
                /* PIC stubs.  */
                ? ((globals->use_blx)
                   /* V5T and above.  */
-                  ? arm_stub_long_branch_any_any_pic
+                  ? arm_stub_long_branch_any_arm_pic
                   /* not yet supported on V4T.  */
                   : arm_stub_none)
 
@@ -2938,7 +2951,7 @@ arm_type_of_stub (struct bfd_link_info *info,
            {
              stub_type = (info->shared | globals->pic_veneer)
                /* PIC stubs.  */
-               ? arm_stub_long_branch_any_any_pic
+               ? arm_stub_long_branch_any_thumb_pic
                /* non-PIC stubs.  */
                : ((globals->use_blx)
                   /* V5T and above.  */
@@ -2955,7 +2968,7 @@ arm_type_of_stub (struct bfd_link_info *info,
            {
              stub_type = (info->shared | globals->pic_veneer)
                /* PIC stubs.  */
-               ? arm_stub_long_branch_any_any_pic
+               ? arm_stub_long_branch_any_arm_pic
                /* non-PIC stubs.  */
                : arm_stub_long_branch_any_any;
            }
@@ -3277,9 +3290,13 @@ arm_size_one_stub (struct bfd_hash_entry *gen_entry,
       template =  elf32_arm_stub_short_branch_v4t_thumb_arm;
       template_size = sizeof (elf32_arm_stub_short_branch_v4t_thumb_arm) / sizeof (insn_sequence);
       break;
-    case arm_stub_long_branch_any_any_pic:
-      template = elf32_arm_stub_long_branch_any_any_pic;
-      template_size = sizeof (elf32_arm_stub_long_branch_any_any_pic) / sizeof (insn_sequence);
+    case arm_stub_long_branch_any_arm_pic:
+      template = elf32_arm_stub_long_branch_any_arm_pic;
+      template_size = sizeof (elf32_arm_stub_long_branch_any_arm_pic) / sizeof (insn_sequence);
+      break;
+    case arm_stub_long_branch_any_thumb_pic:
+      template = elf32_arm_stub_long_branch_any_thumb_pic;
+      template_size = sizeof (elf32_arm_stub_long_branch_any_thumb_pic) / sizeof (insn_sequence);
       break;
     default:
       BFD_FAIL ();

diff --git a/ld/testsuite/ChangeLog b/ld/testsuite/ChangeLog
index cdbc267..16a9a03 100644 (file)
--- a/ld/testsuite/ChangeLog
+++ b/ld/testsuite/ChangeLog
@@ -1,5 +1,12 @@
 2009-02-24  Daniel Jacobowitz  <dan@codesourcery.com>
 
+       * ld-arm/farcall-arm-thumb-blx-pic-veneer.d,
+       ld-arm/farcall-arm-thumb-pic-veneer.d,
+       ld-arm/farcall-thumb-thumb-blx-pic-veneer.d: Update for fixed
+       Thumb PIC stub.
+
+2009-02-24  Daniel Jacobowitz  <dan@codesourcery.com>
+
        * ld-arm/arm-elf.exp (armeabitests): Update duplicate test names.
        Use normal output files for big-endian.
        * ld-arm/farcall-arm-arm-be.d, ld-arm/farcall-thumb-arm-be.d: Delete.

diff --git a/ld/testsuite/ld-arm/farcall-arm-thumb-blx-pic-veneer.d b/ld/testsuite/ld-arm/farcall-arm-thumb-blx-pic-veneer.d
index 15c2274..45bc01c 100644 (file)
--- a/ld/testsuite/ld-arm/farcall-arm-thumb-blx-pic-veneer.d
+++ b/ld/testsuite/ld-arm/farcall-arm-thumb-blx-pic-veneer.d
@@ -7,10 +7,10 @@ Disassembly of section .text:
     1004:      00000000        andeq   r0, r0, r0
 
 00001008 <__bar_from_arm>:
-    1008:      e59fc000        ldr     ip, \[pc, #0\]  ; 1010 <__bar_from_arm\+0x8>
-    100c:      e08ff00c        add     pc, pc, ip
-    1010:      02000001        .word   0x02000001
-    1014:      00000000        .word   0x00000000
+    1008:      e59fc004        ldr     ip, \[pc, #4\]  ; 1014 <__bar_from_arm\+0xc>
+    100c:      e08fc00c        add     ip, pc, ip
+    1010:      e12fff1c        bx      ip
+    1014:      02000001        .word   0x02000001
 Disassembly of section .foo:
 
 02001014 <bar>:

diff --git a/ld/testsuite/ld-arm/farcall-arm-thumb-pic-veneer.d b/ld/testsuite/ld-arm/farcall-arm-thumb-pic-veneer.d
index 15c2274..45bc01c 100644 (file)
--- a/ld/testsuite/ld-arm/farcall-arm-thumb-pic-veneer.d
+++ b/ld/testsuite/ld-arm/farcall-arm-thumb-pic-veneer.d
@@ -7,10 +7,10 @@ Disassembly of section .text:
     1004:      00000000        andeq   r0, r0, r0
 
 00001008 <__bar_from_arm>:
-    1008:      e59fc000        ldr     ip, \[pc, #0\]  ; 1010 <__bar_from_arm\+0x8>
-    100c:      e08ff00c        add     pc, pc, ip
-    1010:      02000001        .word   0x02000001
-    1014:      00000000        .word   0x00000000
+    1008:      e59fc004        ldr     ip, \[pc, #4\]  ; 1014 <__bar_from_arm\+0xc>
+    100c:      e08fc00c        add     ip, pc, ip
+    1010:      e12fff1c        bx      ip
+    1014:      02000001        .word   0x02000001
 Disassembly of section .foo:
 
 02001014 <bar>:

diff --git a/ld/testsuite/ld-arm/farcall-thumb-thumb-blx-pic-veneer.d b/ld/testsuite/ld-arm/farcall-thumb-thumb-blx-pic-veneer.d
index d5bc3b1..86c0ca0 100644 (file)
--- a/ld/testsuite/ld-arm/farcall-thumb-thumb-blx-pic-veneer.d
+++ b/ld/testsuite/ld-arm/farcall-thumb-thumb-blx-pic-veneer.d
@@ -8,10 +8,10 @@ Disassembly of section .text:
        \.\.\.
 
 00001008 <__bar_veneer>:
-    1008:      e59fc000        ldr     ip, \[pc, #0\]  ; 1010 <__bar_veneer\+0x8>
-    100c:      e08ff00c        add     pc, pc, ip
-    1010:      02000001        .word   0x02000001
-    1014:      00000000        .word   0x00000000
+    1008:      e59fc004        ldr     ip, \[pc, #4\]  ; 1014 <__bar_veneer\+0xc>
+    100c:      e08fc00c        add     ip, pc, ip
+    1010:      e12fff1c        bx      ip
+    1014:      02000001        .word   0x02000001
 Disassembly of section .foo:
 
 02001014 <bar>: