bfd/
authorDaniel Jacobowitz <drow@false.org>
Wed, 17 Nov 2004 17:50:28 +0000 (17:50 +0000)
committerDaniel Jacobowitz <drow@false.org>
Wed, 17 Nov 2004 17:50:28 +0000 (17:50 +0000)
* elf32-arm.c (PLT_THUMB_STUB_SIZE): Define.
(elf32_arm_plt_thumb_stub): New.
(struct elf32_arm_link_hash_entry): Add plt_thumb_refcount
and plt_got_offset.
(elf32_arm_link_hash_traverse): Fix typo.
(elf32_arm_link_hash_table): Add obfd.
(elf32_arm_link_hash_newfunc): Initialize new fields.
(elf32_arm_copy_indirect_symbol): Copy plt_thumb_refcount.
(elf32_arm_link_hash_table_create): Initialize obfd.
(record_arm_to_thumb_glue): Mark the glue as a local ARM function.
(record_thumb_to_arm_glue): Mark the glue as a local Thumb function.
(bfd_elf32_arm_get_bfd_for_interworking): Verify that the
interworking BFD is not dynamic.
(bfd_elf32_arm_process_before_allocation): Handle R_ARM_PLT32.  Do
not emit glue for PLT references.
(elf32_arm_final_link_relocate): Handle Thumb functions.  Do not
emit glue for PLT references.  Support the Thumb PLT prefix.
(elf32_arm_gc_sweep_hook): Handle R_ARM_THM_PC22 and
plt_thumb_refcount.
(elf32_arm_check_relocs): Likewise.
(elf32_arm_adjust_dynamic_symbol): Handle Thumb functions and
plt_thumb_refcount.
(allocate_dynrelocs): Handle Thumb PLT references.
(elf32_arm_finish_dynamic_symbol): Likewise.
(elf32_arm_symbol_processing): New function.
(elf_backend_symbol_processing): Define.
opcodes/
* arm-dis.c (WORD_ADDRESS): Define.
(print_insn): Use it.  Correct big-endian end-of-section handling.
gas/testsuite/
* gas/arm/mapping.d: Expect F markers for Thumb code.
* gas/arm/unwind.d: Update big-endian pattern.
ld/
* emultempl/armelf.em (arm_elf_set_bfd_for_interworking): Don't use
a dynamic object for stubs.
ld/testsuite/
* ld-arm/mixed-app.d, ld-arm/mixed-app.r, ld-arm/mixed-app.s,
ld-arm/mixed-app.sym, ld-arm/mixed-lib.d, ld-arm/mixed-lib.r,
ld-arm/mixed-lib.s, ld-arm/mixed-lib.sym, ld-arm/arm-dyn.ld,
ld-arm/arm-lib.ld: New files.
* ld-arm/arm-app-abs32.d, ld-arm/arm-app-abs32.r, ld-arm/arm-app.d,
ld-arm/arm-app.r, ld-arm/arm-lib-plt32.d, ld-arm/arm-lib-plt32.r,
ld-arm/arm-lib.d, ld-arm/arm-lib.r, ld-arm/arm-static-app.d,
ld-arm/arm-static-app.r: Update for big-endian.
* ld-arm/arm-elf.exp: Run the new tests.

31 files changed:
bfd/ChangeLog
bfd/elf32-arm.c
gas/testsuite/ChangeLog
gas/testsuite/gas/arm/mapping.d
gas/testsuite/gas/arm/unwind.d
ld/ChangeLog
ld/emultempl/armelf.em
ld/testsuite/ChangeLog
ld/testsuite/ld-arm/arm-app-abs32.d
ld/testsuite/ld-arm/arm-app-abs32.r
ld/testsuite/ld-arm/arm-app.d
ld/testsuite/ld-arm/arm-app.r
ld/testsuite/ld-arm/arm-dyn.ld [new file with mode: 0644]
ld/testsuite/ld-arm/arm-elf.exp
ld/testsuite/ld-arm/arm-lib-plt32.d
ld/testsuite/ld-arm/arm-lib-plt32.r
ld/testsuite/ld-arm/arm-lib.d
ld/testsuite/ld-arm/arm-lib.ld [new file with mode: 0644]
ld/testsuite/ld-arm/arm-lib.r
ld/testsuite/ld-arm/arm-static-app.d
ld/testsuite/ld-arm/arm-static-app.r
ld/testsuite/ld-arm/mixed-app.d [new file with mode: 0644]
ld/testsuite/ld-arm/mixed-app.r [new file with mode: 0644]
ld/testsuite/ld-arm/mixed-app.s [new file with mode: 0644]
ld/testsuite/ld-arm/mixed-app.sym [new file with mode: 0644]
ld/testsuite/ld-arm/mixed-lib.d [new file with mode: 0644]
ld/testsuite/ld-arm/mixed-lib.r [new file with mode: 0644]
ld/testsuite/ld-arm/mixed-lib.s [new file with mode: 0644]
ld/testsuite/ld-arm/mixed-lib.sym [new file with mode: 0644]
opcodes/ChangeLog
opcodes/arm-dis.c

index 1295b0370bc106cdb45c7443706428f5d9366f01..6d0cf3f99fd233fcdb6d7797940326a5d177281b 100644 (file)
@@ -1,3 +1,32 @@
+2004-11-17  Daniel Jacobowitz  <dan@codesourcery.com>
+
+       * elf32-arm.c (PLT_THUMB_STUB_SIZE): Define.
+       (elf32_arm_plt_thumb_stub): New.
+       (struct elf32_arm_link_hash_entry): Add plt_thumb_refcount
+       and plt_got_offset.
+       (elf32_arm_link_hash_traverse): Fix typo.
+       (elf32_arm_link_hash_table): Add obfd.
+       (elf32_arm_link_hash_newfunc): Initialize new fields.
+       (elf32_arm_copy_indirect_symbol): Copy plt_thumb_refcount.
+       (elf32_arm_link_hash_table_create): Initialize obfd.
+       (record_arm_to_thumb_glue): Mark the glue as a local ARM function.
+       (record_thumb_to_arm_glue): Mark the glue as a local Thumb function.
+       (bfd_elf32_arm_get_bfd_for_interworking): Verify that the
+       interworking BFD is not dynamic.
+       (bfd_elf32_arm_process_before_allocation): Handle R_ARM_PLT32.  Do
+       not emit glue for PLT references.
+       (elf32_arm_final_link_relocate): Handle Thumb functions.  Do not
+       emit glue for PLT references.  Support the Thumb PLT prefix.
+       (elf32_arm_gc_sweep_hook): Handle R_ARM_THM_PC22 and
+       plt_thumb_refcount.
+       (elf32_arm_check_relocs): Likewise.
+       (elf32_arm_adjust_dynamic_symbol): Handle Thumb functions and
+       plt_thumb_refcount.
+       (allocate_dynrelocs): Handle Thumb PLT references.
+       (elf32_arm_finish_dynamic_symbol): Likewise.
+       (elf32_arm_symbol_processing): New function.
+       (elf_backend_symbol_processing): Define.
+
 2004-11-16  Richard Sandiford  <rsandifo@redhat.com>
 
        * elf-bfd.h (eh_cie_fde): Add new fields: add_augmentation_size and
index 53b4ac11cdc7beaabd589fafb1b8dca748312b9f..65c087396b45f086057a6b8f311147b24b20928b 100644 (file)
@@ -1063,6 +1063,14 @@ static const bfd_vma elf32_arm_plt_entry [] =
 
 #endif
 
+/* An initial stub used if the PLT entry is referenced from Thumb code.  */
+#define PLT_THUMB_STUB_SIZE 4
+static const bfd_vma elf32_arm_plt_thumb_stub [] =
+  {
+    0x4778,            /* bx pc */
+    0x46c0             /* nop   */
+  };
+
 /* The entries in a PLT when using a DLL-based target with multiple
    address spaces.  */
 static const bfd_vma elf32_arm_symbian_plt_entry [] = 
@@ -1116,13 +1124,22 @@ struct elf32_arm_link_hash_entry
 
     /* Number of PC relative relocs copied for this symbol.  */
     struct elf32_arm_relocs_copied * relocs_copied;
+
+    /* We reference count Thumb references to a PLT entry separately,
+       so that we can emit the Thumb trampoline only if needed.  */
+    bfd_signed_vma plt_thumb_refcount;
+
+    /* Since PLT entries have variable size if the Thumb prologue is
+       used, we need to record the index into .got.plt instead of
+       recomputing it from the PLT offset.  */
+    bfd_signed_vma plt_got_offset;
   };
 
 /* Traverse an arm ELF linker hash table.  */
 #define elf32_arm_link_hash_traverse(table, func, info)                        \
   (elf_link_hash_traverse                                              \
    (&(table)->root,                                                    \
-    (bfd_boolean (*) (struct elf_link_hash_entry *, void *))) (func), \
+    (bfd_boolean (*) (struct elf_link_hash_entry *, void *)) (func),   \
     (info)))
 
 /* Get the ARM elf linker hash table from a link_info structure.  */
@@ -1178,6 +1195,9 @@ struct elf32_arm_link_hash_table
 
     /* Small local sym to section mapping cache.  */
     struct sym_sec_cache sym_sec;
+
+    /* For convenience in allocate_dynrelocs.  */
+    bfd * obfd;
   };
 
 /* Create an entry in an ARM ELF linker hash table.  */
@@ -1202,7 +1222,11 @@ elf32_arm_link_hash_newfunc (struct bfd_hash_entry * entry,
         _bfd_elf_link_hash_newfunc ((struct bfd_hash_entry *) ret,
                                     table, string));
   if (ret != NULL)
-    ret->relocs_copied = NULL;
+    {
+      ret->relocs_copied = NULL;
+      ret->plt_thumb_refcount = 0;
+      ret->plt_got_offset = -1;
+    }
 
   return (struct bfd_hash_entry *) ret;
 }
@@ -1315,6 +1339,17 @@ elf32_arm_copy_indirect_symbol (const struct elf_backend_data *bed,
       eind->relocs_copied = NULL;
     }
 
+  /* If the direct symbol already has an associated PLT entry, the
+     indirect symbol should not.  If it doesn't, swap refcount information
+     from the indirect symbol.  */
+  if (edir->plt_thumb_refcount == 0)
+    {
+      edir->plt_thumb_refcount = eind->plt_thumb_refcount;
+      eind->plt_thumb_refcount = 0;
+    }
+  else
+    BFD_ASSERT (eind->plt_thumb_refcount == 0);
+
   _bfd_elf_link_hash_copy_indirect (bed, dir, ind);
 }
 
@@ -1360,6 +1395,7 @@ elf32_arm_link_hash_table_create (bfd *abfd)
 #endif
   ret->symbian_p = 0;
   ret->sym_sec.abfd = NULL;
+  ret->obfd = abfd;
 
   return &ret->root.root;
 }
@@ -1559,6 +1595,10 @@ record_arm_to_thumb_glue (struct bfd_link_info * link_info,
                                    tmp_name, BSF_GLOBAL, s, val,
                                    NULL, TRUE, FALSE, &bh);
 
+  myh = (struct elf_link_hash_entry *) bh;
+  myh->type = ELF_ST_INFO (STB_LOCAL, STT_FUNC);
+  myh->forced_local = 1;
+
   free (tmp_name);
 
   globals->arm_glue_size += ARM2THUMB_GLUE_SIZE;
@@ -1576,7 +1616,6 @@ record_thumb_to_arm_glue (struct bfd_link_info *link_info,
   struct elf_link_hash_entry *myh;
   struct bfd_link_hash_entry *bh;
   struct elf32_arm_link_hash_table *hash_table;
-  char bind;
   bfd_vma val;
 
   hash_table = elf32_arm_hash_table (link_info);
@@ -1614,8 +1653,8 @@ record_thumb_to_arm_glue (struct bfd_link_info *link_info,
 
   /* If we mark it 'Thumb', the disassembler will do a better job.  */
   myh = (struct elf_link_hash_entry *) bh;
-  bind = ELF_ST_BIND (myh->type);
-  myh->type = ELF_ST_INFO (bind, STT_ARM_TFUNC);
+  myh->type = ELF_ST_INFO (STB_LOCAL, STT_ARM_TFUNC);
+  myh->forced_local = 1;
 
   free (tmp_name);
 
@@ -1713,6 +1752,9 @@ bfd_elf32_arm_get_bfd_for_interworking (bfd *abfd, struct bfd_link_info *info)
   if (info->relocatable)
     return TRUE;
 
+  /* Make sure we don't attach the glue sections to a dynamic object.  */
+  BFD_ASSERT (!(abfd->flags & DYNAMIC));
+
   globals = elf32_arm_hash_table (info);
 
   BFD_ASSERT (globals != NULL);
@@ -1796,6 +1838,7 @@ bfd_elf32_arm_process_before_allocation (bfd *abfd,
 
          /* These are the only relocation types we care about.  */
          if (   r_type != R_ARM_PC24
+             && r_type != R_ARM_PLT32
 #ifndef OLD_ARM_ABI
              && r_type != R_ARM_CALL
              && r_type != R_ARM_JUMP24
@@ -1834,6 +1877,11 @@ bfd_elf32_arm_process_before_allocation (bfd *abfd,
          if (h == NULL)
            continue;
 
+         /* If the call will go through a PLT entry then we do not need
+            glue.  */
+         if (globals->splt != NULL && h->plt.offset != (bfd_vma) -1)
+           continue;
+
          switch (r_type)
            {
            case R_ARM_PC24:
@@ -2374,6 +2422,8 @@ elf32_arm_final_link_relocate (reloc_howto_type *           howto,
 
              /* This symbol is local, or marked to become local.  */
              relocate = TRUE;
+             if (sym_flags == STT_ARM_TFUNC)
+               value |= 1;
              if (globals->symbian_p)
                {
                  /* On Symbian OS, the data segment and text segement
@@ -2652,8 +2702,11 @@ elf32_arm_final_link_relocate (reloc_howto_type *           howto,
          {
            /* If it is not a call to Thumb, assume call to Arm.
               If it is a call relative to a section name, then it is not a
-              function call at all, but rather a long jump.  */
-           if (sym_flags != STT_ARM_TFUNC && sym_flags != STT_SECTION)
+              function call at all, but rather a long jump.  Calls through
+              the PLT do not require stubs.  */
+           if (sym_flags != STT_ARM_TFUNC && sym_flags != STT_SECTION
+               && (h == NULL || splt == NULL
+                   || h->plt.offset == (bfd_vma) -1))
              {
                if (elf32_thumb_to_arm_stub
                    (info, sym_name, input_bfd, output_bfd, input_section,
@@ -2664,6 +2717,16 @@ elf32_arm_final_link_relocate (reloc_howto_type *           howto,
              }
          }
 
+       /* Handle calls via the PLT.  */
+       if (h != NULL && splt != NULL && h->plt.offset != (bfd_vma) -1)
+         {
+           value = (splt->output_section->vma
+                    + splt->output_offset
+                    + h->plt.offset);
+           /* Target the Thumb stub before the ARM PLT entry.  */
+           value -= 4;
+         }
+
        relocation = value + signed_addend;
 
        relocation -= (input_section->output_section->vma
@@ -2911,6 +2974,13 @@ elf32_arm_final_link_relocate (reloc_howto_type *           howto,
            off &= ~1;
          else
            {
+             /* If we are addressing a Thumb function, we need to
+                adjust the address by one, so that attempts to
+                call the function pointer will correctly
+                interpret it as Thumb code.  */
+             if (sym_flags == STT_ARM_TFUNC)
+               value |= 1;
+
              bfd_put_32 (output_bfd, value, sgot->contents + off);
 
              if (info->shared)
@@ -3876,6 +3946,9 @@ elf32_arm_gc_sweep_hook (bfd *                     abfd ATTRIBUTE_UNUSED,
        case R_ARM_JUMP24:
        case R_ARM_PREL31:
 #endif
+       case R_ARM_THM_PC22:
+         /* Should the interworking branches be here also?  */
+
          r_symndx = ELF32_R_SYM (rel->r_info);
          if (r_symndx >= symtab_hdr->sh_info)
            {
@@ -3884,15 +3957,18 @@ elf32_arm_gc_sweep_hook (bfd *                     abfd ATTRIBUTE_UNUSED,
              struct elf32_arm_relocs_copied *p;
 
              h = sym_hashes[r_symndx - symtab_hdr->sh_info];
+             eh = (struct elf32_arm_link_hash_entry *) h;
 
              if (h->plt.refcount > 0)
-               h->plt.refcount -= 1;
+               {
+                 h->plt.refcount -= 1;
+                 if (ELF32_R_TYPE (rel->r_info) == R_ARM_THM_PC22)
+                   eh->plt_thumb_refcount--;
+               }
 
              if (r_type == R_ARM_ABS32
                  || r_type == R_ARM_REL32)
                {
-                 eh = (struct elf32_arm_link_hash_entry *) h;
-
                  for (pp = &eh->relocs_copied; (p = *pp) != NULL;
                       pp = &p->next)
                  if (p->section == sec)
@@ -3951,6 +4027,7 @@ elf32_arm_check_relocs (bfd *abfd, struct bfd_link_info *info,
   for (rel = relocs; rel < rel_end; rel++)
     {
       struct elf_link_hash_entry *h;
+      struct elf32_arm_link_hash_entry *eh;
       unsigned long r_symndx;
       int r_type;
 
@@ -3964,6 +4041,8 @@ elf32_arm_check_relocs (bfd *abfd, struct bfd_link_info *info,
       else
         h = sym_hashes[r_symndx - symtab_hdr->sh_info];
 
+      eh = (struct elf32_arm_link_hash_entry *) h;
+
       switch (r_type)
         {
          case R_ARM_GOT32:
@@ -4018,6 +4097,8 @@ elf32_arm_check_relocs (bfd *abfd, struct bfd_link_info *info,
          case R_ARM_JUMP24:
          case R_ARM_PREL31:
 #endif
+         case R_ARM_THM_PC22:
+           /* Should the interworking branches be listed here?  */
            if (h != NULL)
              {
                /* If this reloc is in a read-only section, we might
@@ -4039,12 +4120,16 @@ elf32_arm_check_relocs (bfd *abfd, struct bfd_link_info *info,
                    || r_type == R_ARM_JUMP24
                    || r_type == R_ARM_PREL31
 #endif
-                   || r_type == R_ARM_PLT32)
+                   || r_type == R_ARM_PLT32
+                   || r_type == R_ARM_THM_PC22)
                  h->needs_plt = 1;
 
                /* If we create a PLT entry, this relocation will reference
                   it, even if it's an ABS32 relocation.  */
                h->plt.refcount += 1;
+
+               if (r_type == R_ARM_THM_PC22)
+                 eh->plt_thumb_refcount += 1;
              }
 
            /* If we are creating a shared library, and this is a reloc
@@ -4068,7 +4153,8 @@ elf32_arm_check_relocs (bfd *abfd, struct bfd_link_info *info,
                     && r_type != R_ARM_JUMP24
                     && r_type != R_ARM_PREL31
 #endif
-                    && r_type != R_ARM_REL32)
+                    && r_type != R_ARM_REL32
+                    && r_type != R_ARM_THM_PC22)
                    || (h != NULL
                        && (! info->symbolic
                            || !h->def_regular))))
@@ -4317,6 +4403,7 @@ elf32_arm_adjust_dynamic_symbol (struct bfd_link_info * info,
   bfd * dynobj;
   asection * s;
   unsigned int power_of_two;
+  struct elf32_arm_link_hash_entry * eh;
 
   dynobj = elf_hash_table (info)->dynobj;
 
@@ -4328,10 +4415,12 @@ elf32_arm_adjust_dynamic_symbol (struct bfd_link_info * info,
                      && h->ref_regular
                      && !h->def_regular)));
 
+  eh = (struct elf32_arm_link_hash_entry *) h;
+
   /* If this is a function, put it in the procedure linkage table.  We
      will fill in the contents of the procedure linkage table later,
      when we know the address of the .got section.  */
-  if (h->type == STT_FUNC
+  if (h->type == STT_FUNC || h->type == STT_ARM_TFUNC
       || h->needs_plt)
     {
       if (h->plt.refcount <= 0
@@ -4345,18 +4434,22 @@ elf32_arm_adjust_dynamic_symbol (struct bfd_link_info * info,
             such a case, we don't actually need to build a procedure
             linkage table, and we can just do a PC24 reloc instead.  */
          h->plt.offset = (bfd_vma) -1;
+         eh->plt_thumb_refcount = 0;
          h->needs_plt = 0;
        }
 
       return TRUE;
     }
   else
-    /* It's possible that we incorrectly decided a .plt reloc was
-       needed for an R_ARM_PC24 or similar reloc to a non-function sym
-       in check_relocs.  We can't decide accurately between function
-       and non-function syms in check-relocs; Objects loaded later in
-       the link may change h->type.  So fix it now.  */
-    h->plt.offset = (bfd_vma) -1;
+    {
+      /* It's possible that we incorrectly decided a .plt reloc was
+        needed for an R_ARM_PC24 or similar reloc to a non-function sym
+        in check_relocs.  We can't decide accurately between function
+        and non-function syms in check-relocs; Objects loaded later in
+        the link may change h->type.  So fix it now.  */
+      h->plt.offset = (bfd_vma) -1;
+      eh->plt_thumb_refcount = 0;
+    }
 
   /* If this is a weak symbol, and there is a real definition, the
      processor independent code will have arranged for us to see the
@@ -4441,6 +4534,8 @@ allocate_dynrelocs (struct elf_link_hash_entry *h, void * inf)
   struct elf32_arm_link_hash_entry *eh;
   struct elf32_arm_relocs_copied *p;
 
+  eh = (struct elf32_arm_link_hash_entry *) h;
+
   if (h->root.type == bfd_link_hash_indirect)
     return TRUE;
 
@@ -4477,6 +4572,14 @@ allocate_dynrelocs (struct elf_link_hash_entry *h, void * inf)
 
          h->plt.offset = s->size;
 
+         /* If we will insert a Thumb trampoline before this PLT, leave room
+            for it.  */
+         if (!htab->symbian_p && eh->plt_thumb_refcount > 0)
+           {
+             h->plt.offset += PLT_THUMB_STUB_SIZE;
+             s->size += PLT_THUMB_STUB_SIZE;
+           }
+
          /* If this symbol is not defined in a regular file, and we are
             not generating a shared library, then set the symbol to this
             location in the .plt.  This is required to make function
@@ -4487,15 +4590,24 @@ allocate_dynrelocs (struct elf_link_hash_entry *h, void * inf)
            {
              h->root.u.def.section = s;
              h->root.u.def.value = h->plt.offset;
+
+             /* Make sure the function is not marked as Thumb, in case
+                it is the target of an ABS32 relocation, which will
+                point to the PLT entry.  */
+             if (ELF_ST_TYPE (h->type) == STT_ARM_TFUNC)
+               h->type = ELF_ST_INFO (ELF_ST_BIND (h->type), STT_FUNC);
            }
 
          /* Make room for this entry.  */
          s->size += htab->plt_entry_size;
 
          if (!htab->symbian_p)
-           /* We also need to make an entry in the .got.plt section, which
-              will be placed in the .got section by the linker script.  */
-           htab->sgotplt->size += 4;
+           {
+             /* We also need to make an entry in the .got.plt section, which
+                will be placed in the .got section by the linker script.  */
+             eh->plt_got_offset = htab->sgotplt->size;
+             htab->sgotplt->size += 4;
+           }
 
          /* We also need to make an entry in the .rel.plt section.  */
          htab->srelplt->size += sizeof (Elf32_External_Rel);
@@ -4542,7 +4654,6 @@ allocate_dynrelocs (struct elf_link_hash_entry *h, void * inf)
   else
     h->got.offset = (bfd_vma) -1;
 
-  eh = (struct elf32_arm_link_hash_entry *) h;
   if (eh->relocs_copied == NULL)
     return TRUE;
 
@@ -4867,9 +4978,11 @@ elf32_arm_finish_dynamic_symbol (bfd * output_bfd, struct bfd_link_info * info,
 {
   bfd * dynobj;
   struct elf32_arm_link_hash_table *htab;
+  struct elf32_arm_link_hash_entry *eh;
 
   dynobj = elf_hash_table (info)->dynobj;
   htab = elf32_arm_hash_table (info);
+  eh = (struct elf32_arm_link_hash_entry *) h;
 
   if (h->plt.offset != (bfd_vma) -1)
     {
@@ -4888,13 +5001,6 @@ elf32_arm_finish_dynamic_symbol (bfd * output_bfd, struct bfd_link_info * info,
       srel = bfd_get_section_by_name (dynobj, ".rel.plt");
       BFD_ASSERT (splt != NULL && srel != NULL);
 
-      /* Get the index in the procedure linkage table which
-        corresponds to this symbol.  This is the index of this symbol
-        in all the symbols for which we are making plt entries.  The
-        first entry in the procedure linkage table is reserved.  */
-      plt_index = ((h->plt.offset - htab->plt_header_size) 
-                  / htab->plt_entry_size);
-
       /* Fill in the entry in the procedure linkage table.  */
       if (htab->symbian_p)
        {
@@ -4909,6 +5015,13 @@ elf32_arm_finish_dynamic_symbol (bfd * output_bfd, struct bfd_link_info * info,
                          + splt->output_offset
                          + h->plt.offset + 4 * (i - 1));
          rel.r_info = ELF32_R_INFO (h->dynindx, R_ARM_GLOB_DAT);
+
+         /* Get the index in the procedure linkage table which
+            corresponds to this symbol.  This is the index of this symbol
+            in all the symbols for which we are making plt entries.  The
+            first entry in the procedure linkage table is reserved.  */
+         plt_index = ((h->plt.offset - htab->plt_header_size) 
+                      / htab->plt_entry_size);
        }
       else
        {
@@ -4919,13 +5032,21 @@ elf32_arm_finish_dynamic_symbol (bfd * output_bfd, struct bfd_link_info * info,
          sgot = bfd_get_section_by_name (dynobj, ".got.plt");
          BFD_ASSERT (sgot != NULL);
 
-         /* Get the offset into the .got table of the entry that
-            corresponds to this function.  Each .got entry is 4 bytes.
-            The first three are reserved.  */
-         got_offset = (plt_index + 3) * 4;
+         /* Get the offset into the .got.plt table of the entry that
+            corresponds to this function.  */
+         got_offset = eh->plt_got_offset;
+
+         /* Get the index in the procedure linkage table which
+            corresponds to this symbol.  This is the index of this symbol
+            in all the symbols for which we are making plt entries.  The
+            first three entries in .got.plt are reserved; after that
+            symbols appear in the same order as in .plt.  */
+         plt_index = (got_offset - 12) / 4;
 
          /* Calculate the displacement between the PLT slot and the
-            entry in the GOT.  */
+            entry in the GOT.  The eight-byte offset accounts for the
+            value produced by adding to pc in the first instruction
+            of the PLT stub.  */
          got_displacement = (sgot->output_section->vma
                              + sgot->output_offset
                              + got_offset
@@ -4936,6 +5057,14 @@ elf32_arm_finish_dynamic_symbol (bfd * output_bfd, struct bfd_link_info * info,
 
          BFD_ASSERT ((got_displacement & 0xf0000000) == 0);
 
+         if (eh->plt_thumb_refcount > 0)
+           {
+             bfd_put_16 (output_bfd, elf32_arm_plt_thumb_stub[0],
+                         splt->contents + h->plt.offset - 4);
+             bfd_put_16 (output_bfd, elf32_arm_plt_thumb_stub[1],
+                         splt->contents + h->plt.offset - 2);
+           }
+
          bfd_put_32 (output_bfd, elf32_arm_plt_entry[0] | ((got_displacement & 0x0ff00000) >> 20),
                      splt->contents + h->plt.offset + 0);
          bfd_put_32 (output_bfd, elf32_arm_plt_entry[1] | ((got_displacement & 0x000ff000) >> 12),
@@ -5521,6 +5650,18 @@ elf32_arm_write_section (bfd *output_bfd ATTRIBUTE_UNUSED, asection *sec,
   return FALSE;
 }
 
+/* Display STT_ARM_TFUNC symbols as functions.  */
+
+static void
+elf32_arm_symbol_processing (bfd *abfd ATTRIBUTE_UNUSED,
+                            asymbol *asym)
+{
+  elf_symbol_type *elfsym = (elf_symbol_type *) asym;
+
+  if (ELF_ST_TYPE (elfsym->internal_elf_sym.st_info) == STT_ARM_TFUNC)
+    elfsym->symbol.flags |= BSF_FUNCTION;
+}
+
 #define ELF_ARCH                       bfd_arch_arm
 #define ELF_MACHINE_CODE               EM_ARM
 #ifdef __QNXTARGET__
@@ -5559,6 +5700,7 @@ elf32_arm_write_section (bfd *output_bfd ATTRIBUTE_UNUSED, asection *sec,
 #define elf_backend_section_from_shdr                  elf32_arm_section_from_shdr
 #define elf_backend_final_write_processing      elf32_arm_final_write_processing
 #define elf_backend_copy_indirect_symbol        elf32_arm_copy_indirect_symbol
+#define elf_backend_symbol_processing          elf32_arm_symbol_processing
 
 #define elf_backend_can_refcount    1
 #define elf_backend_can_gc_sections 1
@@ -5691,4 +5833,3 @@ elf32_arm_symbian_modify_segment_map (bfd *abfd,
 #define elf_backend_want_got_plt 0
 
 #include "elf32-target.h"
-
index 9de9798c603c5991e44e84ce94208956eff9db2e..beb0c48631d4700eabcebbb1e160f3b3ba9dccc1 100644 (file)
@@ -1,3 +1,8 @@
+2004-11-17  Daniel Jacobowitz  <dan@codesourcery.com>
+
+       * gas/arm/mapping.d: Expect F markers for Thumb code.
+       * gas/arm/unwind.d: Update big-endian pattern.
+
 2004-11-12  Nick Clifton  <nickc@redhat.com>
 
        * gas/mn10300/basic.exp: Add relax test.
index c6ddd090389867bf72d791899914db2ab8d3ed5c..7ad249e27df0f1cb39da89591c19fa485e5648b0 100644 (file)
@@ -10,9 +10,9 @@ SYMBOL TABLE:
 0+00 l    d  .data     0+0 
 0+00 l    d  .bss      0+0 
 0+00 l     F .text     0+0 \$a
-0+08 l       .text     0+0 \$t
+0+08 l     F .text     0+0 \$t
 0+00 l     O .data     0+0 \$d
 0+00 l    d  foo       0+0 
-0+00 l       foo       0+0 \$t
+0+00 l     F foo       0+0 \$t
 0+00 g       .text     0+0 mapping
-0+08 g       .text     0+0 thumb_mapping
+0+08 g     F .text     0+0 thumb_mapping
index 63f9fdbff27bea54768e0c9ff0fdea08656679db..7be60b4b59dacc040c39c6bf484bdb766cd640bb 100644 (file)
@@ -21,13 +21,13 @@ OFFSET   TYPE              VALUE
 
 
 Contents of section .text:
- 0000 (0000a0e3 0100a0e3 0200a0e3 0300a0e3|e3a00000 a0e30001 e3a00002 e3a00003)  .*
+ 0000 (0000a0e3 0100a0e3 0200a0e3 0300a0e3|e3a00000 e3a00001 e3a00002 e3a00003)  .*
  0010 (0420|2004)                                 .*
 Contents of section .ARM.extab:
  0000 (449b0181 b0b08086|81019b44 8680b0b0) 00000000 00000000  .*
- 0010 (8402b101 b0b0b005 2a000000 00c60181|01b10284 05b0b0b0 000000a2 8101c600)  .*
+ 0010 (8402b101 b0b0b005 2a000000 00c60181|01b10284 05b0b0b0 0000002a 8101c600)  .*
  0020 (b0b0c1c1|c1c1b0b0) 00000000                    .*
 Contents of section .ARM.exidx:
  0000 00000000 (b0b0a880 04000000|80a8b0b0 00000004) 00000000  .*
  0010 (08000000 0c000000 0c000000 1c000000|00000008 0000000c 0000000c 0000001c)  .*
- 0020 (10000000 08849780|00000010 80978480)                    .*
+ 0020 (10000000 08849780|00000010 80978408)                    .*
index 7318455c4fefb0f584662c2a4fe5c6826fe544f5..68292f5b0cfda16bca3bded8739cd157c58f74ea 100644 (file)
@@ -1,3 +1,8 @@
+2004-11-17  Daniel Jacobowitz  <dan@codesourcery.com>
+
+       * emultempl/armelf.em (arm_elf_set_bfd_for_interworking): Don't use
+       a dynamic object for stubs.
+
 2004-11-04  Paul Brook  <paul@codesourcery.com>
 
        * ld.texinfo: Document --default-imported-symver.
index 54af832456ee53df7327efdce1f5f09d9800c360..3e594cbdc74d97a9b76acfa774a437604b99ebee 100644 (file)
@@ -78,8 +78,11 @@ arm_elf_set_bfd_for_interworking (lang_statement_union_type *statement)
 
       ASSERT (output_section->owner == output_bfd);
 
+      /* Don't attach the interworking stubs to a dynamic object, to
+        an empty section, etc.  */
       if ((output_section->flags & SEC_HAS_CONTENTS) != 0
          && (i->flags & SEC_NEVER_LOAD) == 0
+         && ! (i->owner->flags & DYNAMIC)
          && ! i->owner->output_has_begun)
        {
          bfd_for_interwork = i->owner;
index 094fe602f66d06e5b378ef7d280de60beefdf72c..ee6081dd09f2e285766ac70f8c08aa43c39ecb1e 100644 (file)
@@ -1,3 +1,15 @@
+2004-11-17  Daniel Jacobowitz  <dan@codesourcery.com>
+
+       * ld-arm/mixed-app.d, ld-arm/mixed-app.r, ld-arm/mixed-app.s,
+       ld-arm/mixed-app.sym, ld-arm/mixed-lib.d, ld-arm/mixed-lib.r,
+       ld-arm/mixed-lib.s, ld-arm/mixed-lib.sym, ld-arm/arm-dyn.ld,
+       ld-arm/arm-lib.ld: New files.
+       * ld-arm/arm-app-abs32.d, ld-arm/arm-app-abs32.r, ld-arm/arm-app.d,
+       ld-arm/arm-app.r, ld-arm/arm-lib-plt32.d, ld-arm/arm-lib-plt32.r,
+       ld-arm/arm-lib.d, ld-arm/arm-lib.r, ld-arm/arm-static-app.d,
+       ld-arm/arm-static-app.r: Update for big-endian.
+       * ld-arm/arm-elf.exp: Run the new tests.
+
 2004-11-16  Richard Sandiford  <rsandifo@redhat.com>
 
        * ld-mips-elf/eh-frame1.{s,ld},
index 4ebff1e838af52befe398834107dfef6746ece7b..f69ed8f9ee54d6c8022a78b21ecea25bc0cc22a4 100644 (file)
@@ -1,5 +1,5 @@
 
-tmpdir/arm-app-abs32:     file format elf32-littlearm
+tmpdir/arm-app-abs32:     file format elf32-(little|big)arm
 architecture: arm, flags 0x00000112:
 EXEC_P, HAS_SYMS, D_PAGED
 start address .*
index b9e99661ea0c75e1b7de738bcd0b8ef0b5732fc3..08d668c297953bd5919aa45feb4f06b2d66c1a6a 100644 (file)
@@ -1,5 +1,5 @@
 
-tmpdir/arm-app-abs32:     file format elf32-littlearm
+tmpdir/arm-app-abs32:     file format elf32-(little|big)arm
 
 DYNAMIC RELOCATION RECORDS
 OFFSET   TYPE              VALUE 
index ae48f53a0a545e6eae4cc49d50528e22f103455b..80bed981c70e1cbe42253efc24d0a229b976acc5 100644 (file)
@@ -1,5 +1,5 @@
 
-tmpdir/arm-app:     file format elf32-littlearm
+tmpdir/arm-app:     file format elf32-(little|big)arm
 architecture: arm, flags 0x00000112:
 EXEC_P, HAS_SYMS, D_PAGED
 start address 0x.*
index f2433cee2eb2d4bd328b6b73d28b4b0e322d129d..a24939281b59105770e62e2130548cb63ef65b91 100644 (file)
@@ -1,5 +1,5 @@
 
-tmpdir/arm-app:     file format elf32-littlearm
+tmpdir/arm-app:     file format elf32-(little|big)arm
 
 DYNAMIC RELOCATION RECORDS
 OFFSET   TYPE              VALUE 
diff --git a/ld/testsuite/ld-arm/arm-dyn.ld b/ld/testsuite/ld-arm/arm-dyn.ld
new file mode 100644 (file)
index 0000000..20b9613
--- /dev/null
@@ -0,0 +1,199 @@
+/* Script for -z combreloc: combine and sort reloc sections */
+OUTPUT_ARCH(arm)
+ENTRY(_start)
+/* Do we need any of these for elf?
+   __DYNAMIC = 0;    */
+SECTIONS
+{
+  /* Read-only sections, merged into text segment: */
+  PROVIDE (__executable_start = 0x8000); . = 0x8000;
+  .interp         : { *(.interp) }
+  .hash           : { *(.hash) }
+  .dynsym         : { *(.dynsym) }
+  .dynstr         : { *(.dynstr) }
+  .gnu.version    : { *(.gnu.version) }
+  .gnu.version_d  : { *(.gnu.version_d) }
+  .gnu.version_r  : { *(.gnu.version_r) }
+  .rel.dyn        :
+    {
+      *(.rel.init)
+      *(.rel.text .rel.text.* .rel.gnu.linkonce.t.*)
+      *(.rel.fini)
+      *(.rel.rodata .rel.rodata.* .rel.gnu.linkonce.r.*)
+      *(.rel.data.rel.ro*)
+      *(.rel.data .rel.data.* .rel.gnu.linkonce.d.*)
+      *(.rel.tdata .rel.tdata.* .rel.gnu.linkonce.td.*)
+      *(.rel.tbss .rel.tbss.* .rel.gnu.linkonce.tb.*)
+      *(.rel.ctors)
+      *(.rel.dtors)
+      *(.rel.got)
+      *(.rel.bss .rel.bss.* .rel.gnu.linkonce.b.*)
+    }
+  .rela.dyn       :
+    {
+      *(.rela.init)
+      *(.rela.text .rela.text.* .rela.gnu.linkonce.t.*)
+      *(.rela.fini)
+      *(.rela.rodata .rela.rodata.* .rela.gnu.linkonce.r.*)
+      *(.rela.data .rela.data.* .rela.gnu.linkonce.d.*)
+      *(.rela.tdata .rela.tdata.* .rela.gnu.linkonce.td.*)
+      *(.rela.tbss .rela.tbss.* .rela.gnu.linkonce.tb.*)
+      *(.rela.ctors)
+      *(.rela.dtors)
+      *(.rela.got)
+      *(.rela.bss .rela.bss.* .rela.gnu.linkonce.b.*)
+    }
+  .rel.plt        : { *(.rel.plt) }
+  .rela.plt       : { *(.rela.plt) }
+  .init           :
+  {
+    KEEP (*(.init))
+  } =0
+  .plt            : { *(.plt) }
+  .text           :
+  {
+    *(.text .stub .text.* .gnu.linkonce.t.*)
+    KEEP (*(.text.*personality*))
+    /* .gnu.warning sections are handled specially by elf32.em.  */
+    *(.gnu.warning)
+    *(.glue_7t) *(.glue_7)
+  } =0
+  .fini           :
+  {
+    KEEP (*(.fini))
+  } =0
+  PROVIDE (__etext = .);
+  PROVIDE (_etext = .);
+  PROVIDE (etext = .);
+  .rodata         : { *(.rodata .rodata.* .gnu.linkonce.r.*) }
+  .rodata1        : { *(.rodata1) }
+  .ARM.extab   : { *(.ARM.extab* .gnu.linkonce.armextab.*) }
+   __exidx_start = .;
+  .ARM.exidx   : { *(.ARM.exidx* .gnu.linkonce.armexidx.*) }
+   __exidx_end = .;
+  .eh_frame_hdr : { *(.eh_frame_hdr) }
+  .eh_frame       : ONLY_IF_RO { KEEP (*(.eh_frame)) }
+  .gcc_except_table   : ONLY_IF_RO { KEEP (*(.gcc_except_table)) *(.gcc_except_table.*) }
+  /* Adjust the address for the data segment.  We want to adjust up to
+     the same address within the page on the next page up.  */
+  . = ALIGN(256) + (. & (256 - 1));
+  /* Exception handling  */
+  .eh_frame       : ONLY_IF_RW { KEEP (*(.eh_frame)) }
+  .gcc_except_table   : ONLY_IF_RW { KEEP (*(.gcc_except_table)) *(.gcc_except_table.*) }
+  /* Thread Local Storage sections  */
+  .tdata         : { *(.tdata .tdata.* .gnu.linkonce.td.*) }
+  .tbss                  : { *(.tbss .tbss.* .gnu.linkonce.tb.*) *(.tcommon) }
+  /* Ensure the __preinit_array_start label is properly aligned.  We
+     could instead move the label definition inside the section, but
+     the linker would then create the section even if it turns out to
+     be empty, which isn't pretty.  */
+  . = ALIGN(32 / 8);
+  PROVIDE (__preinit_array_start = .);
+  .preinit_array     : { KEEP (*(.preinit_array)) }
+  PROVIDE (__preinit_array_end = .);
+  PROVIDE (__init_array_start = .);
+  .init_array     : { KEEP (*(.init_array)) }
+  PROVIDE (__init_array_end = .);
+  PROVIDE (__fini_array_start = .);
+  .fini_array     : { KEEP (*(.fini_array)) }
+  PROVIDE (__fini_array_end = .);
+  .ctors          :
+  {
+    /* gcc uses crtbegin.o to find the start of
+       the constructors, so we make sure it is
+       first.  Because this is a wildcard, it
+       doesn't matter if the user does not
+       actually link against crtbegin.o; the
+       linker won't look for a file to match a
+       wildcard.  The wildcard also means that it
+       doesn't matter which directory crtbegin.o
+       is in.  */
+    KEEP (*crtbegin*.o(.ctors))
+    /* We don't want to include the .ctor section from
+       from the crtend.o file until after the sorted ctors.
+       The .ctor section from the crtend file contains the
+       end of ctors marker and it must be last */
+    KEEP (*(EXCLUDE_FILE (*crtend*.o ) .ctors))
+    KEEP (*(SORT(.ctors.*)))
+    KEEP (*(.ctors))
+  }
+  .dtors          :
+  {
+    KEEP (*crtbegin*.o(.dtors))
+    KEEP (*(EXCLUDE_FILE (*crtend*.o ) .dtors))
+    KEEP (*(SORT(.dtors.*)))
+    KEEP (*(.dtors))
+  }
+  .jcr            : { KEEP (*(.jcr)) }
+  .data.rel.ro : { *(.data.rel.ro.local) *(.data.rel.ro*) }
+  .dynamic        : { *(.dynamic) }
+  .got            : { *(.got.plt) *(.got) }
+  .data           :
+  {
+    __data_start = . ;
+    *(.data .data.* .gnu.linkonce.d.*)
+    KEEP (*(.gnu.linkonce.d.*personality*))
+    SORT(CONSTRUCTORS)
+  }
+  .data1          : { *(.data1) }
+  _edata = .;
+  PROVIDE (edata = .);
+  __bss_start = .;
+  __bss_start__ = .;
+  .bss            :
+  {
+   *(.dynbss)
+   *(.bss .bss.* .gnu.linkonce.b.*)
+   *(COMMON)
+   /* Align here to ensure that the .bss section occupies space up to
+      _end.  Align after .bss to ensure correct alignment even if the
+      .bss section disappears because there are no input sections.  */
+   . = ALIGN(32 / 8);
+  }
+  . = ALIGN(32 / 8);
+  _end = .;
+  _bss_end__ = . ; __bss_end__ = . ; __end__ = . ;
+  PROVIDE (end = .);
+  /* Stabs debugging sections.  */
+  .stab          0 : { *(.stab) }
+  .stabstr       0 : { *(.stabstr) }
+  .stab.excl     0 : { *(.stab.excl) }
+  .stab.exclstr  0 : { *(.stab.exclstr) }
+  .stab.index    0 : { *(.stab.index) }
+  .stab.indexstr 0 : { *(.stab.indexstr) }
+  .comment       0 : { *(.comment) }
+  /* DWARF debug sections.
+     Symbols in the DWARF debugging sections are relative to the beginning
+     of the section so we begin them at 0.  */
+  /* DWARF 1 */
+  .debug          0 : { *(.debug) }
+  .line           0 : { *(.line) }
+  /* GNU DWARF 1 extensions */
+  .debug_srcinfo  0 : { *(.debug_srcinfo) }
+  .debug_sfnames  0 : { *(.debug_sfnames) }
+  /* DWARF 1.1 and DWARF 2 */
+  .debug_aranges  0 : { *(.debug_aranges) }
+  .debug_pubnames 0 : { *(.debug_pubnames) }
+  /* DWARF 2 */
+  .debug_info     0 : { *(.debug_info .gnu.linkonce.wi.*) }
+  .debug_abbrev   0 : { *(.debug_abbrev) }
+  .debug_line     0 : { *(.debug_line) }
+  .debug_frame    0 : { *(.debug_frame) }
+  .debug_str      0 : { *(.debug_str) }
+  .debug_loc      0 : { *(.debug_loc) }
+  .debug_macinfo  0 : { *(.debug_macinfo) }
+  /* SGI/MIPS DWARF 2 extensions */
+  .debug_weaknames 0 : { *(.debug_weaknames) }
+  .debug_funcnames 0 : { *(.debug_funcnames) }
+  .debug_typenames 0 : { *(.debug_typenames) }
+  .debug_varnames  0 : { *(.debug_varnames) }
+    .stack         0x80000 :
+  {
+    _stack = .;
+    *(.stack)
+  }
+  .note.gnu.arm.ident 0 : { KEEP (*(.note.gnu.arm.ident)) }
+  /DISCARD/ : { *(.note.GNU-stack) }
+}
+
+
index 1de19e78916c39cb37b2a18f8008cd6eae1c3bc3..3e747cab83fb016f4691e77c292e8b54b267d074 100644 (file)
@@ -47,6 +47,16 @@ set armelftests {
     {"Non-pcrel function reference" "tmpdir/arm-lib.so" "" {arm-app-abs32.s}
      {{objdump -fdw arm-app-abs32.d} {objdump -Rw arm-app-abs32.r}}
      "arm-app-abs32"}
+    {"Mixed ARM/Thumb shared library" "-shared -T arm-lib.ld" ""
+     {mixed-lib.s}
+     {{objdump -fdw mixed-lib.d} {objdump -Rw mixed-lib.r}
+      {readelf -Ds mixed-lib.sym}}
+     "mixed-lib.so"}
+    {"Mixed ARM/Thumb dynamic application" "tmpdir/mixed-lib.so -T arm-dyn.ld" ""
+     {mixed-app.s}
+     {{objdump -fdw mixed-app.d} {objdump -Rw mixed-app.r}
+      {readelf -Ds mixed-app.sym}}
+     "mixed-app"}
     {"target1-abs" "-static --target1-abs -T arm.ld" "" {arm-target1.s}
      {{objdump -s arm-target1-abs.d}}
      "arm-target1-abs"}
index 449b0f9c0f0e53b9b7d0e3c8a72edd54f1f2c1f9..58206f43796cf38f365fe1401ba691dd1ea69471 100644 (file)
@@ -1,5 +1,5 @@
 
-tmpdir/arm-lib-plt32.so:     file format elf32-littlearm
+tmpdir/arm-lib-plt32.so:     file format elf32-(little|big)arm
 architecture: arm, flags 0x00000150:
 HAS_SYMS, DYNAMIC, D_PAGED
 start address 0x.*
index 2212493b1733be6f0f78c99746b83cc4a4803f0f..35155397103d7b4796dfdf70cfb8514fc7aa5b10 100644 (file)
@@ -1,5 +1,5 @@
 
-tmpdir/arm-lib-plt32.so:     file format elf32-littlearm
+tmpdir/arm-lib-plt32.so:     file format elf32-(little|big)arm
 
 DYNAMIC RELOCATION RECORDS
 OFFSET   TYPE              VALUE 
index b7be230a41204840065a2f296f470fd7ce2d0fc2..e3257c955149610b6283952c147156c553195700 100644 (file)
@@ -1,5 +1,5 @@
 
-tmpdir/arm-lib.so:     file format elf32-littlearm
+tmpdir/arm-lib.so:     file format elf32-(little|big)arm
 architecture: arm, flags 0x00000150:
 HAS_SYMS, DYNAMIC, D_PAGED
 start address 0x.*
diff --git a/ld/testsuite/ld-arm/arm-lib.ld b/ld/testsuite/ld-arm/arm-lib.ld
new file mode 100644 (file)
index 0000000..075eaa4
--- /dev/null
@@ -0,0 +1,192 @@
+/* Script for --shared -z combreloc: shared library, combine & sort relocs */
+OUTPUT_ARCH(arm)
+ENTRY(_start)
+/* Do we need any of these for elf?
+   __DYNAMIC = 0;    */
+SECTIONS
+{
+  /* Read-only sections, merged into text segment: */
+  . = 0 + SIZEOF_HEADERS;
+  .hash           : { *(.hash) }
+  .dynsym         : { *(.dynsym) }
+  .dynstr         : { *(.dynstr) }
+  .gnu.version    : { *(.gnu.version) }
+  .gnu.version_d  : { *(.gnu.version_d) }
+  .gnu.version_r  : { *(.gnu.version_r) }
+  .rel.dyn        :
+    {
+      *(.rel.init)
+      *(.rel.text .rel.text.* .rel.gnu.linkonce.t.*)
+      *(.rel.fini)
+      *(.rel.rodata .rel.rodata.* .rel.gnu.linkonce.r.*)
+      *(.rel.data.rel.ro*)
+      *(.rel.data .rel.data.* .rel.gnu.linkonce.d.*)
+      *(.rel.tdata .rel.tdata.* .rel.gnu.linkonce.td.*)
+      *(.rel.tbss .rel.tbss.* .rel.gnu.linkonce.tb.*)
+      *(.rel.ctors)
+      *(.rel.dtors)
+      *(.rel.got)
+      *(.rel.bss .rel.bss.* .rel.gnu.linkonce.b.*)
+    }
+  .rela.dyn       :
+    {
+      *(.rela.init)
+      *(.rela.text .rela.text.* .rela.gnu.linkonce.t.*)
+      *(.rela.fini)
+      *(.rela.rodata .rela.rodata.* .rela.gnu.linkonce.r.*)
+      *(.rela.data .rela.data.* .rela.gnu.linkonce.d.*)
+      *(.rela.tdata .rela.tdata.* .rela.gnu.linkonce.td.*)
+      *(.rela.tbss .rela.tbss.* .rela.gnu.linkonce.tb.*)
+      *(.rela.ctors)
+      *(.rela.dtors)
+      *(.rela.got)
+      *(.rela.bss .rela.bss.* .rela.gnu.linkonce.b.*)
+    }
+  .rel.plt        : { *(.rel.plt) }
+  .rela.plt       : { *(.rela.plt) }
+  .init           :
+  {
+    KEEP (*(.init))
+  } =0
+  .plt            : { *(.plt) }
+  .text           :
+  {
+    *(.text .stub .text.* .gnu.linkonce.t.*)
+    KEEP (*(.text.*personality*))
+    /* .gnu.warning sections are handled specially by elf32.em.  */
+    *(.gnu.warning)
+    *(.glue_7t) *(.glue_7)
+  } =0
+  .fini           :
+  {
+    KEEP (*(.fini))
+  } =0
+  PROVIDE (__etext = .);
+  PROVIDE (_etext = .);
+  PROVIDE (etext = .);
+  .rodata         : { *(.rodata .rodata.* .gnu.linkonce.r.*) }
+  .rodata1        : { *(.rodata1) }
+  .ARM.extab   : { *(.ARM.extab* .gnu.linkonce.armextab.*) }
+   __exidx_start = .;
+  .ARM.exidx   : { *(.ARM.exidx* .gnu.linkonce.armexidx.*) }
+   __exidx_end = .;
+  .eh_frame_hdr : { *(.eh_frame_hdr) }
+  .eh_frame       : ONLY_IF_RO { KEEP (*(.eh_frame)) }
+  .gcc_except_table   : ONLY_IF_RO { KEEP (*(.gcc_except_table)) *(.gcc_except_table.*) }
+  /* Adjust the address for the data segment.  We want to adjust up to
+     the same address within the page on the next page up.  */
+  . = ALIGN(256) + (. & (256 - 1));
+  /* Exception handling  */
+  .eh_frame       : ONLY_IF_RW { KEEP (*(.eh_frame)) }
+  .gcc_except_table   : ONLY_IF_RW { KEEP (*(.gcc_except_table)) *(.gcc_except_table.*) }
+  /* Thread Local Storage sections  */
+  .tdata         : { *(.tdata .tdata.* .gnu.linkonce.td.*) }
+  .tbss                  : { *(.tbss .tbss.* .gnu.linkonce.tb.*) *(.tcommon) }
+  /* Ensure the __preinit_array_start label is properly aligned.  We
+     could instead move the label definition inside the section, but
+     the linker would then create the section even if it turns out to
+     be empty, which isn't pretty.  */
+  . = ALIGN(32 / 8);
+  .preinit_array     : { KEEP (*(.preinit_array)) }
+  .init_array     : { KEEP (*(.init_array)) }
+  .fini_array     : { KEEP (*(.fini_array)) }
+  .ctors          :
+  {
+    /* gcc uses crtbegin.o to find the start of
+       the constructors, so we make sure it is
+       first.  Because this is a wildcard, it
+       doesn't matter if the user does not
+       actually link against crtbegin.o; the
+       linker won't look for a file to match a
+       wildcard.  The wildcard also means that it
+       doesn't matter which directory crtbegin.o
+       is in.  */
+    KEEP (*crtbegin*.o(.ctors))
+    /* We don't want to include the .ctor section from
+       from the crtend.o file until after the sorted ctors.
+       The .ctor section from the crtend file contains the
+       end of ctors marker and it must be last */
+    KEEP (*(EXCLUDE_FILE (*crtend*.o ) .ctors))
+    KEEP (*(SORT(.ctors.*)))
+    KEEP (*(.ctors))
+  }
+  .dtors          :
+  {
+    KEEP (*crtbegin*.o(.dtors))
+    KEEP (*(EXCLUDE_FILE (*crtend*.o ) .dtors))
+    KEEP (*(SORT(.dtors.*)))
+    KEEP (*(.dtors))
+  }
+  .jcr            : { KEEP (*(.jcr)) }
+  .data.rel.ro : { *(.data.rel.ro.local) *(.data.rel.ro*) }
+  .dynamic        : { *(.dynamic) }
+  .got            : { *(.got.plt) *(.got) }
+  .data           :
+  {
+    __data_start = . ;
+    *(.data .data.* .gnu.linkonce.d.*)
+    KEEP (*(.gnu.linkonce.d.*personality*))
+    SORT(CONSTRUCTORS)
+  }
+  .data1          : { *(.data1) }
+  _edata = .;
+  PROVIDE (edata = .);
+  __bss_start = .;
+  __bss_start__ = .;
+  .bss            :
+  {
+   *(.dynbss)
+   *(.bss .bss.* .gnu.linkonce.b.*)
+   *(COMMON)
+   /* Align here to ensure that the .bss section occupies space up to
+      _end.  Align after .bss to ensure correct alignment even if the
+      .bss section disappears because there are no input sections.  */
+   . = ALIGN(32 / 8);
+  }
+  . = ALIGN(32 / 8);
+  _end = .;
+  _bss_end__ = . ; __bss_end__ = . ; __end__ = . ;
+  PROVIDE (end = .);
+  /* Stabs debugging sections.  */
+  .stab          0 : { *(.stab) }
+  .stabstr       0 : { *(.stabstr) }
+  .stab.excl     0 : { *(.stab.excl) }
+  .stab.exclstr  0 : { *(.stab.exclstr) }
+  .stab.index    0 : { *(.stab.index) }
+  .stab.indexstr 0 : { *(.stab.indexstr) }
+  .comment       0 : { *(.comment) }
+  /* DWARF debug sections.
+     Symbols in the DWARF debugging sections are relative to the beginning
+     of the section so we begin them at 0.  */
+  /* DWARF 1 */
+  .debug          0 : { *(.debug) }
+  .line           0 : { *(.line) }
+  /* GNU DWARF 1 extensions */
+  .debug_srcinfo  0 : { *(.debug_srcinfo) }
+  .debug_sfnames  0 : { *(.debug_sfnames) }
+  /* DWARF 1.1 and DWARF 2 */
+  .debug_aranges  0 : { *(.debug_aranges) }
+  .debug_pubnames 0 : { *(.debug_pubnames) }
+  /* DWARF 2 */
+  .debug_info     0 : { *(.debug_info .gnu.linkonce.wi.*) }
+  .debug_abbrev   0 : { *(.debug_abbrev) }
+  .debug_line     0 : { *(.debug_line) }
+  .debug_frame    0 : { *(.debug_frame) }
+  .debug_str      0 : { *(.debug_str) }
+  .debug_loc      0 : { *(.debug_loc) }
+  .debug_macinfo  0 : { *(.debug_macinfo) }
+  /* SGI/MIPS DWARF 2 extensions */
+  .debug_weaknames 0 : { *(.debug_weaknames) }
+  .debug_funcnames 0 : { *(.debug_funcnames) }
+  .debug_typenames 0 : { *(.debug_typenames) }
+  .debug_varnames  0 : { *(.debug_varnames) }
+    .stack         0x80000 :
+  {
+    _stack = .;
+    *(.stack)
+  }
+  .note.gnu.arm.ident 0 : { KEEP (*(.note.gnu.arm.ident)) }
+  /DISCARD/ : { *(.note.GNU-stack) }
+}
+
+
index cd1a5b76d5bab992f9ba784e84d6a92e6d9ff902..a7dde4745727138026dd0e202518d2f7474fd410 100644 (file)
@@ -1,5 +1,5 @@
 
-tmpdir/arm-lib.so:     file format elf32-littlearm
+tmpdir/arm-lib.so:     file format elf32-(little|big)arm
 
 DYNAMIC RELOCATION RECORDS
 OFFSET   TYPE              VALUE 
index 2d39b55c2e839d76d32109804e786bf4ae6e0de2..9a3309d6136b235d1d987a5800cbaf95ce40fe14 100644 (file)
@@ -1,5 +1,5 @@
 
-tmpdir/arm-static-app:     file format elf32-littlearm
+tmpdir/arm-static-app:     file format elf32-(little|big)arm
 architecture: arm, flags 0x00000112:
 EXEC_P, HAS_SYMS, D_PAGED
 start address 0x.*
index 7c350dcb97e9633741d3b6c786fda45ee833054c..6034b7f8d85173530c7ab23954f955353cae6496 100644 (file)
@@ -1,3 +1,3 @@
 
-tmpdir/arm-static-app:     file format elf32-littlearm
+tmpdir/arm-static-app:     file format elf32-(little|big)arm
 
diff --git a/ld/testsuite/ld-arm/mixed-app.d b/ld/testsuite/ld-arm/mixed-app.d
new file mode 100644 (file)
index 0000000..0774f77
--- /dev/null
@@ -0,0 +1,57 @@
+
+tmpdir/mixed-app:     file format elf32-(little|big)arm
+architecture: arm, flags 0x00000112:
+EXEC_P, HAS_SYMS, D_PAGED
+start address 0x.*
+
+Disassembly of section .plt:
+
+.* <.plt>:
+ .*:   e52de004        str     lr, \[sp, #-4\]!
+ .*:   e59fe004        ldr     lr, \[pc, #4\]  ; .* <.plt\+0x10>
+ .*:   e08fe00e        add     lr, pc, lr
+ .*:   e5bef008        ldr     pc, \[lr, #8\]!
+ .*:   .*
+ .*:   (46c04778       undefined|477846c0      ldrmib  r4, \[r8, -r0, asr #13\]!)
+ .*:   e28fc6.*        add     ip, pc, #.*     ; 0x.*
+ .*:   e28cca.*        add     ip, ip, #.*     ; 0x.*
+ .*:   e5bcf.*         ldr     pc, \[ip, #.*\]!
+ .*:   e28fc6.*        add     ip, pc, #.*     ; 0x.*
+ .*:   e28cca.*        add     ip, ip, #.*     ; 0x.*
+ .*:   e5bcf.*         ldr     pc, \[ip, #.*\]!
+Disassembly of section .text:
+
+.* <_start>:
+ .*:   e1a0c00d        mov     ip, sp
+ .*:   e92dd800        stmdb   sp!, {fp, ip, lr, pc}
+ .*:   eb000004        bl      .* <app_func>
+ .*:   e89d6800        ldmia   sp, {fp, sp, lr}
+ .*:   e12fff1e        bx      lr
+ .*:   e1a00000        nop                     \(mov r0,r0\)
+ .*:   e1a00000        nop                     \(mov r0,r0\)
+ .*:   e1a00000        nop                     \(mov r0,r0\)
+
+.* <app_func>:
+ .*:   e1a0c00d        mov     ip, sp
+ .*:   e92dd800        stmdb   sp!, {fp, ip, lr, pc}
+ .*:   ebffffe.        bl      .* <.text-0x..>
+ .*:   e89d6800        ldmia   sp, {fp, sp, lr}
+ .*:   e12fff1e        bx      lr
+ .*:   e1a00000        nop                     \(mov r0,r0\)
+ .*:   e1a00000        nop                     \(mov r0,r0\)
+ .*:   e1a00000        nop                     \(mov r0,r0\)
+
+.* <app_func2>:
+ .*:   e12fff1e        bx      lr
+ .*:   e1a00000        nop                     \(mov r0,r0\)
+ .*:   e1a00000        nop                     \(mov r0,r0\)
+ .*:   e1a00000        nop                     \(mov r0,r0\)
+
+.* <app_tfunc>:
+ .*:   b500            push    {lr}
+ .*:   (ffc.f7ff|f7ffffc.)     bl      .* <.text-0x..>
+ .*:   bd00            pop     {pc}
+ .*:   4770            bx      lr
+ .*:   46c0            nop                     \(mov r8, r8\)
+ .*:   46c0            nop                     \(mov r8, r8\)
+ .*:   46c0            nop                     \(mov r8, r8\)
diff --git a/ld/testsuite/ld-arm/mixed-app.r b/ld/testsuite/ld-arm/mixed-app.r
new file mode 100644 (file)
index 0000000..601c5f9
--- /dev/null
@@ -0,0 +1,10 @@
+
+tmpdir/mixed-app:     file format elf32-(little|big)arm
+
+DYNAMIC RELOCATION RECORDS
+OFFSET   TYPE              VALUE 
+.* R_ARM_COPY        data_obj
+.* R_ARM_JUMP_SLOT   lib_func2
+.* R_ARM_JUMP_SLOT   lib_func1
+
+
diff --git a/ld/testsuite/ld-arm/mixed-app.s b/ld/testsuite/ld-arm/mixed-app.s
new file mode 100644 (file)
index 0000000..ce82487
--- /dev/null
@@ -0,0 +1,39 @@
+       .text
+       .p2align 4
+       .globl _start
+_start:
+       mov     ip, sp
+       stmdb   sp!, {r11, ip, lr, pc}
+       bl      app_func
+       ldmia   sp, {r11, sp, lr}
+       bx lr
+
+       .p2align 4
+       .globl app_func
+       .type app_func,%function
+app_func:
+       mov     ip, sp
+       stmdb   sp!, {r11, ip, lr, pc}
+       bl      lib_func1
+       ldmia   sp, {r11, sp, lr}
+       bx lr
+
+       .p2align 4
+       .globl app_func2
+       .type app_func2,%function
+app_func2:
+       bx      lr
+
+       .p2align 4
+       .globl app_tfunc
+       .type app_tfunc,%function
+       .thumb_func
+       .code 16
+app_tfunc:
+       push    {lr}
+       bl      lib_func2
+       pop     {pc}
+       bx      lr
+
+       .data
+       .long data_obj
diff --git a/ld/testsuite/ld-arm/mixed-app.sym b/ld/testsuite/ld-arm/mixed-app.sym
new file mode 100644 (file)
index 0000000..0a26e4c
--- /dev/null
@@ -0,0 +1,19 @@
+
+Symbol table for image:
+  Num Buc:    Value  Size   Type   Bind Vis      Ndx Name
+   ..  ..: ........     0  NOTYPE GLOBAL DEFAULT ABS _edata
+   ..  ..: 0*[^0]*.*   20    FUNC GLOBAL DEFAULT UND lib_func1
+   ..  ..: 0*[^0]*.*    2    FUNC GLOBAL DEFAULT UND lib_func2
+   ..  ..: ........     0  NOTYPE GLOBAL DEFAULT ABS _bss_end__
+   ..  ..: ........     0  OBJECT GLOBAL DEFAULT ABS _DYNAMIC
+   ..  ..: ........     0  NOTYPE GLOBAL DEFAULT ABS __bss_end__
+   ..  ..: ........     0  NOTYPE GLOBAL DEFAULT  13 _stack
+   ..  ..: ........     4  OBJECT GLOBAL DEFAULT  12 data_obj
+   ..  ..: ........     0  NOTYPE GLOBAL DEFAULT ABS __bss_start__
+   ..  ..: ........     0  NOTYPE GLOBAL DEFAULT ABS __bss_start
+   ..  ..: ........     0  NOTYPE GLOBAL DEFAULT ABS _end
+   ..  ..: .......0     0    FUNC GLOBAL DEFAULT   8 app_func2
+   ..  ..: ........     0  NOTYPE GLOBAL DEFAULT ABS __exidx_end
+   ..  ..: ........     0  NOTYPE GLOBAL DEFAULT  11 __data_start
+   ..  ..: ........     0  NOTYPE GLOBAL DEFAULT ABS __end__
+   ..  ..: ........     0  NOTYPE GLOBAL DEFAULT ABS __exidx_start
diff --git a/ld/testsuite/ld-arm/mixed-lib.d b/ld/testsuite/ld-arm/mixed-lib.d
new file mode 100644 (file)
index 0000000..b261c67
--- /dev/null
@@ -0,0 +1,38 @@
+
+tmpdir/mixed-lib.so:     file format elf32-(little|big)arm
+architecture: arm, flags 0x00000150:
+HAS_SYMS, DYNAMIC, D_PAGED
+start address 0x.*
+
+Disassembly of section .plt:
+
+.* <.plt>:
+ .*:   e52de004        str     lr, \[sp, #-4\]!
+ .*:   e59fe004        ldr     lr, \[pc, #4\]  ; .* <lib_func1-0x1.>
+ .*:   e08fe00e        add     lr, pc, lr
+ .*:   e5bef008        ldr     pc, \[lr, #8\]!
+ .*:   .*
+ .*:   e28fc6.*        add     ip, pc, #.*     ; 0x.*
+ .*:   e28cca.*        add     ip, ip, #.*     ; 0x.*
+ .*:   e5bcf.*         ldr     pc, \[ip, #.*\]!
+Disassembly of section .text:
+
+.* <lib_func1>:
+ .*:   e1a0c00d        mov     ip, sp
+ .*:   e92dd800        stmdb   sp!, {fp, ip, lr, pc}
+ .*:   ebfffff.        bl      .* <lib_func1-0x..>
+ .*:   e89d6800        ldmia   sp, {fp, sp, lr}
+ .*:   e12fff1e        bx      lr
+ .*:   e1a00000        nop                     \(mov r0,r0\)
+ .*:   e1a00000        nop                     \(mov r0,r0\)
+ .*:   e1a00000        nop                     \(mov r0,r0\)
+
+.* <lib_func2>:
+ .*:   4770            bx      lr
+ .*:   46c0            nop                     \(mov r8, r8\)
+ .*:   46c0            nop                     \(mov r8, r8\)
+ .*:   46c0            nop                     \(mov r8, r8\)
+ .*:   46c0            nop                     \(mov r8, r8\)
+ .*:   46c0            nop                     \(mov r8, r8\)
+ .*:   46c0            nop                     \(mov r8, r8\)
+ .*:   46c0            nop                     \(mov r8, r8\)
diff --git a/ld/testsuite/ld-arm/mixed-lib.r b/ld/testsuite/ld-arm/mixed-lib.r
new file mode 100644 (file)
index 0000000..0137880
--- /dev/null
@@ -0,0 +1,8 @@
+
+tmpdir/mixed-lib.so:     file format elf32-(little|big)arm
+
+DYNAMIC RELOCATION RECORDS
+OFFSET   TYPE              VALUE 
+.* R_ARM_JUMP_SLOT   app_func2
+
+
diff --git a/ld/testsuite/ld-arm/mixed-lib.s b/ld/testsuite/ld-arm/mixed-lib.s
new file mode 100644 (file)
index 0000000..86f5ace
--- /dev/null
@@ -0,0 +1,28 @@
+       .text
+
+       .p2align 4
+       .globl lib_func1
+       .type lib_func1, %function
+lib_func1:
+       mov     ip, sp
+       stmdb   sp!, {r11, ip, lr, pc}
+       bl      app_func2
+       ldmia   sp, {r11, sp, lr}
+       bx lr
+       .size lib_func1, . - lib_func1
+
+       .p2align 4
+       .globl lib_func2
+       .type lib_func2, %function
+       .thumb_func
+       .code 16
+lib_func2:
+       bx lr
+       .size lib_func2, . - lib_func2
+
+       .data
+       .globl data_obj
+       .type data_obj, %object
+data_obj:
+       .long 0
+       .size data_obj, . - data_obj
diff --git a/ld/testsuite/ld-arm/mixed-lib.sym b/ld/testsuite/ld-arm/mixed-lib.sym
new file mode 100644 (file)
index 0000000..1174751
--- /dev/null
@@ -0,0 +1,19 @@
+
+Symbol table for image:
+  Num Buc:    Value  Size   Type   Bind Vis      Ndx Name
+   ..  ..: ........     0  NOTYPE GLOBAL DEFAULT ABS _edata
+   ..  ..: .......0    20    FUNC GLOBAL DEFAULT   6 lib_func1
+   ..  ..: .......0     2  THUMB_FUNC GLOBAL DEFAULT   6 lib_func2
+   ..  ..: ........     0  NOTYPE GLOBAL DEFAULT ABS _bss_end__
+   ..  ..: ........     0  OBJECT GLOBAL DEFAULT ABS _DYNAMIC
+   ..  ..: ........     0  NOTYPE GLOBAL DEFAULT ABS __bss_end__
+   ..  ..: ........     0  NOTYPE GLOBAL DEFAULT  11 _stack
+   ..  ..: ........     4  OBJECT GLOBAL DEFAULT   9 data_obj
+   ..  ..: ........     0  NOTYPE GLOBAL DEFAULT ABS __bss_start__
+   ..  ..: ........     0  NOTYPE GLOBAL DEFAULT ABS __bss_start
+   ..  ..: ........     0  NOTYPE GLOBAL DEFAULT ABS _end
+   ..  ..: 00000000     0  NOTYPE GLOBAL DEFAULT UND app_func2
+   ..  ..: ........     0  NOTYPE GLOBAL DEFAULT ABS __exidx_end
+   ..  ..: ........     0  NOTYPE GLOBAL DEFAULT   9 __data_start
+   ..  ..: ........     0  NOTYPE GLOBAL DEFAULT ABS __end__
+   ..  ..: ........     0  NOTYPE GLOBAL DEFAULT ABS __exidx_start
index a1419eeff965793314b5b19fca518f2fdc5eff3b..9a642ad16cd759b445323de109f82bafb7fec5ce 100644 (file)
@@ -1,3 +1,8 @@
+2004-11-17  Daniel Jacobowitz  <dan@codesourcery.com>
+
+       * arm-dis.c (WORD_ADDRESS): Define.
+       (print_insn): Use it.  Correct big-endian end-of-section handling.
+
 2004-11-08  Inderpreet Singh   <inderpreetb@nioda.hcltech.com>
            Vineet Sharma      <vineets@noida.hcltech.com>
 
index 44cdeac344925a4d369426498b2b1ffe4306244d..b598739e148299ed0d02a39a0e18f1b4cb693826 100644 (file)
@@ -46,6 +46,8 @@
 #define NUM_ELEM(a)     (sizeof (a) / sizeof (a)[0])
 #endif
 
+#define WORD_ADDRESS(pc) ((pc) & ~0x3)
+
 static char * arm_conditional[] =
 {"eq", "ne", "cs", "cc", "mi", "pl", "vs", "vc",
  "hi", "ls", "ge", "lt", "gt", "le", "", "nv"};
@@ -1253,7 +1255,7 @@ print_insn (pc, info, little)
   unsigned char      b[4];
   long               given;
   int                status;
-  int                is_thumb;
+  int                is_thumb, second_half_valid = 1;
 
   if (info->disassembler_options)
     {
@@ -1299,6 +1301,7 @@ print_insn (pc, info, little)
       if (status != 0 && is_thumb)
        {
          info->bytes_per_chunk = 2;
+         second_half_valid = 0;
 
          status = info->read_memory_func (pc, (bfd_byte *) b, 2, info);
          b[3] = b[2] = 0;
@@ -1315,10 +1318,10 @@ print_insn (pc, info, little)
   else
     {
       status = info->read_memory_func
-       (pc & ~ 0x3, (bfd_byte *) &b[0], 4, info);
+       (WORD_ADDRESS (pc), (bfd_byte *) &b[0], 4, info);
       if (status != 0)
        {
-         info->memory_error_func (status, pc, info);
+         info->memory_error_func (status, WORD_ADDRESS (pc), info);
          return -1;
        }
 
@@ -1329,14 +1332,11 @@ print_insn (pc, info, little)
              given = (b[2] << 8) | b[3];
 
              status = info->read_memory_func
-               ((pc + 4) & ~ 0x3, (bfd_byte *) b, 4, info);
+               (WORD_ADDRESS (pc + 4), (bfd_byte *) b, 4, info);
              if (status != 0)
-               {
-                 info->memory_error_func (status, pc + 4, info);
-                 return -1;
-               }
-
-             given |= (b[0] << 24) | (b[1] << 16);
+               second_half_valid = 0;
+             else
+               given |= (b[0] << 24) | (b[1] << 16);
            }
          else
            given = (b[0] << 8) | b[1] | (b[2] << 24) | (b[3] << 16);
@@ -1358,6 +1358,12 @@ print_insn (pc, info, little)
   else
     status = print_insn_arm (pc, info, given);
 
+  if (is_thumb && status == 4 && second_half_valid == 0)
+    {
+      info->memory_error_func (status, WORD_ADDRESS (pc + 4), info);
+      return -1;
+    }
+
   return status;
 }
 
This page took 0.051618 seconds and 4 git commands to generate.