* dep-in.sed: Cope with absolute paths.
[deliverable/binutils-gdb.git] / bfd / coff-arm.c
index 8bb4e6c4940b0fbaf45133af3bad7e9272ba257c..798cd49be181f30bb7eb5d70a518364ae95a1b33 100644 (file)
@@ -1,6 +1,6 @@
 /* BFD back-end for ARM COFF files.
    Copyright 1990, 1991, 1992, 1993, 1994, 1995, 1996, 1997, 1998, 1999,
-   2000, 2001
+   2000, 2001, 2002
    Free Software Foundation, Inc.
    Written by Cygnus Support.
 
@@ -36,18 +36,39 @@ Foundation, Inc., 59 Temple Place - Suite 330, Boston, MA 02111-1307, USA.  */
 
 /* 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)
-#define APCS_SET(           abfd )     (coff_data (abfd)->flags & F_APCS_SET)
-#define SET_APCS_FLAGS(     abfd, flgs)        (coff_data (abfd)->flags = \
-                                       (coff_data (abfd)->flags & ~ (F_APCS_26 | F_APCS_FLOAT | F_PIC)) \
-                                        | (flgs | F_APCS_SET))
-#define INTERWORK_FLAG(     abfd )     (coff_data (abfd)->flags & F_INTERWORK)
-#define INTERWORK_SET(      abfd )     (coff_data (abfd)->flags & F_INTERWORK_SET)
-#define SET_INTERWORK_FLAG( abfd, flg )        (coff_data (abfd)->flags = \
-                                       (coff_data (abfd)->flags & ~ F_INTERWORK) \
-                                        | (flg | F_INTERWORK_SET))
+#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)
+
+#define APCS_SET(abfd) \
+  (coff_data (abfd)->flags & F_APCS_SET)
+
+#define SET_APCS_FLAGS(abfd, flgs) \
+  do                                                                   \
+    {                                                                  \
+      coff_data (abfd)->flags &= ~(F_APCS_26 | F_APCS_FLOAT | F_PIC);  \
+      coff_data (abfd)->flags |= (flgs) | F_APCS_SET;                  \
+    }                                                                  \
+  while (0)
+
+#define INTERWORK_FLAG(abfd) \
+  (coff_data (abfd)->flags & F_INTERWORK)
+
+#define INTERWORK_SET(abfd) \
+  (coff_data (abfd)->flags & F_INTERWORK_SET)
+
+#define SET_INTERWORK_FLAG(abfd, flg) \
+  do                                                                   \
+    {                                                                  \
+      coff_data (abfd)->flags &= ~F_INTERWORK;                         \
+      coff_data (abfd)->flags |= (flg) | F_INTERWORK_SET;              \
+    }                                                                  \
+  while (0)
 
 #ifndef NUM_ELEM
 #define NUM_ELEM(a) ((sizeof (a)) / sizeof ((a)[0]))
@@ -79,20 +100,21 @@ static boolean coff_arm_adjust_symndx
           asection *, struct internal_reloc *, boolean *));
 static reloc_howto_type * coff_arm_rtype_to_howto
   PARAMS ((bfd *, asection *, struct internal_reloc *,
-          struct coff_link_hash_entry *, struct internal_syment *, bfd_vma *));
+          struct coff_link_hash_entry *, struct internal_syment *,
+          bfd_vma *));
 static bfd_reloc_status_type coff_thumb_pcrel_common
   PARAMS ((bfd *, arelent *, asymbol *, PTR, asection *, bfd *, char **,
           thumb_pcrel_branchtype));
-static CONST struct reloc_howto_struct * coff_arm_reloc_type_lookup
+static const struct reloc_howto_struct * coff_arm_reloc_type_lookup
   PARAMS ((bfd *, bfd_reloc_code_real_type));
 static struct bfd_link_hash_table * coff_arm_link_hash_table_create
   PARAMS ((bfd *));
 static insn32 insert_thumb_branch
   PARAMS ((insn32, int));
 static struct coff_link_hash_entry * find_thumb_glue
-  PARAMS ((struct bfd_link_info *, CONST char *, bfd *));
+  PARAMS ((struct bfd_link_info *, const char *, bfd *));
 static struct coff_link_hash_entry * find_arm_glue
-  PARAMS ((struct bfd_link_info *, CONST char *, bfd *));
+  PARAMS ((struct bfd_link_info *, const char *, bfd *));
 #ifndef COFF_IMAGE_WITH_PE
 static void record_arm_to_thumb_glue
   PARAMS ((struct bfd_link_info *, struct coff_link_hash_entry *));
@@ -113,12 +135,13 @@ static boolean coff_arm_link_output_has_begun
   PARAMS ((bfd *, struct coff_final_link_info *));
 static boolean coff_arm_final_link_postscript
   PARAMS ((bfd *, struct coff_final_link_info *));
+static void arm_emit_base_file_entry
+  PARAMS ((struct bfd_link_info *, bfd *, asection *, bfd_vma));
 
 /* The linker script knows the section names for placement.
    The entry_names are used to do simple name mangling on the stubs.
    Given a function name, and its type, the stub can be found. The
-   name can be changed. The only requirement is the %s be present.
-   */
+   name can be changed. The only requirement is the %s be present.  */
 
 #define THUMB2ARM_GLUE_SECTION_NAME ".glue_7t"
 #define THUMB2ARM_GLUE_ENTRY_NAME   "__%s_from_thumb"
@@ -127,6 +150,7 @@ static boolean coff_arm_final_link_postscript
 #define ARM2THUMB_GLUE_ENTRY_NAME   "__%s_from_arm"
 
 /* Used by the assembler.  */
+
 static bfd_reloc_status_type
 coff_arm_reloc (abfd, reloc_entry, symbol, data, input_section, output_bfd,
                 error_message)
@@ -144,8 +168,9 @@ coff_arm_reloc (abfd, reloc_entry, symbol, data, input_section, output_bfd,
 
   diff = reloc_entry->addend;
 
-#define DOIT(x) \
-  x = ((x & ~howto->dst_mask) | (((x & howto->src_mask) + diff) & howto->dst_mask))
+#define DOIT(x)                                                        \
+  x = ((x & ~howto->dst_mask)                                  \
+       | (((x & howto->src_mask) + diff) & howto->dst_mask))
 
     if (diff != 0)
       {
@@ -166,7 +191,7 @@ coff_arm_reloc (abfd, reloc_entry, symbol, data, input_section, output_bfd,
            {
              short x = bfd_get_16 (abfd, addr);
              DOIT (x);
-             bfd_put_16 (abfd, x, addr);
+             bfd_put_16 (abfd, (bfd_vma) x, addr);
            }
            break;
 
@@ -174,7 +199,7 @@ coff_arm_reloc (abfd, reloc_entry, symbol, data, input_section, output_bfd,
            {
              long x = bfd_get_32 (abfd, addr);
              DOIT (x);
-             bfd_put_32 (abfd, x, addr);
+             bfd_put_32 (abfd, (bfd_vma) x, addr);
            }
            break;
 
@@ -233,300 +258,300 @@ coff_arm_reloc (abfd, reloc_entry, symbol, data, input_section, output_bfd,
 #endif
 
 static reloc_howto_type aoutarm_std_reloc_howto[] =
-{
-  /* type              rs size bsz  pcrel bitpos ovrf                     sf name     part_inpl readmask  setmask    pcdone */
+  {
 #ifdef ARM_WINCE
-  EMPTY_HOWTO (-1),
-  HOWTO (ARM_32,
-       0,
-       2,
-       32,
-       false,
-       0,
-       complain_overflow_bitfield,
-       coff_arm_reloc,
-       "ARM_32",
-        true,
-       0xffffffff,
-       0xffffffff,
-       PCRELOFFSET),
-  HOWTO (ARM_RVA32,
-       0,
-       2,
-       32,
-       false,
-       0,
-       complain_overflow_bitfield,
-       coff_arm_reloc,
-       "ARM_RVA32",
-        true,
-       0xffffffff,
-       0xffffffff,
-       PCRELOFFSET),
-  HOWTO (ARM_26,
-       2,
-       2,
-       24,
-       true,
-       0,
-       complain_overflow_signed,
-       aoutarm_fix_pcrel_26 ,
-       "ARM_26",
-       false,
-       0x00ffffff,
-       0x00ffffff,
-       PCRELOFFSET),
-  HOWTO (ARM_THUMB12,
-       1,
-       1,
-       11,
-       true,
-       0,
-       complain_overflow_signed,
-       coff_thumb_pcrel_12 ,
-       "ARM_THUMB12",
-       false,
-       0x000007ff,
-       0x000007ff,
-       PCRELOFFSET),
-  HOWTO (ARM_26D,
-       2,
-       2,
-       24,
-       false,
-       0,
-       complain_overflow_dont,
-       aoutarm_fix_pcrel_26_done,
-       "ARM_26D",
-       true,
-       0x00ffffff,
-       0x0,
-       false),
-  EMPTY_HOWTO (-1),
-  EMPTY_HOWTO (-1),
-  EMPTY_HOWTO (-1),
-  EMPTY_HOWTO (-1),
-  EMPTY_HOWTO (-1),
-  EMPTY_HOWTO (-1),
-  EMPTY_HOWTO (-1),
-  EMPTY_HOWTO (-1),
-  HOWTO (ARM_SECTION,
-       0,
-       1,
-       16,
-       false,
-       0,
-       complain_overflow_bitfield,
-       coff_arm_reloc,
-       "ARM_16",
-       true,
-       0x0000ffff,
-       0x0000ffff,
-       PCRELOFFSET),
-  HOWTO (ARM_SECREL,
-       0,
-       2,
-       32,
-       false,
-       0,
-       complain_overflow_bitfield,
-       coff_arm_reloc,
-       "ARM_32",
-        true,
-       0xffffffff,
-       0xffffffff,
-       PCRELOFFSET),
+    EMPTY_HOWTO (-1),
+    HOWTO (ARM_32,
+          0,
+          2,
+          32,
+          false,
+          0,
+          complain_overflow_bitfield,
+          coff_arm_reloc,
+          "ARM_32",
+          true,
+          0xffffffff,
+          0xffffffff,
+          PCRELOFFSET),
+    HOWTO (ARM_RVA32,
+          0,
+          2,
+          32,
+          false,
+          0,
+          complain_overflow_bitfield,
+          coff_arm_reloc,
+          "ARM_RVA32",
+          true,
+          0xffffffff,
+          0xffffffff,
+          PCRELOFFSET),
+    HOWTO (ARM_26,
+          2,
+          2,
+          24,
+          true,
+          0,
+          complain_overflow_signed,
+          aoutarm_fix_pcrel_26 ,
+          "ARM_26",
+          false,
+          0x00ffffff,
+          0x00ffffff,
+          PCRELOFFSET),
+    HOWTO (ARM_THUMB12,
+          1,
+          1,
+          11,
+          true,
+          0,
+          complain_overflow_signed,
+          coff_thumb_pcrel_12 ,
+          "ARM_THUMB12",
+          false,
+          0x000007ff,
+          0x000007ff,
+          PCRELOFFSET),
+    HOWTO (ARM_26D,
+          2,
+          2,
+          24,
+          false,
+          0,
+          complain_overflow_dont,
+          aoutarm_fix_pcrel_26_done,
+          "ARM_26D",
+          true,
+          0x00ffffff,
+          0x0,
+          false),
+    EMPTY_HOWTO (-1),
+    EMPTY_HOWTO (-1),
+    EMPTY_HOWTO (-1),
+    EMPTY_HOWTO (-1),
+    EMPTY_HOWTO (-1),
+    EMPTY_HOWTO (-1),
+    EMPTY_HOWTO (-1),
+    EMPTY_HOWTO (-1),
+    HOWTO (ARM_SECTION,
+          0,
+          1,
+          16,
+          false,
+          0,
+          complain_overflow_bitfield,
+          coff_arm_reloc,
+          "ARM_16",
+          true,
+          0x0000ffff,
+          0x0000ffff,
+          PCRELOFFSET),
+    HOWTO (ARM_SECREL,
+          0,
+          2,
+          32,
+          false,
+          0,
+          complain_overflow_bitfield,
+          coff_arm_reloc,
+          "ARM_32",
+          true,
+          0xffffffff,
+          0xffffffff,
+          PCRELOFFSET),
 #else /* not ARM_WINCE */
-  HOWTO(ARM_8,                 /* type */
-       0,                      /* rightshift */
-       0,                      /* size */
-       8,                      /* bitsize */
-       false,                  /* pc_relative */
-       0,                      /* bitpos */
-       complain_overflow_bitfield, /* complain_on_overflow */
-       coff_arm_reloc,         /* special_function */
-       "ARM_8",                /* name */
-        true,                  /* partial_inplace */
-       0x000000ff,             /* src_mask */
-       0x000000ff,             /* dst_mask */
-       PCRELOFFSET             /* pcrel_offset */),
-  HOWTO(ARM_16,
-       0,
-       1,
-       16,
-       false,
-       0,
-       complain_overflow_bitfield,
-       coff_arm_reloc,
-       "ARM_16",
-       true,
-       0x0000ffff,
-       0x0000ffff,
-       PCRELOFFSET),
-  HOWTO(ARM_32,
-       0,
-       2,
-       32,
-       false,
-       0,
-       complain_overflow_bitfield,
-       coff_arm_reloc,
-       "ARM_32",
-        true,
-       0xffffffff,
-       0xffffffff,
-       PCRELOFFSET),
-  HOWTO(ARM_26,
-       2,
-       2,
-       24,
-       true,
-       0,
-       complain_overflow_signed,
-       aoutarm_fix_pcrel_26 ,
-       "ARM_26",
-       false,
-       0x00ffffff,
-       0x00ffffff,
-       PCRELOFFSET),
-  HOWTO(ARM_DISP8,
-       0,
-       0,
-       8,
-       true,
-       0,
-       complain_overflow_signed,
-       coff_arm_reloc,
-       "ARM_DISP8",
-       true,
-       0x000000ff,
-       0x000000ff,
-       true),
-  HOWTO( ARM_DISP16,
-       0,
-       1,
-       16,
-       true,
-       0,
-       complain_overflow_signed,
-       coff_arm_reloc,
-       "ARM_DISP16",
-       true,
-       0x0000ffff,
-       0x0000ffff,
-       true),
-  HOWTO( ARM_DISP32,
-       0,
-       2,
-       32,
-       true,
-       0,
-       complain_overflow_signed,
-       coff_arm_reloc,
-       "ARM_DISP32",
-       true,
-       0xffffffff,
-       0xffffffff,
-       true),
-  HOWTO( ARM_26D,
-       2,
-       2,
-       24,
-       false,
-       0,
-       complain_overflow_dont,
-       aoutarm_fix_pcrel_26_done,
-       "ARM_26D",
-       true,
-       0x00ffffff,
-       0x0,
-       false),
-  /* 8 is unused */
-  EMPTY_HOWTO (-1),
-  HOWTO( ARM_NEG16,
-       0,
-       -1,
-       16,
-       false,
-       0,
-       complain_overflow_bitfield,
-       coff_arm_reloc,
-       "ARM_NEG16",
-        true,
-       0x0000ffff,
-       0x0000ffff,
-       false),
-  HOWTO( ARM_NEG32,
-       0,
-       -2,
-       32,
-       false,
-       0,
-       complain_overflow_bitfield,
-       coff_arm_reloc,
-       "ARM_NEG32",
-        true,
-       0xffffffff,
-       0xffffffff,
-       false),
-  HOWTO( ARM_RVA32,
-       0,
-       2,
-       32,
-       false,
-       0,
-       complain_overflow_bitfield,
-       coff_arm_reloc,
-       "ARM_RVA32",
-        true,
-       0xffffffff,
-       0xffffffff,
-       PCRELOFFSET),
-  HOWTO( ARM_THUMB9,
-       1,
-       1,
-       8,
-       true,
-       0,
-       complain_overflow_signed,
-       coff_thumb_pcrel_9 ,
-       "ARM_THUMB9",
-       false,
-       0x000000ff,
-       0x000000ff,
-       PCRELOFFSET),
-  HOWTO( ARM_THUMB12,
-       1,
-       1,
-       11,
-       true,
-       0,
-       complain_overflow_signed,
-       coff_thumb_pcrel_12 ,
-       "ARM_THUMB12",
-       false,
-       0x000007ff,
-       0x000007ff,
-       PCRELOFFSET),
-  HOWTO( ARM_THUMB23,
-       1,
-       2,
-       22,
-       true,
-       0,
-       complain_overflow_signed,
-       coff_thumb_pcrel_23 ,
-       "ARM_THUMB23",
-       false,
-       0x07ff07ff,
-       0x07ff07ff,
-       PCRELOFFSET)
+    HOWTO (ARM_8,                      /* type */
+          0,                   /* rightshift */
+          0,                   /* size */
+          8,                   /* bitsize */
+          false,                       /* pc_relative */
+          0,                   /* bitpos */
+          complain_overflow_bitfield, /* complain_on_overflow */
+          coff_arm_reloc,              /* special_function */
+          "ARM_8",             /* name */
+          true,                        /* partial_inplace */
+          0x000000ff,          /* src_mask */
+          0x000000ff,          /* dst_mask */
+          PCRELOFFSET          /* pcrel_offset */),
+    HOWTO (ARM_16,
+          0,
+          1,
+          16,
+          false,
+          0,
+          complain_overflow_bitfield,
+          coff_arm_reloc,
+          "ARM_16",
+          true,
+          0x0000ffff,
+          0x0000ffff,
+          PCRELOFFSET),
+    HOWTO (ARM_32,
+          0,
+          2,
+          32,
+          false,
+          0,
+          complain_overflow_bitfield,
+          coff_arm_reloc,
+          "ARM_32",
+          true,
+          0xffffffff,
+          0xffffffff,
+          PCRELOFFSET),
+    HOWTO (ARM_26,
+          2,
+          2,
+          24,
+          true,
+          0,
+          complain_overflow_signed,
+          aoutarm_fix_pcrel_26 ,
+          "ARM_26",
+          false,
+          0x00ffffff,
+          0x00ffffff,
+          PCRELOFFSET),
+    HOWTO (ARM_DISP8,
+          0,
+          0,
+          8,
+          true,
+          0,
+          complain_overflow_signed,
+          coff_arm_reloc,
+          "ARM_DISP8",
+          true,
+          0x000000ff,
+          0x000000ff,
+          true),
+    HOWTO (ARM_DISP16,
+          0,
+          1,
+          16,
+          true,
+          0,
+          complain_overflow_signed,
+          coff_arm_reloc,
+          "ARM_DISP16",
+          true,
+          0x0000ffff,
+          0x0000ffff,
+          true),
+    HOWTO (ARM_DISP32,
+          0,
+          2,
+          32,
+          true,
+          0,
+          complain_overflow_signed,
+          coff_arm_reloc,
+          "ARM_DISP32",
+          true,
+          0xffffffff,
+          0xffffffff,
+          true),
+    HOWTO (ARM_26D,
+          2,
+          2,
+          24,
+          false,
+          0,
+          complain_overflow_dont,
+          aoutarm_fix_pcrel_26_done,
+          "ARM_26D",
+          true,
+          0x00ffffff,
+          0x0,
+          false),
+    /* 8 is unused */
+    EMPTY_HOWTO (-1),
+    HOWTO (ARM_NEG16,
+          0,
+          -1,
+          16,
+          false,
+          0,
+          complain_overflow_bitfield,
+          coff_arm_reloc,
+          "ARM_NEG16",
+          true,
+          0x0000ffff,
+          0x0000ffff,
+          false),
+    HOWTO (ARM_NEG32,
+          0,
+          -2,
+          32,
+          false,
+          0,
+          complain_overflow_bitfield,
+          coff_arm_reloc,
+          "ARM_NEG32",
+          true,
+          0xffffffff,
+          0xffffffff,
+          false),
+    HOWTO (ARM_RVA32,
+          0,
+          2,
+          32,
+          false,
+          0,
+          complain_overflow_bitfield,
+          coff_arm_reloc,
+          "ARM_RVA32",
+          true,
+          0xffffffff,
+          0xffffffff,
+          PCRELOFFSET),
+    HOWTO (ARM_THUMB9,
+          1,
+          1,
+          8,
+          true,
+          0,
+          complain_overflow_signed,
+          coff_thumb_pcrel_9 ,
+          "ARM_THUMB9",
+          false,
+          0x000000ff,
+          0x000000ff,
+          PCRELOFFSET),
+    HOWTO (ARM_THUMB12,
+          1,
+          1,
+          11,
+          true,
+          0,
+          complain_overflow_signed,
+          coff_thumb_pcrel_12 ,
+          "ARM_THUMB12",
+          false,
+          0x000007ff,
+          0x000007ff,
+          PCRELOFFSET),
+    HOWTO (ARM_THUMB23,
+          1,
+          2,
+          22,
+          true,
+          0,
+          complain_overflow_signed,
+          coff_thumb_pcrel_23 ,
+          "ARM_THUMB23",
+          false,
+          0x07ff07ff,
+          0x07ff07ff,
+          PCRELOFFSET)
 #endif /* not ARM_WINCE */
-};
+  };
 
 #define NUM_RELOCS NUM_ELEM (aoutarm_std_reloc_howto)
 
 #ifdef COFF_WITH_PE
+static boolean in_reloc_p PARAMS ((bfd *, reloc_howto_type *));
 /* Return true if this relocation should
    appear in the output .reloc section.  */
 
@@ -568,6 +593,7 @@ coff_arm_rtype_to_howto (abfd, sec, rel, h, sym, addendp)
 
   return howto;
 }
+
 /* Used by the assembler.  */
 
 static bfd_reloc_status_type
@@ -603,7 +629,7 @@ aoutarm_fix_pcrel_26 (abfd, reloc_entry, symbol, data, input_section,
   long target = bfd_get_32 (abfd, (bfd_byte *) data + addr);
   bfd_reloc_status_type flag = bfd_reloc_ok;
 
-  /* If this is an undefined symbol, return error */
+  /* If this is an undefined symbol, return error */
   if (symbol->section == &bfd_und_section
       && (symbol->flags & BSF_WEAK) == 0)
     return output_bfd ? bfd_reloc_continue : bfd_reloc_undefined;
@@ -615,7 +641,7 @@ aoutarm_fix_pcrel_26 (abfd, reloc_entry, symbol, data, input_section,
     return bfd_reloc_continue;
 
   relocation = (target & 0x00ffffff) << 2;
-  relocation = (relocation ^ 0x02000000) - 0x02000000; /* Sign extend */
+  relocation = (relocation ^ 0x02000000) - 0x02000000; /* Sign extend */
   relocation += symbol->value;
   relocation += symbol->section->output_section->vma;
   relocation += symbol->section->output_offset;
@@ -627,18 +653,18 @@ aoutarm_fix_pcrel_26 (abfd, reloc_entry, symbol, data, input_section,
   if (relocation & 3)
     return bfd_reloc_overflow;
 
-  /* Check for overflow */
+  /* Check for overflow */
   if (relocation & 0x02000000)
     {
       if ((relocation & ~ (bfd_vma) 0x03ffffff) != ~ (bfd_vma) 0x03ffffff)
        flag = bfd_reloc_overflow;
     }
-  else if (relocation & ~0x03ffffff)
+  else if (relocation & ~(bfd_vma) 0x03ffffff)
     flag = bfd_reloc_overflow;
 
   target &= ~0x00ffffff;
   target |= (relocation >> 2) & 0x00ffffff;
-  bfd_put_32 (abfd, target, (bfd_byte *) data + addr);
+  bfd_put_32 (abfd, (bfd_vma) target, (bfd_byte *) data + addr);
 
   /* Now the ARM magic... Change the reloc type so that it is marked as done.
      Strictly this is only necessary if we are doing a partial relocation.  */
@@ -694,7 +720,7 @@ coff_thumb_pcrel_common (abfd, reloc_entry, symbol, data, input_section,
       abort ();
     }
 
-  /* If this is an undefined symbol, return error */
+  /* If this is an undefined symbol, return error */
   if (symbol->section == &bfd_und_section
       && (symbol->flags & BSF_WEAK) == 0)
     return output_bfd ? bfd_reloc_continue : bfd_reloc_undefined;
@@ -723,7 +749,7 @@ coff_thumb_pcrel_common (abfd, reloc_entry, symbol, data, input_section,
       abort ();
     }
 
-  relocation = (relocation ^ signbit) - signbit; /* Sign extend */
+  relocation = (relocation ^ signbit) - signbit; /* Sign extend */
   relocation += symbol->value;
   relocation += symbol->section->output_section->vma;
   relocation += symbol->section->output_offset;
@@ -735,7 +761,7 @@ coff_thumb_pcrel_common (abfd, reloc_entry, symbol, data, input_section,
   if (relocation & 1)
     return bfd_reloc_overflow;
 
-  /* Check for overflow */
+  /* Check for overflow */
   if (relocation & signbit)
     {
       if ((relocation & ~offmsk) != ~offmsk)
@@ -754,22 +780,24 @@ coff_thumb_pcrel_common (abfd, reloc_entry, symbol, data, input_section,
 
    case b23:
      if (bfd_big_endian (abfd))
-       target |= ((relocation & 0xfff) >> 1)  | ((relocation << 4)  & 0x07ff0000);
+       target |= (((relocation & 0xfff) >> 1)
+                 | ((relocation << 4)  & 0x07ff0000));
      else
-       target |= ((relocation & 0xffe) << 15) | ((relocation >> 12) & 0x7ff);
+       target |= (((relocation & 0xffe) << 15)
+                 | ((relocation >> 12) & 0x7ff));
      break;
 
    default:
      abort ();
    }
 
-  bfd_put_32 (abfd, target, (bfd_byte *) data + addr);
+  bfd_put_32 (abfd, (bfd_vma) target, (bfd_byte *) data + addr);
 
   /* Now the ARM magic... Change the reloc type so that it is marked as done.
      Strictly this is only necessary if we are doing a partial relocation.  */
   reloc_entry->howto = & aoutarm_std_reloc_howto [ARM_26D];
 
-  /* TODO: We should possibly have DONE entries for the THUMB PCREL relocations */
+  /* TODO: We should possibly have DONE entries for the THUMB PCREL relocations */
   return flag;
 }
 
@@ -785,7 +813,8 @@ coff_thumb_pcrel_23 (abfd, reloc_entry, symbol, data, input_section,
      char **error_message;
 {
   return coff_thumb_pcrel_common (abfd, reloc_entry, symbol, data,
-                                  input_section, output_bfd, error_message, b23);
+                                  input_section, output_bfd, error_message,
+                                 b23);
 }
 
 static bfd_reloc_status_type
@@ -800,7 +829,8 @@ coff_thumb_pcrel_12 (abfd, reloc_entry, symbol, data, input_section,
      char **error_message;
 {
   return coff_thumb_pcrel_common (abfd, reloc_entry, symbol, data,
-                                  input_section, output_bfd, error_message, b12);
+                                  input_section, output_bfd, error_message,
+                                 b12);
 }
 
 static bfd_reloc_status_type
@@ -815,10 +845,11 @@ coff_thumb_pcrel_9 (abfd, reloc_entry, symbol, data, input_section,
      char **error_message;
 {
   return coff_thumb_pcrel_common (abfd, reloc_entry, symbol, data,
-                                  input_section, output_bfd, error_message, b9);
+                                  input_section, output_bfd, error_message,
+                                 b9);
 }
 
-static CONST struct reloc_howto_struct *
+static const struct reloc_howto_struct *
 coff_arm_reloc_type_lookup (abfd, code)
       bfd * abfd;
       bfd_reloc_code_real_type code;
@@ -831,7 +862,8 @@ coff_arm_reloc_type_lookup (abfd, code)
       case 32:
         code = BFD_RELOC_32;
         break;
-      default: return (CONST struct reloc_howto_struct *) 0;
+      default:
+       return (const struct reloc_howto_struct *) 0;
       }
 
   switch (code)
@@ -856,7 +888,7 @@ coff_arm_reloc_type_lookup (abfd, code)
       ASTD (BFD_RELOC_THUMB_PCREL_BRANCH23, ARM_THUMB23);
       ASTD (BFD_RELOC_THUMB_PCREL_BLX,      ARM_THUMB23);
 #endif
-    default: return (CONST struct reloc_howto_struct *) 0;
+    default: return (const struct reloc_howto_struct *) 0;
     }
 }
 
@@ -868,25 +900,30 @@ coff_arm_reloc_type_lookup (abfd, code)
 #define BADMAG(x) ARMBADMAG(x)
 #define ARM 1                  /* Customize coffcode.h */
 
+/* Make sure that the 'r_offset' field is copied properly
+   so that identical binaries will compare the same.  */
+#define SWAP_IN_RELOC_OFFSET   H_GET_32
+#define SWAP_OUT_RELOC_OFFSET  H_PUT_32
+
 /* 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 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 Thumb-to-ARM glue.  */
+    bfd_size_type              thumb_glue_size;
 
-  /* The size in bytes of the section containg the ARM-to-Thumb glue.  */
-  long int                     arm_glue_size;
+    /* The size in bytes of the section containg the ARM-to-Thumb glue.  */
+    bfd_size_type              arm_glue_size;
 
-  /* An arbitary input BFD chosen to hold the glue sections.  */
-  bfd *                                bfd_of_glue_owner;
+    /* An arbitary input BFD chosen to hold the glue sections.  */
+    bfd *                      bfd_of_glue_owner;
 
-  /* Support interworking with old, non-interworking aware ARM code.  */
-  int                          support_old_code;
+    /* Support interworking with old, non-interworking aware ARM code.  */
+    int                        support_old_code;
 };
 
 /* Get the ARM coff linker hash table from a link_info structure.  */
@@ -900,9 +937,9 @@ coff_arm_link_hash_table_create (abfd)
      bfd * abfd;
 {
   struct coff_arm_link_hash_table * ret;
+  bfd_size_type amt = sizeof (struct coff_arm_link_hash_table);
 
-  ret = ((struct coff_arm_link_hash_table *)
-        bfd_alloc (abfd, sizeof (struct coff_arm_link_hash_table)));
+  ret = (struct coff_arm_link_hash_table *) bfd_alloc (abfd, amt);
   if (ret == (struct coff_arm_link_hash_table *) NULL)
     return NULL;
 
@@ -932,9 +969,9 @@ arm_emit_base_file_entry (info, output_bfd, input_section, reloc_offset)
                 + input_section->output_offset
                   + input_section->output_section->vma;
 
-  if (coff_data(output_bfd)->pe)
-     addr -= pe_data(output_bfd)->pe_opthdr.ImageBase;
-  fwrite (&addr, 1, sizeof (addr), (FILE *) info->base_file);
+  if (coff_data (output_bfd)->pe)
+     addr -= pe_data (output_bfd)->pe_opthdr.ImageBase;
+  fwrite (& addr, 1, sizeof (addr), (FILE *) info->base_file);
 
 }
 \f
@@ -964,7 +1001,7 @@ arm_emit_base_file_entry (info, output_bfd, input_section, reloc_offset)
    moves the computed address into the PC, so it must be the second one
    in the sequence.  The problem, however is that whilst little endian code
    stores the instructions in HI then LOW order, big endian code does the
-   reverse.  nickc@cygnus.com  */
+   reverse.  nickc@cygnus.com.  */
 
 #define LOW_HI_ORDER 0xF800F000
 #define HI_LOW_ORDER 0xF000F800
@@ -988,24 +1025,24 @@ insert_thumb_branch (br_insn, rel_off)
   else if ((br_insn & HI_LOW_ORDER) == HI_LOW_ORDER)
     br_insn = HI_LOW_ORDER | (high_bits << 16) | low_bits;
   else
-    abort (); /* error - not a valid branch instruction form */
-
-  /* FIXME: abort is probably not the right call. krk@cygnus.com */
+    /* FIXME: the BFD library should never abort except for internal errors
+       - it should return an error status.  */
+    abort (); /* Error - not a valid branch instruction form.  */
 
   return br_insn;
 }
 \f
 static struct coff_link_hash_entry *
 find_thumb_glue (info, name, input_bfd)
-     struct bfd_link_info * info;
-     CONST char *           name;
-     bfd *                  input_bfd;
+     struct bfd_link_info *info;
+     const char *name;
+     bfd *input_bfd;
 {
-  char *                        tmp_name;
-  struct coff_link_hash_entry * myh;
+  char *tmp_name;
+  struct coff_link_hash_entry *myh;
+  bfd_size_type amt = strlen (name) + strlen (THUMB2ARM_GLUE_ENTRY_NAME) + 1;
 
-  tmp_name = ((char *)
-        bfd_malloc (strlen (name) + strlen (THUMB2ARM_GLUE_ENTRY_NAME) + 1));
+  tmp_name = (char *) bfd_malloc (amt);
 
   BFD_ASSERT (tmp_name);
 
@@ -1017,7 +1054,7 @@ find_thumb_glue (info, name, input_bfd)
   if (myh == NULL)
     /* xgettext:c-format */
     _bfd_error_handler (_("%s: unable to find THUMB glue '%s' for `%s'"),
-                       bfd_get_filename (input_bfd), tmp_name, name);
+                       bfd_archive_filename (input_bfd), tmp_name, name);
 
   free (tmp_name);
 
@@ -1026,15 +1063,15 @@ find_thumb_glue (info, name, input_bfd)
 
 static struct coff_link_hash_entry *
 find_arm_glue (info, name, input_bfd)
-     struct bfd_link_info * info;
-     CONST char *           name;
-     bfd *                  input_bfd;
+     struct bfd_link_info *info;
+     const char *name;
+     bfd *input_bfd;
 {
-  char *                        tmp_name;
+  char *tmp_name;
   struct coff_link_hash_entry * myh;
+  bfd_size_type amt = strlen (name) + strlen (ARM2THUMB_GLUE_ENTRY_NAME) + 1;
 
-  tmp_name = ((char *)
-             bfd_malloc (strlen (name) + strlen (ARM2THUMB_GLUE_ENTRY_NAME) + 1));
+  tmp_name = (char *) bfd_malloc (amt);
 
   BFD_ASSERT (tmp_name);
 
@@ -1046,7 +1083,7 @@ find_arm_glue (info, name, input_bfd)
   if (myh == NULL)
     /* xgettext:c-format */
     _bfd_error_handler (_("%s: unable to find ARM glue '%s' for `%s'"),
-                       bfd_get_filename (input_bfd), tmp_name, name);
+                       bfd_archive_filename (input_bfd), tmp_name, name);
 
   free (tmp_name);
 
@@ -1208,7 +1245,8 @@ coff_arm_relocate_section (output_bfd, info, input_bfd, input_section,
           && info->relocateable
           && (h->root.type == bfd_link_hash_defined
              || h->root.type == bfd_link_hash_defweak)
-          && h->root.u.def.section->output_section == input_section->output_section)
+          && (h->root.u.def.section->output_section
+             == input_section->output_section))
         {
           static reloc_howto_type fake_arm26_reloc =
            HOWTO (ARM_26,
@@ -1246,10 +1284,10 @@ coff_arm_relocate_section (output_bfd, info, input_bfd, input_section,
             continue;
          /* FIXME - it is not clear which targets need this next test
             and which do not.  It is known that it is needed for the
-            VXworks target (hence the #ifdef), but it is also known
-            that it was supressed for other (arm) targets.  This ought
-            to be sorted out one day.  */
-#ifdef VXWORKS
+            VxWorks and EPOC-PE targets, but it is also known that it
+            was supressed for other ARM targets.  This ought to be
+            sorted out one day.  */
+#ifdef ARM_COFF_BUGFIX
          /* We must not ignore the symbol value.  If the symbol is
             within the same section, the relocation should have already
             been fixed, but if it is not, we'll be handed a reloc into
@@ -1306,9 +1344,9 @@ coff_arm_relocate_section (output_bfd, info, input_bfd, input_section,
                   if (   h->class == C_THUMBSTATFUNC
                      || h->class == C_THUMBEXTFUNC)
                    {
-                     /* Arm code calling a Thumb function */
+                     /* Arm code calling a Thumb function */
                      unsigned long int                 tmp;
-                     long int                          my_offset;
+                     bfd_vma                           my_offset;
                      asection *                        s;
                      long int                          ret_offset;
                      struct coff_link_hash_entry *     myh;
@@ -1340,20 +1378,20 @@ coff_arm_relocate_section (output_bfd, info, input_bfd, input_section,
                              _bfd_error_handler
                                /* xgettext:c-format */
                                (_("%s(%s): warning: interworking not enabled."),
-                                bfd_get_filename (h_sec->owner), name);
+                                bfd_archive_filename (h_sec->owner), name);
                              _bfd_error_handler
                                /* xgettext:c-format */
                                (_("  first occurrence: %s: arm call to thumb"),
-                                bfd_get_filename (input_bfd));
+                                bfd_archive_filename (input_bfd));
                            }
 
                          --my_offset;
                          myh->root.u.def.value = my_offset;
 
-                         bfd_put_32 (output_bfd, a2t1_ldr_insn,
+                         bfd_put_32 (output_bfd, (bfd_vma) a2t1_ldr_insn,
                                      s->contents + my_offset);
 
-                         bfd_put_32 (output_bfd, a2t2_bx_r12_insn,
+                         bfd_put_32 (output_bfd, (bfd_vma) a2t2_bx_r12_insn,
                                      s->contents + my_offset + 4);
 
                          /* It's a thumb address.  Add the low order bit.  */
@@ -1362,7 +1400,7 @@ coff_arm_relocate_section (output_bfd, info, input_bfd, input_section,
 
                           if (info->base_file)
                             arm_emit_base_file_entry (info, output_bfd, s,
-                                                            my_offset + 8);
+                                                      my_offset + 8);
 
                        }
 
@@ -1385,14 +1423,14 @@ 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, (bfd_vma) tmp,
+                                 contents + rel->r_vaddr - input_section->vma);
                      done = 1;
                    }
                 }
 
 #ifndef ARM_WINCE
-             /* Note: We used to check for ARM_THUMB9 and ARM_THUMB12 */
+             /* Note: We used to check for ARM_THUMB9 and ARM_THUMB12 */
               else if (howto->type == ARM_THUMB23)
                 {
                   if (   h->class == C_EXT
@@ -1401,7 +1439,7 @@ coff_arm_relocate_section (output_bfd, info, input_bfd, input_section,
                    {
                      /* Thumb code calling an ARM function */
                      asection *                         s = 0;
-                     long int                           my_offset;
+                     bfd_vma                            my_offset;
                      unsigned long int                  tmp;
                      long int                           ret_offset;
                      struct coff_link_hash_entry *      myh;
@@ -1435,11 +1473,11 @@ coff_arm_relocate_section (output_bfd, info, input_bfd, input_section,
                              _bfd_error_handler
                                /* xgettext:c-format */
                                (_("%s(%s): warning: interworking not enabled."),
-                                bfd_get_filename (h_sec->owner), name);
+                                bfd_archive_filename (h_sec->owner), name);
                              _bfd_error_handler
                                /* xgettext:c-format */
                                (_("  first occurrence: %s: thumb call to arm"),
-                                bfd_get_filename (input_bfd));
+                                bfd_archive_filename (input_bfd));
                              _bfd_error_handler
                                (_("  consider relinking with --support-old-code enabled"));
                            }
@@ -1449,22 +1487,22 @@ coff_arm_relocate_section (output_bfd, info, input_bfd, input_section,
 
                          if (globals->support_old_code)
                            {
-                             bfd_put_16 (output_bfd, t2a1_push_insn,
+                             bfd_put_16 (output_bfd, (bfd_vma) t2a1_push_insn,
                                          s->contents + my_offset);
 
-                             bfd_put_16 (output_bfd, t2a2_ldr_insn,
+                             bfd_put_16 (output_bfd, (bfd_vma) t2a2_ldr_insn,
                                          s->contents + my_offset + 2);
 
-                             bfd_put_16 (output_bfd, t2a3_mov_insn,
+                             bfd_put_16 (output_bfd, (bfd_vma) t2a3_mov_insn,
                                          s->contents + my_offset + 4);
 
-                             bfd_put_16 (output_bfd, t2a4_bx_insn,
+                             bfd_put_16 (output_bfd, (bfd_vma) t2a4_bx_insn,
                                          s->contents + my_offset + 6);
 
-                             bfd_put_32 (output_bfd, t2a5_pop_insn,
+                             bfd_put_32 (output_bfd, (bfd_vma) t2a5_pop_insn,
                                          s->contents + my_offset + 8);
 
-                             bfd_put_32 (output_bfd, t2a6_bx_insn,
+                             bfd_put_32 (output_bfd, (bfd_vma) t2a6_bx_insn,
                                          s->contents + my_offset + 12);
 
                              /* Store the address of the function in the last word of the stub.  */
@@ -1472,18 +1510,19 @@ coff_arm_relocate_section (output_bfd, info, input_bfd, input_section,
                                          s->contents + my_offset + 16);
 
                               if (info->base_file)
-                                arm_emit_base_file_entry (info, output_bfd, s, my_offset + 16);
+                                arm_emit_base_file_entry (info, output_bfd, s,
+                                                         my_offset + 16);
                            }
                          else
                            {
-                             bfd_put_16 (output_bfd, t2a1_bx_pc_insn,
+                             bfd_put_16 (output_bfd, (bfd_vma) t2a1_bx_pc_insn,
                                          s->contents + my_offset);
 
-                             bfd_put_16 (output_bfd, t2a2_noop_insn,
+                             bfd_put_16 (output_bfd, (bfd_vma) t2a2_noop_insn,
                                          s->contents + my_offset + 2);
 
                              ret_offset =
-                               ((bfd_signed_vma) h_val)        /* Address of destination of the stub */
+                               ((bfd_signed_vma) h_val)        /* Address of destination of the stub */
                                - ((bfd_signed_vma)
                                   (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.  */
@@ -1492,7 +1531,7 @@ coff_arm_relocate_section (output_bfd, info, input_bfd, input_section,
                                   + 8);                        /* ARM branches work from the pc of the instruction + 8.  */
 
                              bfd_put_32 (output_bfd,
-                                         t2a3_b_insn | ((ret_offset >> 2) & 0x00FFFFFF),
+                                         (bfd_vma) t2a3_b_insn | ((ret_offset >> 2) & 0x00FFFFFF),
                                          s->contents + my_offset + 4);
 
                            }
@@ -1513,9 +1552,9 @@ coff_arm_relocate_section (output_bfd, info, input_bfd, input_section,
                                        - input_section->vma);
 
                      bfd_put_32 (output_bfd,
-                                 insert_thumb_branch (tmp, ret_offset),
-                                 contents + rel->r_vaddr
-                                 - input_section->vma);
+                                 (bfd_vma) insert_thumb_branch (tmp,
+                                                                ret_offset),
+                                 contents + rel->r_vaddr - input_section->vma);
 
                      done = 1;
                     }
@@ -1555,7 +1594,8 @@ coff_arm_relocate_section (output_bfd, info, input_bfd, input_section,
        {
          /* Emit a reloc if the backend thinks it needs it.  */
          if (sym && pe_data(output_bfd)->in_reloc_p(output_bfd, howto))
-            arm_emit_base_file_entry (info, output_bfd, input_section, rel->r_vaddr);
+            arm_emit_base_file_entry (info, output_bfd, input_section,
+                                     rel->r_vaddr);
        }
 
 #if 1 /* THUMBEXTENSION */
@@ -1730,7 +1770,7 @@ coff_arm_relocate_section (output_bfd, info, input_bfd, input_section,
        case bfd_reloc_outofrange:
          (*_bfd_error_handler)
            (_("%s: bad reloc address 0x%lx in section `%s'"),
-            bfd_get_filename (input_bfd),
+            bfd_archive_filename (input_bfd),
             (unsigned long) rel->r_vaddr,
             bfd_get_section_name (input_bfd, input_section));
          return false;
@@ -1787,10 +1827,10 @@ bfd_arm_allocate_interworking_sections (info)
 
       BFD_ASSERT (s != NULL);
 
-      foo = (bfd_byte *) bfd_alloc
-       (globals->bfd_of_glue_owner, globals->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);
+      memset (foo, test_char, (size_t) globals->arm_glue_size);
 #endif
 
       s->_raw_size = s->_cooked_size = globals->arm_glue_size;
@@ -1806,10 +1846,10 @@ bfd_arm_allocate_interworking_sections (info)
 
       BFD_ASSERT (s != NULL);
 
-      foo = (bfd_byte *) bfd_alloc
-       (globals->bfd_of_glue_owner, globals->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);
+      memset (foo, test_char, (size_t) globals->thumb_glue_size);
 #endif
 
       s->_raw_size = s->_cooked_size = globals->thumb_glue_size;
@@ -1829,6 +1869,8 @@ record_arm_to_thumb_glue (info, h)
   char *                            tmp_name;
   struct coff_link_hash_entry *     myh;
   struct coff_arm_link_hash_table * globals;
+  bfd_vma val;
+  bfd_size_type amt;
 
   globals = coff_arm_hash_table (info);
 
@@ -1840,8 +1882,8 @@ record_arm_to_thumb_glue (info, h)
 
   BFD_ASSERT (s != NULL);
 
-  tmp_name = ((char *)
-             bfd_malloc (strlen (name) + strlen (ARM2THUMB_GLUE_ENTRY_NAME) + 1));
+  amt = strlen (name) + strlen (ARM2THUMB_GLUE_ENTRY_NAME) + 1;
+  tmp_name = (char *) bfd_malloc (amt);
 
   BFD_ASSERT (tmp_name);
 
@@ -1860,10 +1902,9 @@ record_arm_to_thumb_glue (info, h)
      though the section isn't allocated yet, this is where we will be putting
      it.  */
 
+  val = globals->arm_glue_size + 1;
   bfd_coff_link_add_one_symbol (info, globals->bfd_of_glue_owner, tmp_name,
-                               BSF_GLOBAL,
-                               s, globals->arm_glue_size + 1,
-                               NULL, true, false,
+                               BSF_GLOBAL, s, val, NULL, true, false,
                                (struct bfd_link_hash_entry **) & myh);
 
   free (tmp_name);
@@ -1883,6 +1924,8 @@ record_thumb_to_arm_glue (info, h)
   char *                             tmp_name;
   struct coff_link_hash_entry *      myh;
   struct coff_arm_link_hash_table *  globals;
+  bfd_vma val;
+  bfd_size_type amt;
 
   globals = coff_arm_hash_table (info);
 
@@ -1894,7 +1937,8 @@ record_thumb_to_arm_glue (info, h)
 
   BFD_ASSERT (s != NULL);
 
-  tmp_name = (char *) bfd_malloc (strlen (name) + strlen (THUMB2ARM_GLUE_ENTRY_NAME) + 1);
+  amt = strlen (name) + strlen (THUMB2ARM_GLUE_ENTRY_NAME) + 1;
+  tmp_name = (char *) bfd_malloc (amt);
 
   BFD_ASSERT (tmp_name);
 
@@ -1909,10 +1953,10 @@ record_thumb_to_arm_glue (info, h)
       return; /* we've already seen this guy */
     }
 
+  val = globals->thumb_glue_size + 1;
   bfd_coff_link_add_one_symbol (info, globals->bfd_of_glue_owner, tmp_name,
-                               BSF_GLOBAL, s, globals->thumb_glue_size + 1,
-                               NULL, true, false,
-                               (struct bfd_link_hash_entry **) & myh);
+                               BSF_GLOBAL, s, val, NULL, true, false,
+                               (struct bfd_link_hash_entry **) &myh);
 
   /* If we mark it 'thumb', the disassembler will do a better job.  */
   myh->class = C_THUMBEXTFUNC;
@@ -1924,18 +1968,17 @@ record_thumb_to_arm_glue (info, h)
 #define CHANGE_TO_ARM "__%s_change_to_arm"
 #define BACK_FROM_ARM "__%s_back_from_arm"
 
-  tmp_name = (char *) bfd_malloc (strlen (name) + strlen (CHANGE_TO_ARM) + 1);
+  amt = strlen (name) + strlen (CHANGE_TO_ARM) + 1;
+  tmp_name = (char *) bfd_malloc (amt);
 
   BFD_ASSERT (tmp_name);
 
   sprintf (tmp_name, globals->support_old_code ? BACK_FROM_ARM : CHANGE_TO_ARM, name);
 
   myh = NULL;
-
+  val = globals->thumb_glue_size + (globals->support_old_code ? 8 : 4);
   bfd_coff_link_add_one_symbol (info, globals->bfd_of_glue_owner, tmp_name,
-                               BSF_LOCAL, s, globals->thumb_glue_size
-                               + (globals->support_old_code ? 8 : 4),
-                               NULL, true, false,
+                               BSF_LOCAL, s, val, NULL, true, false,
                                (struct bfd_link_hash_entry **) & myh);
 
   free (tmp_name);
@@ -2067,7 +2110,7 @@ bfd_arm_process_before_allocation (abfd, info, support_old_code)
          if (symndx >= obj_conv_table_size (abfd))
            {
              _bfd_error_handler (_("%s: illegal symbol index in reloc: %d"),
-                                 bfd_get_filename (abfd), symndx);
+                                 bfd_archive_filename (abfd), symndx);
              continue;
            }
 
@@ -2194,8 +2237,8 @@ coff_arm_merge_private_bfd_data (ibfd, obfd)
            {
              _bfd_error_handler
                /* xgettext: c-format */
-               (_("%s: ERROR: compiled for APCS-%d whereas target %s uses APCS-%d"),
-                bfd_get_filename (ibfd), APCS_26_FLAG (ibfd) ? 26 : 32,
+               (_("ERROR: %s is compiled for APCS-%d, whereas %s is compiled for APCS-%d"),
+                bfd_archive_filename (ibfd), APCS_26_FLAG (ibfd) ? 26 : 32,
                 bfd_get_filename (obfd), APCS_26_FLAG (obfd) ? 26 : 32
                 );
 
@@ -2209,12 +2252,12 @@ coff_arm_merge_private_bfd_data (ibfd, obfd)
 
              if (APCS_FLOAT_FLAG (ibfd))
                /* xgettext: c-format */
-               msg = _("%s: ERROR: passes floats in float registers whereas target %s uses integer registers");
+               msg = _("ERROR: %s passes floats in float registers, whereas %s passes them in integer registers");
              else
                /* xgettext: c-format */
-               msg = _("%s: ERROR: passes floats in integer registers whereas target %s uses float registers");
+               msg = _("ERROR: %s passes floats in integer registers, whereas %s passes them in float registers");
 
-             _bfd_error_handler (msg, bfd_get_filename (ibfd),
+             _bfd_error_handler (msg, bfd_archive_filename (ibfd),
                                  bfd_get_filename (obfd));
 
              bfd_set_error (bfd_error_wrong_format);
@@ -2227,11 +2270,11 @@ coff_arm_merge_private_bfd_data (ibfd, obfd)
 
              if (PIC_FLAG (ibfd))
                /* xgettext: c-format */
-               msg = _("%s: ERROR: compiled as position independent code, whereas target %s is absolute position");
+               msg = _("ERROR: %s is compiled as position independent code, whereas target %s is absolute position");
              else
                /* xgettext: c-format */
-               msg = _("%s: ERROR: compiled as absolute position code, whereas target %s is position independent");
-             _bfd_error_handler (msg, bfd_get_filename (ibfd),
+               msg = _("ERROR: %s is compiled as absolute position code, whereas target %s is position independent");
+             _bfd_error_handler (msg, bfd_archive_filename (ibfd),
                                  bfd_get_filename (obfd));
 
              bfd_set_error (bfd_error_wrong_format);
@@ -2259,12 +2302,12 @@ coff_arm_merge_private_bfd_data (ibfd, obfd)
 
              if (INTERWORK_FLAG (ibfd))
                /* xgettext: c-format */
-               msg = _("Warning: input file %s supports interworking, whereas %s does not.");
+               msg = _("Warning: %s supports interworking, whereas %s does not");
              else
                /* xgettext: c-format */
-               msg = _("Warning: input file %s does not support interworking, whereas %s does.");
+               msg = _("Warning: %s does not support interworking, whereas %s does");
 
-             _bfd_error_handler (msg, bfd_get_filename (ibfd),
+             _bfd_error_handler (msg, bfd_archive_filename (ibfd),
                                  bfd_get_filename (obfd));
            }
        }
@@ -2360,12 +2403,12 @@ _bfd_coff_arm_set_private_flags (abfd, flags)
     {
       if (flag)
        /* xgettext: c-format */
-       _bfd_error_handler (_("Warning: Not setting interworking flag of %s, since it has already been specified as non-interworking"),
-                           bfd_get_filename (abfd));
+       _bfd_error_handler (_("Warning: Not setting interworking flag of %s since it has already been specified as non-interworking"),
+                           bfd_archive_filename (abfd));
       else
        /* xgettext: c-format */
        _bfd_error_handler (_("Warning: Clearing the interworking flag of %s due to outside request"),
-                           bfd_get_filename (abfd));
+                           bfd_archive_filename (abfd));
       flag = 0;
     }
 
@@ -2423,9 +2466,10 @@ coff_arm_copy_private_bfd_data (src, dest)
              if (INTERWORK_FLAG (dest))
                {
                  /* xgettext:c-format */
-                 _bfd_error_handler (("Warning: Clearing the interworking bit of %s, because the non-interworking code in %s has been copied into it"),
+                 _bfd_error_handler (("\
+Warning: Clearing the interworking flag of %s because non-interworking code in %s has been linked with it"),
                                      bfd_get_filename (dest),
-                                     bfd_get_filename (src));
+                                     bfd_archive_filename (src));
                }
 
              SET_INTERWORK_FLAG (dest, 0);
@@ -2471,7 +2515,7 @@ coff_arm_is_local_label_name (abfd, name)
 
   if (LOCAL_LABEL_PREFIX[0] != 0)
     {
-      int len = strlen (LOCAL_LABEL_PREFIX);
+      size_t len = strlen (LOCAL_LABEL_PREFIX);
 
       if (strncmp (name, LOCAL_LABEL_PREFIX, len) != 0)
        return false;
This page took 0.050333 seconds and 4 git commands to generate.