Arm specific code changed to conform to BFD coding conventions.
[deliverable/binutils-gdb.git] / bfd / coff-arm.c
index 9b7ce2356956b7200bd0d945186c06140f77e0ea..a64e0be06cc7f65377f4e5a8091770f7dc061594 100644 (file)
@@ -1,5 +1,6 @@
 /* BFD back-end for ARM COFF files.
-   Copyright 1990, 91, 92, 93, 94, 95, 96, 1997 Free Software Foundation, Inc.
+   Copyright 1990, 91, 92, 93, 94, 95, 96, 97, 1998
+   Free Software Foundation, Inc.
    Written by Cygnus Support.
 
 This file is part of BFD, the Binary File Descriptor library.
@@ -32,6 +33,8 @@ Foundation, Inc., 59 Temple Place - Suite 330, Boston, MA 02111-1307, USA.  */
 
 #include "libcoff.h"
 
+/* Macros for manipulation the bits in the flags field of the coff data
+   structure.  */
 #define APCS_26_FLAG(       abfd )     (coff_data (abfd)->flags & F_APCS_26)
 #define APCS_FLOAT_FLAG(    abfd )     (coff_data (abfd)->flags & F_APCS_FLOAT)
 #define PIC_FLAG(           abfd )     (coff_data (abfd)->flags & F_PIC)
@@ -222,7 +225,7 @@ static reloc_howto_type aoutarm_std_reloc_howto[] =
   HOWTO(ARM_26,
        2,
        2,
-       26,
+       24,
        true,
        0,
        complain_overflow_signed,
@@ -274,7 +277,7 @@ static reloc_howto_type aoutarm_std_reloc_howto[] =
   HOWTO( ARM_26D,  
        2, 
        2,
-       26,
+       24,
        false,
        0,
        complain_overflow_signed,
@@ -328,7 +331,7 @@ static reloc_howto_type aoutarm_std_reloc_howto[] =
   HOWTO( ARM_THUMB9,
        1,
        1,
-       9,
+       8,
        true,
        0,
        complain_overflow_signed,
@@ -341,7 +344,7 @@ static reloc_howto_type aoutarm_std_reloc_howto[] =
   HOWTO( ARM_THUMB12,
        1,
        1,
-       12,
+       11,
        true,
        0,
        complain_overflow_signed,
@@ -354,7 +357,7 @@ static reloc_howto_type aoutarm_std_reloc_howto[] =
   HOWTO( ARM_THUMB23,
        1,
        2,
-       23,
+       22,
        true,
        0,
        complain_overflow_signed,
@@ -365,15 +368,17 @@ static reloc_howto_type aoutarm_std_reloc_howto[] =
        0x07ff07ff, 
        PCRELOFFSET),
 };
+
 #ifdef COFF_WITH_PE
 /* Return true if this relocation should
    appear in the output .reloc section. */
 
-static boolean in_reloc_p (abfd, howto)
+static boolean
+in_reloc_p (abfd, howto)
      bfd * abfd;
-     reloc_howto_type *howto;
+     reloc_howto_type * howto;
 {
-  return !howto->pc_relative && howto->type != 11;
+  return !howto->pc_relative && howto->type != ARM_RVA32;
 }     
 #endif
 
@@ -416,7 +421,7 @@ coff_arm_rtype_to_howto (abfd, sec, rel, h, sym, addendp)
        HOWTO (ARM_26,
               2,
               2,
-              26,
+              24,
               true,
               0,
               complain_overflow_signed,
@@ -496,7 +501,7 @@ aoutarm_fix_pcrel_26 (abfd, reloc_entry, symbol, data, input_section,
   /* Check for overflow */
   if (relocation & 0x02000000)
     {
-      if ((relocation & ~0x03ffffff) != ~0x03ffffff)
+      if ((relocation & ~ (bfd_vma) 0x03ffffff) != ~ (bfd_vma) 0x03ffffff)
        flag = bfd_reloc_overflow;
     }
   else if (relocation & ~0x03ffffff)
@@ -527,7 +532,7 @@ coff_thumb_pcrel_common (abfd, reloc_entry, symbol, data, input_section,
      char **error_message;
      thumb_pcrel_branchtype btype;
 {
-  bfd_vma relocation;
+  bfd_vma relocation = 0;
   bfd_size_type addr = reloc_entry->address;
   long target = bfd_get_32 (abfd, (bfd_byte *) data + addr);
   bfd_reloc_status_type flag = bfd_reloc_ok;
@@ -586,6 +591,9 @@ coff_thumb_pcrel_common (abfd, reloc_entry, symbol, data, input_section,
       else
        relocation = ((target & 0x7ff) << 12) | ((target & 0x07ff0000) >> 15);
       break;
+
+    default:
+      abort ();
     }
 
   relocation = (relocation ^ signbit) - signbit; /* Sign extend */
@@ -623,6 +631,9 @@ coff_thumb_pcrel_common (abfd, reloc_entry, symbol, data, input_section,
      else
        target |= ((relocation & 0xffe) << 15) | ((relocation >> 12) & 0x7ff);
      break;
+
+   default:
+     abort ();
    }
 
   bfd_put_32 (abfd, target, (bfd_byte *) data + addr);
@@ -682,8 +693,8 @@ coff_thumb_pcrel_9 (abfd, reloc_entry, symbol, data, input_section,
 
 
 static CONST struct reloc_howto_struct *
-arm_reloc_type_lookup(abfd,code)
-      bfd *abfd;
+coff_arm_reloc_type_lookup (abfd, code)
+      bfd * abfd;
       bfd_reloc_code_real_type code;
 {
 #define ASTD(i,j)       case i: return &aoutarm_std_reloc_howto[j]
@@ -713,9 +724,6 @@ arm_reloc_type_lookup(abfd,code)
     }
 }
 
-
-#define coff_bfd_reloc_type_lookup arm_reloc_type_lookup
-
 #define COFF_DEFAULT_SECTION_ALIGNMENT_POWER (2)
 #define COFF_PAGE_SIZE 0x1000
 /* Turn a howto into a reloc  nunmber */
@@ -724,12 +732,54 @@ arm_reloc_type_lookup(abfd,code)
 #define BADMAG(x) ARMBADMAG(x)
 #define ARM 1                  /* Customize coffcode.h */
 
-/* The set of global variables that mark the total size of each kind
-   of glue required. */
-static long int global_thumb_glue_size = 0;
-static long int global_arm_glue_size = 0;
+/* Extend the coff_link_hash_table structure with a few ARM specific fields.
+   This allows us to store global data here without actually creating any
+   global variables, which is a no-no in the BFD world.  */
+struct coff_arm_link_hash_table
+{
+  /* The original coff_link_hash_table structure.  MUST be first field.  */
+  struct coff_link_hash_table  root;
+  
+  /* The size in bytes of the section containg the Thumb-to-ARM glue.  */
+  long int                     thumb_glue_size;
+  
+  /* The size in bytes of the section containg the ARM-to-Thumb glue.  */
+  long int                     arm_glue_size;
+
+  /* An arbitary input BFD chosen to hold the glue sections.  */
+  bfd *                                bfd_of_glue_owner;
+};
+
+/* Get the ARM coff linker hash table from a link_info structure.  */
+#define coff_arm_hash_table(info) \
+  ((struct coff_arm_link_hash_table *) ((info)->hash))
+
+/* Create an ARM coff linker hash table.  */
+
+static struct bfd_link_hash_table *
+coff_arm_link_hash_table_create (abfd)
+     bfd * abfd;
+{
+  struct coff_arm_link_hash_table * ret;
+
+  ret = ((struct coff_arm_link_hash_table *)
+        bfd_alloc (abfd, sizeof (struct coff_arm_link_hash_table)));
+  if (ret == (struct coff_arm_link_hash_table *) NULL)
+    return NULL;
+
+  if (! _bfd_coff_link_hash_table_init
+      (& ret->root, abfd, _bfd_coff_link_hash_newfunc))
+    {
+      bfd_release (abfd, ret);
+      return (struct bfd_link_hash_table *) NULL;
+    }
 
-static bfd * bfd_of_glue_owner = 0;
+  ret->thumb_glue_size   = 0;
+  ret->arm_glue_size     = 0;
+  ret->bfd_of_glue_owner = NULL;
+
+  return & ret->root.root;
+}
 
 /* some typedefs for holding instructions */
 typedef unsigned long int insn32;
@@ -780,7 +830,7 @@ insert_thumb_branch (br_insn, rel_off)
 
   rel_off >>= 1;                              /* half word aligned address */
   low_bits = rel_off & 0x000007FF;            /* the bottom 11 bits */
-  high_bits = ( rel_off >> 11 ) & 0x000007FF; /* the top 11 bits */
+  high_bits = (rel_off >> 11) & 0x000007FF;   /* the top 11 bits */
 
   if ((br_insn & LOW_HI_ORDER) == LOW_HI_ORDER)
     br_insn = LOW_HI_ORDER | (low_bits << 16) | high_bits;
@@ -801,9 +851,8 @@ find_thumb_glue (info, name, input_bfd)
      char *                 name;
      bfd *                  input_bfd;
 {
-  char *tmp_name = 0;
-
-  struct coff_link_hash_entry *myh = 0;
+  char *                        tmp_name;
+  struct coff_link_hash_entry * myh;
 
   tmp_name = ((char *)
         bfd_malloc (strlen (name) + strlen (THUMB2ARM_GLUE_ENTRY_NAME)));
@@ -812,14 +861,12 @@ find_thumb_glue (info, name, input_bfd)
 
   sprintf (tmp_name, THUMB2ARM_GLUE_ENTRY_NAME, name);
   
-  myh = coff_link_hash_lookup (coff_hash_table (info),
-                              tmp_name,
-                              false, false, true);
+  myh = coff_link_hash_lookup
+    (coff_hash_table (info), tmp_name, false, false, true);
+  
   if (myh == NULL)
-    {
-      _bfd_error_handler ("%s: unable to find THUMB glue '%s' for `%s'",
-                         bfd_get_filename (input_bfd), tmp_name, name);
-    }
+    _bfd_error_handler ("%s: unable to find THUMB glue '%s' for `%s'",
+                       bfd_get_filename (input_bfd), tmp_name, name);
   
   free (tmp_name);
 
@@ -832,8 +879,8 @@ find_arm_glue (info, name, input_bfd)
      char *                 name;
      bfd *                  input_bfd;
 {
-  char *tmp_name = 0;
-  struct coff_link_hash_entry *myh = 0;
+  char *                        tmp_name;
+  struct coff_link_hash_entry * myh;
 
   tmp_name = ((char *)
              bfd_malloc (strlen (name) + strlen (ARM2THUMB_GLUE_ENTRY_NAME)));
@@ -842,15 +889,12 @@ find_arm_glue (info, name, input_bfd)
 
   sprintf (tmp_name, ARM2THUMB_GLUE_ENTRY_NAME, name);
   
-  myh = coff_link_hash_lookup (coff_hash_table (info),
-                              tmp_name,
-                              false, false, true);
+  myh = coff_link_hash_lookup
+    (coff_hash_table (info), tmp_name, false, false, true);
 
   if (myh == NULL)
-    {
-      _bfd_error_handler ("%s: unable to find ARM glue '%s' for `%s'",
-                         bfd_get_filename (input_bfd), tmp_name, name);
-    }
+    _bfd_error_handler ("%s: unable to find ARM glue '%s' for `%s'",
+                       bfd_get_filename (input_bfd), tmp_name, name);
   
   free (tmp_name);
 
@@ -874,9 +918,9 @@ find_arm_glue (info, name, input_bfd)
 */
 
 #define ARM2THUMB_GLUE_SIZE 12
-static insn32 
-        a2t1_ldr_insn = 0xe59fc000,
-       a2t2_bx_r12_insn = 0xe12fff1c,
+static const insn32 
+        a2t1_ldr_insn       = 0xe59fc000,
+       a2t2_bx_r12_insn    = 0xe12fff1c,
        a2t3_func_addr_insn = 0x00000001;
 
 /*
@@ -899,10 +943,10 @@ static insn32
 */
 
 #define THUMB2ARM_GLUE_SIZE 8
-static insn16
+static const insn16
         t2a1_bx_pc_insn    = 0x4778,
         t2a2_noop_insn     = 0x46c0;
-static insn32
+static const insn32
         t2a3_b_insn        = 0xea000000;
 
 /* TODO:
@@ -958,8 +1002,8 @@ coff_arm_relocate_section (output_bfd, info, input_bfd, input_section,
      struct internal_syment *syms;
      asection **sections;
 {
-  struct internal_reloc *rel;
-  struct internal_reloc *relend;
+  struct internal_reloc * rel;
+  struct internal_reloc * relend;
 
   rel = relocs;
   relend = rel + input_section->reloc_count;
@@ -1057,7 +1101,6 @@ coff_arm_relocate_section (output_bfd, info, input_bfd, input_section,
              && (   h->root.type == bfd_link_hash_defined
                  || h->root.type == bfd_link_hash_defweak))
             {
-             asection *   sec;
              asection *   h_sec = h->root.u.def.section;
              const char * name  = h->root.root.string;
              
@@ -1072,50 +1115,65 @@ coff_arm_relocate_section (output_bfd, info, input_bfd, input_section,
                      || h->class == C_THUMBEXTFUNC)
                    {
                      /* Arm code calling a Thumb function */
-                     signed long int                final_disp;
-                     unsigned long int              tmp;
-                     long int                       my_offset;
-                     long int                       offset;
-                     asection *                     s = 0;
-                     unsigned long int              return_address;
-                     long int                       ret_offset;
-                     long int                       disp;
-                     struct coff_link_hash_entry *  myh; 
-
+                     unsigned long int                 tmp;
+                     long int                          my_offset;
+                     asection *                        s;
+                     long int                          ret_offset;
+                     struct coff_link_hash_entry *     myh; 
+                     struct coff_arm_link_hash_table * globals;
+                     
                      myh = find_arm_glue (info, name, input_bfd);
                      if (myh == NULL)
                        return false;
 
-                     my_offset = myh->root.u.def.value;
+                     globals = coff_arm_hash_table (info);
 
-                     s = bfd_get_section_by_name (bfd_of_glue_owner, 
+                     BFD_ASSERT (globals != NULL);
+                     BFD_ASSERT (globals->bfd_of_glue_owner != NULL);
+                       
+                     my_offset = myh->root.u.def.value;
+                     
+                     s = bfd_get_section_by_name (globals->bfd_of_glue_owner, 
                                                  ARM2THUMB_GLUE_SECTION_NAME);
                      BFD_ASSERT (s != NULL);
                      BFD_ASSERT (s->contents != NULL);
+                     BFD_ASSERT (s->output_section != NULL);
 
                      if ((my_offset & 0x01) == 0x01)
                        {
-                         if (INTERWORK_SET (h_sec->owner) && ! INTERWORK_FLAG (h_sec->owner))
-                           _bfd_error_handler ("%s: warning: interworking not enabled.",
-                                               bfd_get_filename (h_sec->owner));
-                         
+                         if (h_sec->owner != NULL
+                             && INTERWORK_SET (h_sec->owner)
+                             && ! INTERWORK_FLAG (h_sec->owner))
+                           {
+                             _bfd_error_handler
+                               ("%s(%s): warning: interworking not enabled.",
+                                bfd_get_filename (h_sec->owner), name);
+                             _bfd_error_handler
+                               ("  first occurrence: %s: arm call to thumb",
+                                bfd_get_filename (input_bfd));
+                           }
+
                          --my_offset;
                          myh->root.u.def.value = my_offset;
 
-                         bfd_put_32 (output_bfd, a2t1_ldr_insn, s->contents + my_offset);
+                         bfd_put_32 (output_bfd, a2t1_ldr_insn,
+                                     s->contents + my_offset);
                          
-                         bfd_put_32 (output_bfd, a2t2_bx_r12_insn, s->contents + my_offset + 4);
+                         bfd_put_32 (output_bfd, a2t2_bx_r12_insn,
+                                     s->contents + my_offset + 4);
                          
                          /* It's a thumb address.  Add the low order bit.  */
-                         bfd_put_32 (output_bfd, h_val | a2t3_func_addr_insn, s->contents + my_offset + 8);
+                         bfd_put_32 (output_bfd, h_val | a2t3_func_addr_insn,
+                                     s->contents + my_offset + 8);
                        }
 
-                     BFD_ASSERT (my_offset <= global_arm_glue_size);
+                     BFD_ASSERT (my_offset <= globals->arm_glue_size);
 
-                     tmp = bfd_get_32 (input_bfd, contents + rel->r_vaddr - input_section->vma);
+                     tmp = bfd_get_32 (input_bfd, contents + rel->r_vaddr
+                                       - input_section->vma);
                      
                      tmp = tmp & 0xFF000000;
-                     
+
                      /* Somehow these are both 4 too far, so subtract 8. */
                      ret_offset =
                        s->output_offset
@@ -1128,7 +1186,8 @@ coff_arm_relocate_section (output_bfd, info, input_bfd, input_section,
 
                      tmp = tmp | ((ret_offset >> 2) & 0x00FFFFFF);
                      
-                     bfd_put_32 (output_bfd, tmp, contents + rel->r_vaddr - input_section->vma);
+                     bfd_put_32 (output_bfd, tmp, contents + rel->r_vaddr
+                                 - input_section->vma);
                      
                      done = 1;
                    }
@@ -1142,31 +1201,44 @@ coff_arm_relocate_section (output_bfd, info, input_bfd, input_section,
                      || h->class == C_LABEL)
                    {
                      /* Thumb code calling an ARM function */
-                     unsigned long int              return_address;
-                     signed long int                final_disp;
-                     asection *                     s = 0;
-                     long int                       my_offset;
-                     unsigned long int              tmp;
-                     long int                       ret_offset;
-                     struct coff_link_hash_entry *  myh;
+                     asection *                         s = 0;
+                     long int                           my_offset;
+                     unsigned long int                  tmp;
+                     long int                           ret_offset;
+                     struct coff_link_hash_entry *      myh;
+                     struct coff_arm_link_hash_table *  globals;
 
                      myh = find_thumb_glue (info, name, input_bfd);
                      if (myh == NULL)
                        return false;
 
+                     globals = coff_arm_hash_table (info);
+                     
+                     BFD_ASSERT (globals != NULL);
+                     BFD_ASSERT (globals->bfd_of_glue_owner != NULL);
+                     
                      my_offset = myh->root.u.def.value;
                      
-                     s = bfd_get_section_by_name (bfd_of_glue_owner, 
+                     s = bfd_get_section_by_name (globals->bfd_of_glue_owner, 
                                                   THUMB2ARM_GLUE_SECTION_NAME);
                      
                      BFD_ASSERT (s != NULL);
                      BFD_ASSERT (s->contents != NULL);
+                     BFD_ASSERT (s->output_section != NULL);
                      
                      if ((my_offset & 0x01) == 0x01)
                        {
-                         if (INTERWORK_SET (h_sec->owner) && ! INTERWORK_FLAG (h_sec->owner))
-                           _bfd_error_handler ("%s: warning: interworking not enabled.",
-                                               bfd_get_filename (h_sec->owner));
+                         if (h_sec->owner != NULL
+                             && INTERWORK_SET (h_sec->owner)
+                             && ! INTERWORK_FLAG (h_sec->owner))
+                           {
+                             _bfd_error_handler
+                               ("%s(%s): warning: interworking not enabled.",
+                                bfd_get_filename (h_sec->owner), name);
+                             _bfd_error_handler
+                               ("  first occurrence: %s: thumb call to arm",
+                                bfd_get_filename (input_bfd));
+                           }
                          
                          -- my_offset;
                          myh->root.u.def.value = my_offset;
@@ -1178,20 +1250,22 @@ coff_arm_relocate_section (output_bfd, info, input_bfd, input_section,
                                     s->contents + my_offset + 2);
                      
                          ret_offset =
-                           ((signed)h_val) - 
-                           ((signed)(s->output_offset
-                                     + my_offset 
-                                     + s->output_section->vma))
-                           - 12;
-                     
-                         t2a3_b_insn |= ((ret_offset >> 2) & 0x00FFFFFF);
-                         
-                         bfd_put_32 (output_bfd, t2a3_b_insn, s->contents + my_offset + 4);
+                           ((signed)h_val) -                   /* Address of destination of the stub */
+                           ((signed)(s->output_offset          /* Offset from the start of the current section to the start of the stubs.  */
+                                     + my_offset               /* Offset of the start of this stub from the start of the stubs.  */
+                                     + s->output_section->vma) /* Address of the start of the current section.  */
+                                     + 4                       /* The branch instruction is 4 bytes into the stub.  */
+                                     + 8);                     /* ARM branches work from the pc of the instruction + 8.  */
+
+                         bfd_put_32 (output_bfd,
+                                     t2a3_b_insn | ((ret_offset >> 2) & 0x00FFFFFF),
+                                     s->contents + my_offset + 4);
                        }
 
-                     BFD_ASSERT (my_offset <= global_thumb_glue_size);
+                     BFD_ASSERT (my_offset <= globals->thumb_glue_size);
 
-                     /* Now go back and fix up the original bl insn to point here.  */
+                     /* Now go back and fix up the original BL insn to point
+                        to here.  */
                      ret_offset =
                        s->output_offset
                        + my_offset
@@ -1199,11 +1273,13 @@ coff_arm_relocate_section (output_bfd, info, input_bfd, input_section,
                           + rel->r_vaddr)
                        -4;
                      
-                     tmp = bfd_get_32 (input_bfd, contents + rel->r_vaddr - input_section->vma);
-                     
+                     tmp = bfd_get_32 (input_bfd, contents + rel->r_vaddr
+                                       - input_section->vma);
+
                      bfd_put_32 (output_bfd,
                                  insert_thumb_branch (tmp, ret_offset),
-                                 contents + rel->r_vaddr - input_section->vma);
+                                 contents + rel->r_vaddr
+                                 - input_section->vma);
                      
                      done = 1;
                     }
@@ -1259,7 +1335,7 @@ coff_arm_relocate_section (output_bfd, info, input_bfd, input_section,
   
 #if 1 /* THUMBEXTENSION */
       if (done)
-       ;
+       rstat = bfd_reloc_ok;
       /* Only perform this fix during the final link, not a relocatable link.  nickc@cygnus.com  */
       else if (! info->relocateable
               && howto->type == ARM_THUMB23)
@@ -1275,7 +1351,7 @@ coff_arm_relocate_section (output_bfd, info, input_bfd, input_section,
           bfd_vma address = rel->r_vaddr - input_section->vma;
          
           if (address > input_section->_raw_size)
-            rstat = bfd_reloc_outofrange;
+           rstat = bfd_reloc_outofrange;
           else
             {
               bfd_vma         relocation       = val + addend;
@@ -1408,7 +1484,7 @@ coff_arm_relocate_section (output_bfd, info, input_bfd, input_section,
                  || sym->n_sclass == C_THUMBEXTFUNC)
                patchit = true;
            }
-                       
+
          if (patchit)
            {
              bfd_byte * location = contents + rel->r_vaddr - input_section->vma;
@@ -1459,41 +1535,57 @@ coff_arm_relocate_section (output_bfd, info, input_bfd, input_section,
   return true;
 }
 
-static boolean
-arm_allocate_interworking_sections (info) 
-     struct bfd_link_info *info;
+#ifndef COFF_WITH_PE
+boolean
+bfd_arm_allocate_interworking_sections (info) 
+     struct bfd_link_info * info;
 {
-  asection *s;
-  bfd_byte *foo;
-  static char test_char = '1';
+  asection *                        s;
+  bfd_byte *                        foo;
+  struct coff_arm_link_hash_table * globals;
+#if 0
+  static char                       test_char = '1';
+#endif
+
+  globals = coff_arm_hash_table (info);
+  
+  BFD_ASSERT (globals != NULL);
 
-  if ( global_arm_glue_size != 0 )
+  if (globals->arm_glue_size != 0)
     {
-      BFD_ASSERT (bfd_of_glue_owner != NULL);
+      BFD_ASSERT (globals->bfd_of_glue_owner != NULL);
       
-      s = bfd_get_section_by_name (bfd_of_glue_owner, ARM2THUMB_GLUE_SECTION_NAME);
+      s = bfd_get_section_by_name
+       (globals->bfd_of_glue_owner, ARM2THUMB_GLUE_SECTION_NAME);
 
       BFD_ASSERT (s != NULL);
       
-      foo = (bfd_byte *) bfd_alloc(bfd_of_glue_owner, global_arm_glue_size);
-      memset(foo, test_char, global_arm_glue_size);
+      foo = (bfd_byte *) bfd_alloc
+       (globals->bfd_of_glue_owner, globals->arm_glue_size);
+#if 0
+      memset (foo, test_char, globals->arm_glue_size);
+#endif
       
-      s->_raw_size = s->_cooked_size = global_arm_glue_size;
+      s->_raw_size = s->_cooked_size = globals->arm_glue_size;
       s->contents = foo;
     }
 
-  if (global_thumb_glue_size != 0)
+  if (globals->thumb_glue_size != 0)
     {
-      BFD_ASSERT (bfd_of_glue_owner != NULL);
+      BFD_ASSERT (globals->bfd_of_glue_owner != NULL);
       
-      s = bfd_get_section_by_name (bfd_of_glue_owner, THUMB2ARM_GLUE_SECTION_NAME);
+      s = bfd_get_section_by_name
+       (globals->bfd_of_glue_owner, THUMB2ARM_GLUE_SECTION_NAME);
 
       BFD_ASSERT (s != NULL);
       
-      foo = (bfd_byte *) bfd_alloc(bfd_of_glue_owner, global_thumb_glue_size);
-      memset(foo, test_char, global_thumb_glue_size);
+      foo = (bfd_byte *) bfd_alloc
+       (globals->bfd_of_glue_owner, globals->thumb_glue_size);
+#if 0
+      memset (foo, test_char, globals->thumb_glue_size);
+#endif
       
-      s->_raw_size = s->_cooked_size = global_thumb_glue_size;
+      s->_raw_size = s->_cooked_size = globals->thumb_glue_size;
       s->contents = foo;
     }
 
@@ -1505,16 +1597,19 @@ record_arm_to_thumb_glue (info, h)
      struct bfd_link_info *        info;
      struct coff_link_hash_entry * h;
 {
-  const char *                   name = h->root.root.string;
-  register asection *            s;
-  char *                         tmp_name;
-  struct coff_link_hash_entry *  myh;
+  const char *                      name = h->root.root.string;
+  register asection *               s;
+  char *                            tmp_name;
+  struct coff_link_hash_entry *     myh;
+  struct coff_arm_link_hash_table * globals;
 
-  
-  BFD_ASSERT (bfd_of_glue_owner != NULL);
+  globals = coff_arm_hash_table (info);
 
-  s = bfd_get_section_by_name (bfd_of_glue_owner, 
-                              ARM2THUMB_GLUE_SECTION_NAME);
+  BFD_ASSERT (globals != NULL);
+  BFD_ASSERT (globals->bfd_of_glue_owner != NULL);
+
+  s = bfd_get_section_by_name
+    (globals->bfd_of_glue_owner, ARM2THUMB_GLUE_SECTION_NAME);
 
   BFD_ASSERT (s != NULL);
 
@@ -1525,9 +1620,8 @@ record_arm_to_thumb_glue (info, h)
 
   sprintf (tmp_name, ARM2THUMB_GLUE_ENTRY_NAME, name);
   
-  myh = coff_link_hash_lookup (coff_hash_table (info),
-                              tmp_name,
-                              false, false, true);
+  myh = coff_link_hash_lookup
+    (coff_hash_table (info), tmp_name, false, false, true);
   
   if (myh != NULL)
     {
@@ -1535,18 +1629,17 @@ record_arm_to_thumb_glue (info, h)
       return; /* we've already seen this guy */
     }
 
-
-  /* The only trick here is using global_arm_glue_size as the value. Even
+  /* The only trick here is using globals->arm_glue_size as the value. Even
      though the section isn't allocated yet, this is where we will be putting
      it.  */
 
-  bfd_coff_link_add_one_symbol (info, bfd_of_glue_owner, tmp_name,
+  bfd_coff_link_add_one_symbol (info, globals->bfd_of_glue_owner, tmp_name,
                                BSF_EXPORT | BSF_GLOBAL, 
-                               s, global_arm_glue_size + 1,
+                               s, globals->arm_glue_size + 1,
                                NULL, true, false, 
                                (struct bfd_link_hash_entry **) & myh);
   
-  global_arm_glue_size += ARM2THUMB_GLUE_SIZE;
+  globals->arm_glue_size += ARM2THUMB_GLUE_SIZE;
 
   free (tmp_name);
 
@@ -1558,15 +1651,20 @@ record_thumb_to_arm_glue (info, h)
      struct bfd_link_info *        info;
      struct coff_link_hash_entry * h;
 {
-  const char *                   name = h->root.root.string;
-  register asection *            s;
-  char *                         tmp_name;
-  struct coff_link_hash_entry *  myh;
+  const char *                       name = h->root.root.string;
+  register asection *                s;
+  char *                             tmp_name;
+  struct coff_link_hash_entry *      myh;
+  struct coff_arm_link_hash_table *  globals;
 
   
-  BFD_ASSERT (bfd_of_glue_owner != NULL);
+  globals = coff_arm_hash_table (info);
+  
+  BFD_ASSERT (globals != NULL);
+  BFD_ASSERT (globals->bfd_of_glue_owner != NULL);
 
-  s = bfd_get_section_by_name (bfd_of_glue_owner, THUMB2ARM_GLUE_SECTION_NAME);
+  s = bfd_get_section_by_name
+    (globals->bfd_of_glue_owner, THUMB2ARM_GLUE_SECTION_NAME);
 
   BFD_ASSERT (s != NULL);
 
@@ -1576,9 +1674,8 @@ record_thumb_to_arm_glue (info, h)
 
   sprintf (tmp_name, THUMB2ARM_GLUE_ENTRY_NAME, name);
 
-  myh = coff_link_hash_lookup (coff_hash_table (info),
-                              tmp_name,
-                              false, false, true);
+  myh = coff_link_hash_lookup
+    (coff_hash_table (info), tmp_name, false, false, true);
   
   if (myh != NULL)
     {
@@ -1586,12 +1683,12 @@ record_thumb_to_arm_glue (info, h)
       return; /* we've already seen this guy */
     }
 
-  bfd_coff_link_add_one_symbol (info, bfd_of_glue_owner, tmp_name,
-                               BSF_LOCAL, s, global_thumb_glue_size + 1,
+  bfd_coff_link_add_one_symbol (info, globals->bfd_of_glue_owner, tmp_name,
+                               BSF_LOCAL, s, globals->thumb_glue_size + 1,
                                NULL, true, false, 
                                (struct bfd_link_hash_entry **) & myh);
   
-  /* if we mark it 'thumb', the disassembler will do a better job */
+  /* If we mark it 'thumb', the disassembler will do a better job.  */
   myh->class = C_THUMBEXTFUNC;
 
   free (tmp_name);
@@ -1607,25 +1704,84 @@ record_thumb_to_arm_glue (info, h)
 
   myh = NULL;
 
-  /* now allocate another symbol to switch back to arm mode */
-  bfd_coff_link_add_one_symbol (info, bfd_of_glue_owner, tmp_name,
-                               BSF_LOCAL, s, global_thumb_glue_size + 4,
+  /* Now allocate another symbol to switch back to arm mode.  */
+  bfd_coff_link_add_one_symbol (info, globals->bfd_of_glue_owner, tmp_name,
+                               BSF_LOCAL, s, globals->thumb_glue_size + 4,
                                NULL, true, false, 
                                (struct bfd_link_hash_entry **) & myh);
 
   free (tmp_name);  
 
-  global_thumb_glue_size += THUMB2ARM_GLUE_SIZE;
+  globals->thumb_glue_size += THUMB2ARM_GLUE_SIZE;
 
   return;
 }
 
-static boolean
-arm_process_before_allocation (abfd, info)
+/* Select a BFD to be used to hold the sections used by the glue code.
+   This function is called from the linker scripts in ld/emultempl/
+   {armcoff/pe}.em  */
+boolean
+bfd_arm_get_bfd_for_interworking (abfd, info)
+     bfd *                 abfd;
+     struct bfd_link_info * info;
+{
+  struct coff_arm_link_hash_table * globals;
+  flagword                         flags;
+  asection *                       sec;
+    
+  /* If we are only performing a partial link do not bother
+     getting a bfd to hold the glue.  */
+  if (info->relocateable)
+    return true;
+  
+  globals = coff_arm_hash_table (info);
+  
+  BFD_ASSERT (globals != NULL);
+
+  if (globals->bfd_of_glue_owner != NULL)
+    return true;
+  
+  sec = bfd_get_section_by_name (abfd, ARM2THUMB_GLUE_SECTION_NAME);
+  
+  if (sec == NULL) 
+    {
+      flags = SEC_ALLOC | SEC_LOAD | SEC_HAS_CONTENTS | SEC_IN_MEMORY;
+      
+      sec = bfd_make_section (abfd, ARM2THUMB_GLUE_SECTION_NAME);
+      
+      if (sec == NULL
+         || ! bfd_set_section_flags (abfd, sec, flags)
+         || ! bfd_set_section_alignment (abfd, sec, 2))
+       return false;
+    }
+
+  sec = bfd_get_section_by_name (abfd, THUMB2ARM_GLUE_SECTION_NAME);
+
+  if (sec == NULL) 
+    {
+      flags = SEC_ALLOC | SEC_LOAD | SEC_HAS_CONTENTS | SEC_IN_MEMORY;
+      
+      sec = bfd_make_section (abfd, THUMB2ARM_GLUE_SECTION_NAME);
+      
+      if (sec == NULL
+         || ! bfd_set_section_flags (abfd, sec, flags)
+         || ! bfd_set_section_alignment (abfd, sec, 2))
+       return false;
+    }
+  
+  /* Save the bfd for later use.  */
+  globals->bfd_of_glue_owner = abfd;
+  
+  return true;
+}
+
+boolean
+bfd_arm_process_before_allocation (abfd, info)
      bfd *                   abfd;
      struct bfd_link_info *  info;
 {
   asection * sec;
+  struct coff_arm_link_hash_table * globals;
 
   /* If we are only performing a partial link do not bother
      to construct any glue.  */
@@ -1637,6 +1793,11 @@ arm_process_before_allocation (abfd, info)
 
   _bfd_coff_get_external_symbols (abfd);
 
+  globals = coff_arm_hash_table (info);
+  
+  BFD_ASSERT (globals != NULL);
+  BFD_ASSERT (globals->bfd_of_glue_owner != NULL);
+  
   /* Rummage around all the relocs and map the glue vectors.  */
   sec = abfd->sections;
 
@@ -1647,7 +1808,7 @@ arm_process_before_allocation (abfd, info)
     {
       struct internal_reloc * i;
       struct internal_reloc * rel;
-      
+
       if (sec->reloc_count == 0) 
        continue;
 
@@ -1666,14 +1827,14 @@ arm_process_before_allocation (abfd, info)
 
          symndx = rel->r_symndx;
 
-         /* If the relocation is not against a symbol it cannot concern us.  */
+         /* If the relocation is not against a symbol it cannot concern us. */
          if (symndx == -1)
            continue;
 
          h = obj_coff_sym_hashes (abfd)[symndx];
 
          /* If the relocation is against a static symbol it must be within
-            the current section and so canot be a cross ARM/Thumb relocation.  */
+            the current section and so cannot be a cross ARM/Thumb relocation.  */
          if (h == NULL)
            continue;
 
@@ -1685,17 +1846,18 @@ arm_process_before_allocation (abfd, info)
                 insert glue.  */
              
              if (h->class == C_THUMBEXTFUNC)
-               {
-                 record_arm_to_thumb_glue (info, h);
-               }
+               record_arm_to_thumb_glue (info, h);
              break;
              
            case ARM_THUMB23:
              /* This one is a call from thumb code.  We used to look
                 for ARM_THUMB9 and ARM_THUMB12 as well.  We need to look
                 up the target of the call. If it is an arm target, we
-                insert glue.  */
-             
+                insert glue.  If the symbol does not exist it will be
+                given a class of C_EXT and so we will generate a stub
+                for it.  This is not really a problem, since the link
+                is doomed anyway.  */
+
              switch (h->class)
                {
                case C_EXT:
@@ -1716,8 +1878,20 @@ arm_process_before_allocation (abfd, info)
 
   return true;
 }
+#endif /* ! COFF_WITH_PE */
+
+#define coff_bfd_reloc_type_lookup             coff_arm_reloc_type_lookup
+#define coff_relocate_section                  coff_arm_relocate_section
+#define coff_bfd_is_local_label_name           coff_arm_is_local_label_name
+#define coff_adjust_symndx                     coff_arm_adjust_symndx
+#define coff_link_output_has_begun             coff_arm_link_output_has_begun
+#define coff_final_link_postscript             coff_arm_final_link_postscript
+#define coff_bfd_merge_private_bfd_data                coff_arm_merge_private_bfd_data
+#define coff_bfd_print_private_bfd_data                coff_arm_print_private_bfd_data
+#define coff_bfd_set_private_flags              _bfd_coff_arm_set_private_flags
+#define coff_bfd_copy_private_bfd_data          coff_arm_copy_private_bfd_data
+#define coff_bfd_link_hash_table_create                coff_arm_link_hash_table_create
 
-#define coff_relocate_section coff_arm_relocate_section
 
 /* When doing a relocateable link, we want to convert ARM26 relocs
    into ARM26D relocs.  */
@@ -1746,14 +1920,13 @@ coff_arm_adjust_symndx (obfd, info, ibfd, sec, irel, adjustedp)
   return true;
 }
 
-
 /* Called when merging the private data areas of two BFDs.
    This is important as it allows us to detect if we are
    attempting to merge binaries compiled for different ARM
    targets, eg different CPUs or differents APCS's.     */
 
 static boolean
-coff_arm_bfd_merge_private_bfd_data (ibfd, obfd)
+coff_arm_merge_private_bfd_data (ibfd, obfd)
      bfd *   ibfd;
      bfd *   obfd;
 {
@@ -1762,12 +1935,12 @@ coff_arm_bfd_merge_private_bfd_data (ibfd, obfd)
   if (ibfd == obfd)
     return true;
 
-  /* If the two formats are different we cannot merge anything.  */
-  if (ibfd->xvec != obfd->xvec)
-    {
-      bfd_set_error (bfd_error_wrong_format);
-      return false;
-    }
+  /* If the two formats are different we cannot merge anything.
+     This is not an error, since it is permissable to change the
+     input and output formats.  */
+  if (   ibfd->xvec->flavour != bfd_target_coff_flavour
+      || obfd->xvec->flavour != bfd_target_coff_flavour)
+    return true;
 
   /* Verify that the APCS is the same for the two BFDs */
   if (APCS_SET (ibfd))
@@ -1819,7 +1992,7 @@ coff_arm_bfd_merge_private_bfd_data (ibfd, obfd)
          bfd_set_arch_mach (obfd, bfd_get_arch (ibfd), bfd_get_mach (ibfd));
        }
     }
-  
+
   /* Check the interworking support.  */
   if (INTERWORK_SET (ibfd))
     {
@@ -1847,10 +2020,10 @@ coff_arm_bfd_merge_private_bfd_data (ibfd, obfd)
 }
 
 
-/* Display the flags field */
+/* Display the flags field */
 
 static boolean
-coff_arm_bfd_print_private_bfd_data (abfd, ptr)
+coff_arm_print_private_bfd_data (abfd, ptr)
      bfd *   abfd;
      PTR     ptr;
 {
@@ -1858,7 +2031,7 @@ coff_arm_bfd_print_private_bfd_data (abfd, ptr)
   
   BFD_ASSERT (abfd != NULL && ptr != NULL)
   
-  fprintf (file, "private flags = %x", coff_data( abfd )->flags);
+  fprintf (file, "private flags = %x", coff_data (abfd)->flags);
   
   if (APCS_SET (abfd))
     fprintf (file, ": [APCS-%d] [floats passed in %s registers] [%s]",
@@ -1868,8 +2041,10 @@ coff_arm_bfd_print_private_bfd_data (abfd, ptr)
             );
   
   if (INTERWORK_SET (abfd))
-    fprintf (file, ": [interworking %ssupported]",
+    fprintf (file, " [interworking %ssupported]",
             INTERWORK_FLAG (abfd) ? "" : "not " );
+  else
+    fprintf (file, " [interworking flag not initialised]");
   
   fputc ('\n', file);
   
@@ -1880,14 +2055,16 @@ coff_arm_bfd_print_private_bfd_data (abfd, ptr)
 /* Copies the given flags into the coff_tdata.flags field.
    Typically these flags come from the f_flags[] field of
    the COFF filehdr structure, which contains important,
-   target specific information.  */
+   target specific information.
+   Note: Although this function is static, it is explicitly
+   called from both coffcode.h and peicode.h.  */
 
 static boolean
-coff_arm_bfd_set_private_flags (abfd, flags)
+_bfd_coff_arm_set_private_flags (abfd, flags)
        bfd *      abfd;
        flagword   flags;
 {
-  int flag;
+  flagword flag;
 
   BFD_ASSERT (abfd != NULL);
 
@@ -1921,12 +2098,12 @@ coff_arm_bfd_set_private_flags (abfd, flags)
    from one instance of a BFD to another.  */
 
 static boolean
-coff_arm_bfd_copy_private_bfd_data (src, dest)
+coff_arm_copy_private_bfd_data (src, dest)
      bfd *  src;
      bfd *  dest;
 {
   BFD_ASSERT (src != NULL && dest != NULL);
-
   if (src == dest)
     return true;
 
@@ -2008,583 +2185,65 @@ coff_arm_is_local_label_name (abfd, name)
     }
 }
 
-#define coff_bfd_is_local_label_name           coff_arm_is_local_label_name
-#define coff_adjust_symndx                     coff_arm_adjust_symndx
-#define coff_bfd_final_link                    coff_arm_bfd_final_link 
-#define coff_bfd_merge_private_bfd_data                coff_arm_bfd_merge_private_bfd_data
-#define coff_bfd_print_private_bfd_data                coff_arm_bfd_print_private_bfd_data
-#define coff_bfd_set_private_flags              coff_arm_bfd_set_private_flags
-#define coff_bfd_copy_private_bfd_data          coff_arm_bfd_copy_private_bfd_data
-
-#include "coffcode.h"
-
-static boolean
-arm_do_last (abfd)
-     bfd * abfd;
-{
-  if (abfd == bfd_of_glue_owner)
-    return true;
-  else
-    return false;
-}
-
-static bfd *
-arm_get_last()
-{
-  return bfd_of_glue_owner;
-}
-
 /* This piece of machinery exists only to guarantee that the bfd that holds
    the glue section is written last. 
 
    This does depend on bfd_make_section attaching a new section to the
    end of the section list for the bfd. 
 
-   This is otherwise intended to be functionally the same as 
-   cofflink.c:_bfd_coff_final_link(). It is specifically different only 
-   where the ARM_HACKS macro modifies the code. It is left in as a 
-   precise form of comment. krk@cygnus.com  */
-
-#define ARM_HACKS
-
-/* Do the final link step.  */
+   krk@cygnus.com  */
 
 static boolean
-coff_arm_bfd_final_link (abfd, info)
-     bfd *abfd;
-     struct bfd_link_info *info;
+coff_arm_link_output_has_begun (sub, info)
+     bfd * sub;
+     struct coff_final_link_info * info;
 {
-  bfd_size_type symesz;
-  struct coff_final_link_info finfo;
-  boolean debug_merge_allocated;
-  asection *o;
-  struct bfd_link_order *p;
-  size_t max_sym_count;
-  size_t max_lineno_count;
-  size_t max_reloc_count;
-  size_t max_output_reloc_count;
-  size_t max_contents_size;
-  file_ptr rel_filepos;
-  unsigned int relsz;
-  file_ptr line_filepos;
-  unsigned int linesz;
-  bfd *sub;
-  bfd_byte *external_relocs = NULL;
-  char strbuf[STRING_SIZE_SIZE];
-
-  symesz = bfd_coff_symesz (abfd);
-
-  finfo.info = info;
-  finfo.output_bfd = abfd;
-  finfo.strtab = NULL;
-  finfo.section_info = NULL;
-  finfo.last_file_index = -1;
-  finfo.last_bf_index = -1;
-  finfo.internal_syms = NULL;
-  finfo.sec_ptrs = NULL;
-  finfo.sym_indices = NULL;
-  finfo.outsyms = NULL;
-  finfo.linenos = NULL;
-  finfo.contents = NULL;
-  finfo.external_relocs = NULL;
-  finfo.internal_relocs = NULL;
-  debug_merge_allocated = false;
-
-  coff_data (abfd)->link_info = info;
-
-  finfo.strtab = _bfd_stringtab_init ();
-  if (finfo.strtab == NULL)
-    goto error_return;
-
-  if (! coff_debug_merge_hash_table_init (&finfo.debug_merge))
-    goto error_return;
-  debug_merge_allocated = true;
-
-  /* Compute the file positions for all the sections.  */
-  if (! abfd->output_has_begun)
-    bfd_coff_compute_section_file_positions (abfd);
-
-  /* Count the line numbers and relocation entries required for the
-     output file.  Set the file positions for the relocs.  */
-  rel_filepos = obj_relocbase (abfd);
-  relsz = bfd_coff_relsz (abfd);
-  max_contents_size = 0;
-  max_lineno_count = 0;
-  max_reloc_count = 0;
-
-  for (o = abfd->sections; o != NULL; o = o->next)
-    {
-      o->reloc_count = 0;
-      o->lineno_count = 0;
-      for (p = o->link_order_head; p != NULL; p = p->next)
-       {
-
-         if (p->type == bfd_indirect_link_order)
-           {
-             asection *sec;
-
-             sec = p->u.indirect.section;
-
-             /* Mark all sections which are to be included in the
-                link.  This will normally be every section.  We need
-                to do this so that we can identify any sections which
-                the linker has decided to not include.  */
-             sec->linker_mark = true;
-
-             if (info->strip == strip_none
-                 || info->strip == strip_some)
-               o->lineno_count += sec->lineno_count;
-
-             if (info->relocateable)
-               o->reloc_count += sec->reloc_count;
-
-             if (sec->_raw_size > max_contents_size)
-               max_contents_size = sec->_raw_size;
-             if (sec->lineno_count > max_lineno_count)
-               max_lineno_count = sec->lineno_count;
-             if (sec->reloc_count > max_reloc_count)
-               max_reloc_count = sec->reloc_count;
-           }
-         else if (info->relocateable
-                  && (p->type == bfd_section_reloc_link_order
-                      || p->type == bfd_symbol_reloc_link_order))
-           ++o->reloc_count;
-       }
-      if (o->reloc_count == 0)
-       o->rel_filepos = 0;
-      else
-       {
-         o->flags |= SEC_RELOC;
-         o->rel_filepos = rel_filepos;
-         rel_filepos += o->reloc_count * relsz;
-       }
-    }
-
-  /* If doing a relocateable link, allocate space for the pointers we
-     need to keep.  */
-  if (info->relocateable)
-    {
-      unsigned int i;
-
-      /* We use section_count + 1, rather than section_count, because
-         the target_index fields are 1 based.  */
-      finfo.section_info =
-       ((struct coff_link_section_info *)
-        bfd_malloc ((abfd->section_count + 1)
-                    * sizeof (struct coff_link_section_info)));
-      if (finfo.section_info == NULL)
-       goto error_return;
-      for (i = 0; i <= abfd->section_count; i++)
-       {
-         finfo.section_info[i].relocs = NULL;
-         finfo.section_info[i].rel_hashes = NULL;
-       }
-    }
-
-  /* We now know the size of the relocs, so we can determine the file
-     positions of the line numbers.  */
-  line_filepos = rel_filepos;
-  linesz = bfd_coff_linesz (abfd);
-  max_output_reloc_count = 0;
-  for (o = abfd->sections; o != NULL; o = o->next)
-    {
-      if (o->lineno_count == 0)
-       o->line_filepos = 0;
-      else
-       {
-         o->line_filepos = line_filepos;
-         line_filepos += o->lineno_count * linesz;
-       }
-
-      if (o->reloc_count != 0)
-       {
-         /* We don't know the indices of global symbols until we have
-             written out all the local symbols.  For each section in
-             the output file, we keep an array of pointers to hash
-             table entries.  Each entry in the array corresponds to a
-             reloc.  When we find a reloc against a global symbol, we
-             set the corresponding entry in this array so that we can
-             fix up the symbol index after we have written out all the
-             local symbols.
-
-            Because of this problem, we also keep the relocs in
-            memory until the end of the link.  This wastes memory,
-            but only when doing a relocateable link, which is not the
-            common case.  */
-         BFD_ASSERT (info->relocateable);
-         finfo.section_info[o->target_index].relocs =
-           ((struct internal_reloc *)
-            bfd_malloc (o->reloc_count * sizeof (struct internal_reloc)));
-         finfo.section_info[o->target_index].rel_hashes =
-           ((struct coff_link_hash_entry **)
-            bfd_malloc (o->reloc_count
-                    * sizeof (struct coff_link_hash_entry *)));
-         if (finfo.section_info[o->target_index].relocs == NULL
-             || finfo.section_info[o->target_index].rel_hashes == NULL)
-           goto error_return;
-
-         if (o->reloc_count > max_output_reloc_count)
-           max_output_reloc_count = o->reloc_count;
-       }
-
-      /* Reset the reloc and lineno counts, so that we can use them to
-        count the number of entries we have output so far.  */
-      o->reloc_count = 0;
-      o->lineno_count = 0;
-    }
-
-  obj_sym_filepos (abfd) = line_filepos;
-
-  /* Figure out the largest number of symbols in an input BFD.  Take
-     the opportunity to clear the output_has_begun fields of all the
-     input BFD's.  */
-  max_sym_count = 0;
-  for (sub = info->input_bfds; sub != NULL; sub = sub->link_next)
-    {
-      size_t sz;
-
-      sub->output_has_begun = false;
-      sz = obj_raw_syment_count (sub);
-      if (sz > max_sym_count)
-       max_sym_count = sz;
-    }
-
-  /* Allocate some buffers used while linking.  */
-  finfo.internal_syms = ((struct internal_syment *)
-                        bfd_malloc (max_sym_count
-                                    * sizeof (struct internal_syment)));
-  finfo.sec_ptrs = (asection **) bfd_malloc (max_sym_count
-                                            * sizeof (asection *));
-  finfo.sym_indices = (long *) bfd_malloc (max_sym_count * sizeof (long));
-  finfo.outsyms = ((bfd_byte *)
-                  bfd_malloc ((size_t) ((max_sym_count + 1) * symesz)));
-  finfo.linenos = (bfd_byte *) bfd_malloc (max_lineno_count
-                                      * bfd_coff_linesz (abfd));
-  finfo.contents = (bfd_byte *) bfd_malloc (max_contents_size);
-  finfo.external_relocs = (bfd_byte *) bfd_malloc (max_reloc_count * relsz);
-  if (! info->relocateable)
-    finfo.internal_relocs = ((struct internal_reloc *)
-                            bfd_malloc (max_reloc_count
-                                        * sizeof (struct internal_reloc)));
-  if ((finfo.internal_syms == NULL && max_sym_count > 0)
-      || (finfo.sec_ptrs == NULL && max_sym_count > 0)
-      || (finfo.sym_indices == NULL && max_sym_count > 0)
-      || finfo.outsyms == NULL
-      || (finfo.linenos == NULL && max_lineno_count > 0)
-      || (finfo.contents == NULL && max_contents_size > 0)
-      || (finfo.external_relocs == NULL && max_reloc_count > 0)
-      || (! info->relocateable
-         && finfo.internal_relocs == NULL
-         && max_reloc_count > 0))
-    goto error_return;
-
-  /* We now know the position of everything in the file, except that
-     we don't know the size of the symbol table and therefore we don't
-     know where the string table starts.  We just build the string
-     table in memory as we go along.  We process all the relocations
-     for a single input file at once.  */
-  obj_raw_syment_count (abfd) = 0;
-
-  if (coff_backend_info (abfd)->_bfd_coff_start_final_link)
-    {
-      if (! bfd_coff_start_final_link (abfd, info))
-       goto error_return;
-    }
-
-  for (o = abfd->sections; o != NULL; o = o->next)
-    {
-      for (p = o->link_order_head; p != NULL; p = p->next)
-       {
-         if (p->type == bfd_indirect_link_order
-             && (bfd_get_flavour (p->u.indirect.section->owner)
-                 == bfd_target_coff_flavour))
-           {
-             sub = p->u.indirect.section->owner;
-#ifdef ARM_HACKS
-             if (! sub->output_has_begun && !arm_do_last(sub))
-#else
-             if (! sub->output_has_begun)
-#endif
-               {
-                 if (! _bfd_coff_link_input_bfd (&finfo, sub))
-                   goto error_return;
-                 sub->output_has_begun = true;
-               }
-           }
-         else if (p->type == bfd_section_reloc_link_order
-                  || p->type == bfd_symbol_reloc_link_order)
-           {
-             if (! _bfd_coff_reloc_link_order (abfd, &finfo, o, p))
-               goto error_return;
-           }
-         else
-           {
-             if (! _bfd_default_link_order (abfd, info, o, p))
-               goto error_return;
-           }
-       }
-    }
-
-#ifdef ARM_HACKS
-  {
-    extern bfd* arm_get_last();
-    bfd* last_one = arm_get_last();
-    if (last_one)
-      {
-       if (! _bfd_coff_link_input_bfd (&finfo, last_one))
-         goto error_return;
-      }
-    last_one->output_has_begun = true;
-  }
-#endif
-
-  /* Free up the buffers used by _bfd_coff_link_input_bfd.  */
-
-  coff_debug_merge_hash_table_free (&finfo.debug_merge);
-  debug_merge_allocated = false;
-
-  if (finfo.internal_syms != NULL)
-    {
-      free (finfo.internal_syms);
-      finfo.internal_syms = NULL;
-    }
-  if (finfo.sec_ptrs != NULL)
-    {
-      free (finfo.sec_ptrs);
-      finfo.sec_ptrs = NULL;
-    }
-  if (finfo.sym_indices != NULL)
-    {
-      free (finfo.sym_indices);
-      finfo.sym_indices = NULL;
-    }
-  if (finfo.linenos != NULL)
-    {
-      free (finfo.linenos);
-      finfo.linenos = NULL;
-    }
-  if (finfo.contents != NULL)
-    {
-      free (finfo.contents);
-      finfo.contents = NULL;
-    }
-  if (finfo.external_relocs != NULL)
-    {
-      free (finfo.external_relocs);
-      finfo.external_relocs = NULL;
-    }
-  if (finfo.internal_relocs != NULL)
-    {
-      free (finfo.internal_relocs);
-      finfo.internal_relocs = NULL;
-    }
-
-  /* The value of the last C_FILE symbol is supposed to be the symbol
-     index of the first external symbol.  Write it out again if
-     necessary.  */
-  if (finfo.last_file_index != -1
-      && (unsigned int) finfo.last_file.n_value != obj_raw_syment_count (abfd))
-    {
-      finfo.last_file.n_value = obj_raw_syment_count (abfd);
-      bfd_coff_swap_sym_out (abfd, (PTR) &finfo.last_file,
-                            (PTR) finfo.outsyms);
-      if (bfd_seek (abfd,
-                   (obj_sym_filepos (abfd)
-                    + finfo.last_file_index * symesz),
-                   SEEK_SET) != 0
-         || bfd_write (finfo.outsyms, symesz, 1, abfd) != symesz)
-       return false;
-    }
-
-  /* Write out the global symbols.  */
-  finfo.failed = false;
-  coff_link_hash_traverse (coff_hash_table (info), _bfd_coff_write_global_sym,
-                          (PTR) &finfo);
-  if (finfo.failed)
-    goto error_return;
-
-  /* The outsyms buffer is used by _bfd_coff_write_global_sym.  */
-  if (finfo.outsyms != NULL)
-    {
-      free (finfo.outsyms);
-      finfo.outsyms = NULL;
-    }
-
-  if (info->relocateable)
-    {
-      /* Now that we have written out all the global symbols, we know
-        the symbol indices to use for relocs against them, and we can
-        finally write out the relocs.  */
-      external_relocs = ((bfd_byte *)
-                        bfd_malloc (max_output_reloc_count * relsz));
-      if (external_relocs == NULL)
-       goto error_return;
-
-      for (o = abfd->sections; o != NULL; o = o->next)
-       {
-         struct internal_reloc *irel;
-         struct internal_reloc *irelend;
-         struct coff_link_hash_entry **rel_hash;
-         bfd_byte *erel;
-
-         if (o->reloc_count == 0)
-           continue;
-
-         irel = finfo.section_info[o->target_index].relocs;
-         irelend = irel + o->reloc_count;
-         rel_hash = finfo.section_info[o->target_index].rel_hashes;
-         erel = external_relocs;
-         for (; irel < irelend; irel++, rel_hash++, erel += relsz)
-           {
-             if (*rel_hash != NULL)
-               {
-                 BFD_ASSERT ((*rel_hash)->indx >= 0);
-                 irel->r_symndx = (*rel_hash)->indx;
-               }
-             bfd_coff_swap_reloc_out (abfd, (PTR) irel, (PTR) erel);
-           }
-
-         if (bfd_seek (abfd, o->rel_filepos, SEEK_SET) != 0
-             || bfd_write ((PTR) external_relocs, relsz, o->reloc_count,
-                           abfd) != relsz * o->reloc_count)
-           goto error_return;
-       }
-
-      free (external_relocs);
-      external_relocs = NULL;
-    }
-
-  /* Free up the section information.  */
-  if (finfo.section_info != NULL)
-    {
-      unsigned int i;
-
-      for (i = 0; i < abfd->section_count; i++)
-       {
-         if (finfo.section_info[i].relocs != NULL)
-           free (finfo.section_info[i].relocs);
-         if (finfo.section_info[i].rel_hashes != NULL)
-           free (finfo.section_info[i].rel_hashes);
-       }
-      free (finfo.section_info);
-      finfo.section_info = NULL;
-    }
-
-  /* If we have optimized stabs strings, output them.  */
-  if (coff_hash_table (info)->stab_info != NULL)
-    {
-      if (! _bfd_write_stab_strings (abfd, &coff_hash_table (info)->stab_info))
-       return false;
-    }
+  return (sub->output_has_begun || sub == coff_arm_hash_table (info->info)->bfd_of_glue_owner);
+}
 
-  /* Write out the string table.  */
-  if (obj_raw_syment_count (abfd) != 0)
+static boolean
+coff_arm_final_link_postscript (abfd, pfinfo)
+     bfd * abfd;
+     struct coff_final_link_info * pfinfo;
+{
+  struct coff_arm_link_hash_table * globals = coff_arm_hash_table (pfinfo->info);
+  
+  BFD_ASSERT (globals != NULL);
+  
+  if (globals->bfd_of_glue_owner != NULL)
     {
-      if (bfd_seek (abfd,
-                   (obj_sym_filepos (abfd)
-                    + obj_raw_syment_count (abfd) * symesz),
-                   SEEK_SET) != 0)
-       return false;
-
-#if STRING_SIZE_SIZE == 4
-      bfd_h_put_32 (abfd,
-                   _bfd_stringtab_size (finfo.strtab) + STRING_SIZE_SIZE,
-                   (bfd_byte *) strbuf);
-#else
- #error Change bfd_h_put_32
-#endif
-
-      if (bfd_write (strbuf, 1, STRING_SIZE_SIZE, abfd) != STRING_SIZE_SIZE)
-       return false;
-
-      if (! _bfd_stringtab_emit (abfd, finfo.strtab))
+      if (! _bfd_coff_link_input_bfd (pfinfo, globals->bfd_of_glue_owner))
        return false;
+      
+      globals->bfd_of_glue_owner->output_has_begun = true;
     }
-
-  _bfd_stringtab_free (finfo.strtab);
-
-  /* Setting bfd_get_symcount to 0 will cause write_object_contents to
-     not try to write out the symbols.  */
-  bfd_get_symcount (abfd) = 0;
-
+  
   return true;
+}
 
- error_return:
-  if (debug_merge_allocated)
-    coff_debug_merge_hash_table_free (&finfo.debug_merge);
-  if (finfo.strtab != NULL)
-    _bfd_stringtab_free (finfo.strtab);
-  if (finfo.section_info != NULL)
-    {
-      unsigned int i;
+#if 0
+#define coff_SWAP_sym_in  arm_bfd_coff_swap_sym_in
 
-      for (i = 0; i < abfd->section_count; i++)
-       {
-         if (finfo.section_info[i].relocs != NULL)
-           free (finfo.section_info[i].relocs);
-         if (finfo.section_info[i].rel_hashes != NULL)
-           free (finfo.section_info[i].rel_hashes);
-       }
-      free (finfo.section_info);
-    }
-  if (finfo.internal_syms != NULL)
-    free (finfo.internal_syms);
-  if (finfo.sec_ptrs != NULL)
-    free (finfo.sec_ptrs);
-  if (finfo.sym_indices != NULL)
-    free (finfo.sym_indices);
-  if (finfo.outsyms != NULL)
-    free (finfo.outsyms);
-  if (finfo.linenos != NULL)
-    free (finfo.linenos);
-  if (finfo.contents != NULL)
-    free (finfo.contents);
-  if (finfo.external_relocs != NULL)
-    free (finfo.external_relocs);
-  if (finfo.internal_relocs != NULL)
-    free (finfo.internal_relocs);
-  if (external_relocs != NULL)
-    free (external_relocs);
-  return false;
-}
+static void coff_swap_sym_in PARAMS ((bfd *, PTR, PTR));
 
+/* Sepcial version of symbol swapper, used to grab a bfd
+   onto which the glue sections can be attached.  */
 static void
 arm_bfd_coff_swap_sym_in (abfd, ext1, in1)
-     bfd            *abfd;
-     PTR ext1;
-     PTR in1;
+     bfd abfd;
+     PTR   ext1;
+     PTR   in1;
 {
-  SYMENT *ext = (SYMENT *)ext1;
-  struct internal_syment      *in = (struct internal_syment *)in1;
   flagword flags;
-  register asection *s;
-
-  if( ext->e.e_name[0] == 0) {
-    in->_n._n_n._n_zeroes = 0;
-    in->_n._n_n._n_offset = bfd_h_get_32(abfd, (bfd_byte *) ext->e.e.e_offset);
-  }
-  else {
-#if SYMNMLEN != E_SYMNMLEN
-   -> Error, we need to cope with truncating or extending SYMNMLEN!;
-#else
-    memcpy(in->_n._n_name, ext->e.e_name, SYMNMLEN);
-#endif
-  }
-  in->n_value = bfd_h_get_32(abfd, (bfd_byte *) ext->e_value); 
-  in->n_scnum = bfd_h_get_16(abfd, (bfd_byte *) ext->e_scnum);
-  if (sizeof(ext->e_type) == 2){
-    in->n_type = bfd_h_get_16(abfd, (bfd_byte *) ext->e_type);
-  }
-  else {
-    in->n_type = bfd_h_get_32(abfd, (bfd_byte *) ext->e_type);
-  }
-  in->n_sclass = bfd_h_get_8(abfd, ext->e_sclass);
-  in->n_numaux = bfd_h_get_8(abfd, ext->e_numaux);
-
-  if (bfd_of_glue_owner != 0) /* we already have a toc, so go home */
+  register asection * s;
+
+  /* Do the normal swap in.  */
+  coff_swap_sym_in (abfd, ext1, in1);
+
+  if (bfd_of_glue_owner != NULL) /* we already have a toc, so go home */
     return;
 
-  /* save the bfd for later allocation */
+  /* Save the bfd for later allocation.  */
   bfd_of_glue_owner = abfd;
 
   s = bfd_get_section_by_name (bfd_of_glue_owner , 
@@ -2594,7 +2253,7 @@ arm_bfd_coff_swap_sym_in (abfd, ext1, in1)
     {
       flags = SEC_ALLOC | SEC_LOAD | SEC_HAS_CONTENTS | SEC_IN_MEMORY ;
       
-      s = bfd_make_section (bfd_of_glue_owner , ARM2THUMB_GLUE_SECTION_NAME);
+      s = bfd_make_section (bfd_of_glue_owner, ARM2THUMB_GLUE_SECTION_NAME);
 
       if (s == NULL
          || !bfd_set_section_flags (bfd_of_glue_owner, s, flags)
@@ -2605,13 +2264,13 @@ arm_bfd_coff_swap_sym_in (abfd, ext1, in1)
        }
     }
 
-  s = bfd_get_section_by_name ( bfd_of_glue_owner , THUMB2ARM_GLUE_SECTION_NAME);
+  s = bfd_get_section_by_name (bfd_of_glue_owner, THUMB2ARM_GLUE_SECTION_NAME);
 
   if (s == NULL) 
     {
       flags = SEC_ALLOC | SEC_LOAD | SEC_HAS_CONTENTS | SEC_IN_MEMORY ;
       
-      s = bfd_make_section (bfd_of_glue_owner , THUMB2ARM_GLUE_SECTION_NAME);
+      s = bfd_make_section (bfd_of_glue_owner, THUMB2ARM_GLUE_SECTION_NAME);
       
       if (s == NULL
          || !bfd_set_section_flags (bfd_of_glue_owner, s, flags)
@@ -2624,35 +2283,9 @@ arm_bfd_coff_swap_sym_in (abfd, ext1, in1)
   
   return;
 }
-
-static bfd_coff_backend_data arm_bfd_coff_std_swap_table =
-{
-  coff_swap_aux_in, arm_bfd_coff_swap_sym_in, coff_swap_lineno_in,
-  coff_swap_aux_out, coff_swap_sym_out,
-  coff_swap_lineno_out, coff_swap_reloc_out,
-  coff_swap_filehdr_out, coff_swap_aouthdr_out,
-  coff_swap_scnhdr_out,
-  FILHSZ, AOUTSZ, SCNHSZ, SYMESZ, AUXESZ, RELSZ, LINESZ,
-#ifdef COFF_LONG_FILENAMES
-  true,
-#else
-  false,
-#endif
-#ifdef COFF_LONG_SECTION_NAMES
-  true,
-#else
-  false,
 #endif
-  COFF_DEFAULT_SECTION_ALIGNMENT_POWER,
-  coff_swap_filehdr_in, coff_swap_aouthdr_in, coff_swap_scnhdr_in,
-  coff_swap_reloc_in, coff_bad_format_hook, coff_set_arch_mach_hook,
-  coff_mkobject_hook, styp_to_sec_flags, coff_set_alignment_hook,
-  coff_slurp_symbol_table, symname_in_debug_hook, coff_pointerize_aux_hook,
-  coff_print_aux, coff_reloc16_extra_cases, coff_reloc16_estimate,
-  coff_sym_is_global, coff_compute_section_file_positions,
-  coff_start_final_link, coff_relocate_section, coff_rtype_to_howto,
-  coff_adjust_symndx, coff_link_add_one_symbol
-};
+
+#include "coffcode.h"
 
 const bfd_target
 #ifdef TARGET_LITTLE_SYM
@@ -2714,7 +2347,7 @@ armcoff_little_vec =
      BFD_JUMP_TABLE_LINK (coff),
      BFD_JUMP_TABLE_DYNAMIC (_bfd_nodynamic),
 
-  & arm_bfd_coff_std_swap_table,
+  (PTR) & bfd_coff_std_swap_table,
 };
 
 const bfd_target
@@ -2777,5 +2410,5 @@ armcoff_big_vec =
      BFD_JUMP_TABLE_LINK (coff),
      BFD_JUMP_TABLE_DYNAMIC (_bfd_nodynamic),
 
-  & arm_bfd_coff_std_swap_table,
+  (PTR) & bfd_coff_std_swap_table,
 };
This page took 0.044381 seconds and 4 git commands to generate.