* Makefile.am: Run "make dep-am".
[deliverable/binutils-gdb.git] / bfd / coff-arm.c
index af129eb68230cd9f1745d648b56f858787be67a9..ca08131d1cb46dc0aba27885b39889b303a1c168 100644 (file)
@@ -1,5 +1,7 @@
 /* BFD back-end for ARM COFF files.
-   Copyright 1990, 91, 92, 93, 94, 95, 96, 1997 Free Software Foundation, Inc.
+   Copyright 1990, 1991, 1992, 1993, 1994, 1995, 1996, 1997, 1998, 1999,
+   2000, 2001
+   Free Software Foundation, Inc.
    Written by Cygnus Support.
 
 This file is part of BFD, the Binary File Descriptor library.
@@ -32,76 +34,133 @@ Foundation, Inc., 59 Temple Place - Suite 330, Boston, MA 02111-1307, USA.  */
 
 #include "libcoff.h"
 
-#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))
+/* 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)
 
-static boolean coff_arm_relocate_section
-  PARAMS ((bfd *, struct bfd_link_info *, bfd *, asection *, bfd_byte *,
-           struct internal_reloc *, struct internal_syment *, asection **));
+#define PIC_FLAG(abfd) \
+  (coff_data (abfd)->flags & F_PIC)
 
-static bfd_reloc_status_type
-aoutarm_fix_pcrel_26_done PARAMS ((bfd *, arelent *, asymbol *, PTR,
-                                 asection *, bfd *, char **));
+#define APCS_SET(abfd) \
+  (coff_data (abfd)->flags & F_APCS_SET)
 
-static bfd_reloc_status_type
-aoutarm_fix_pcrel_26 PARAMS ((bfd *, arelent *, asymbol *, PTR,
-                            asection *, bfd *, char **));
+#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)
 
-static bfd_reloc_status_type
-coff_thumb_pcrel_23 PARAMS ((bfd *, arelent *, asymbol *, PTR,
-                             asection *, bfd *, char **));
+#define INTERWORK_FLAG(abfd) \
+  (coff_data (abfd)->flags & F_INTERWORK)
 
-static bfd_reloc_status_type
-coff_thumb_pcrel_12 PARAMS ((bfd *, arelent *, asymbol *, PTR,
-                             asection *, bfd *, char **));
+#define INTERWORK_SET(abfd) \
+  (coff_data (abfd)->flags & F_INTERWORK_SET)
 
-static bfd_reloc_status_type
-coff_thumb_pcrel_9 PARAMS ((bfd *, arelent *, asymbol *, PTR,
-                            asection *, bfd *, char **));
+#define SET_INTERWORK_FLAG(abfd, flg) \
+  do                                                                   \
+    {                                                                  \
+      coff_data (abfd)->flags &= ~F_INTERWORK;                         \
+      coff_data (abfd)->flags |= (flg) | F_INTERWORK_SET;              \
+    }                                                                  \
+  while (0)
 
-static bfd_reloc_status_type
-coff_arm_reloc PARAMS ((bfd *, arelent *, asymbol *, PTR, asection *, bfd *,
-                       char **));
+#ifndef NUM_ELEM
+#define NUM_ELEM(a) ((sizeof (a)) / sizeof ((a)[0]))
+#endif
 
-static boolean
-coff_arm_adjust_symndx PARAMS ((bfd *, struct bfd_link_info *, bfd *,
-                               asection *, struct internal_reloc *,
-                               boolean *));
+typedef enum {bunknown, b9, b12, b23} thumb_pcrel_branchtype;
+/* some typedefs for holding instructions */
+typedef unsigned long int insn32;
+typedef unsigned short int insn16;
+
+     /* Forward declarations for stupid compilers.  */
+static boolean coff_arm_relocate_section
+  PARAMS ((bfd *, struct bfd_link_info *, bfd *, asection *, bfd_byte *,
+           struct internal_reloc *, struct internal_syment *, asection **));
+static bfd_reloc_status_type aoutarm_fix_pcrel_26_done
+  PARAMS ((bfd *, arelent *, asymbol *, PTR, asection *, bfd *, char **));
+static bfd_reloc_status_type aoutarm_fix_pcrel_26
+  PARAMS ((bfd *, arelent *, asymbol *, PTR, asection *, bfd *, char **));
+static bfd_reloc_status_type coff_thumb_pcrel_23
+  PARAMS ((bfd *, arelent *, asymbol *, PTR, asection *, bfd *, char **));
+static bfd_reloc_status_type coff_thumb_pcrel_12
+  PARAMS ((bfd *, arelent *, asymbol *, PTR, asection *, bfd *, char **));
+static bfd_reloc_status_type coff_thumb_pcrel_9
+  PARAMS ((bfd *, arelent *, asymbol *, PTR, asection *, bfd *, char **));
+static bfd_reloc_status_type coff_arm_reloc
+  PARAMS ((bfd *, arelent *, asymbol *, PTR, asection *, bfd *, char **));
+static boolean coff_arm_adjust_symndx
+  PARAMS ((bfd *, struct bfd_link_info *, bfd *,
+          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 *));
+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
+  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 *));
+static struct coff_link_hash_entry * find_arm_glue
+  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 *));
+static void record_thumb_to_arm_glue
+  PARAMS ((struct bfd_link_info *, struct coff_link_hash_entry *));
+#endif
+static boolean coff_arm_merge_private_bfd_data
+  PARAMS ((bfd *, bfd *));
+static boolean coff_arm_print_private_bfd_data
+  PARAMS ((bfd *, PTR));
+static boolean _bfd_coff_arm_set_private_flags
+  PARAMS ((bfd *, flagword));
+static boolean coff_arm_copy_private_bfd_data
+  PARAMS ((bfd *, bfd *));
+static boolean coff_arm_is_local_label_name
+  PARAMS ((bfd *, const char *));
+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"
 
 #define ARM2THUMB_GLUE_SECTION_NAME ".glue_7"
 #define ARM2THUMB_GLUE_ENTRY_NAME   "__%s_from_arm"
 
-/* Used by the assembler. */
+/* Used by the assembler.  */
+
 static bfd_reloc_status_type
 coff_arm_reloc (abfd, reloc_entry, symbol, data, input_section, output_bfd,
                 error_message)
      bfd *abfd;
      arelent *reloc_entry;
-     asymbol *symbol;
+     asymbol *symbol ATTRIBUTE_UNUSED;
      PTR data;
-     asection *input_section;
+     asection *input_section ATTRIBUTE_UNUSED;
      bfd *output_bfd;
-     char **error_message;
+     char **error_message ATTRIBUTE_UNUSED;
 {
   symvalue diff;
   if (output_bfd == (bfd *) NULL)
@@ -109,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)
       {
@@ -131,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;
 
@@ -139,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;
 
@@ -152,15 +212,19 @@ coff_arm_reloc (abfd, reloc_entry, symbol, data, input_section, output_bfd,
   return bfd_reloc_continue;
 }
 
+/* If USER_LABEL_PREFIX is defined as "_" (see coff_arm_is_local_label_name()
+   in this file), then TARGET_UNDERSCORE should be defined, otherwise it
+   should not.  */
+#ifndef TARGET_UNDERSCORE
 #define TARGET_UNDERSCORE '_'
+#endif
 
 #ifndef PCRELOFFSET
 #define PCRELOFFSET true
 #endif
 
 /* These most certainly belong somewhere else. Just had to get rid of
-   the manifest constants in the code. */
-
+   the manifest constants in the code.  */
 #define ARM_8        0
 #define ARM_16       1
 #define ARM_32       2
@@ -177,281 +241,377 @@ coff_arm_reloc (abfd, reloc_entry, symbol, data, input_section, output_bfd,
 #define ARM_THUMB12 13
 #define ARM_THUMB23 14
 
-static reloc_howto_type aoutarm_std_reloc_howto[] = 
-{
-  /* type              rs size bsz  pcrel bitpos ovrf                     sf name     part_inpl readmask  setmask    pcdone */
-  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_signed,
-       aoutarm_fix_pcrel_26_done, 
-       "ARM_26D",
-       true,
-       0x00ffffff,
-       0x0,
-       false),
-  /* 8 is unused */
-  {-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),
-};
+#ifdef ARM_WINCE
+#undef  ARM_32
+#undef  ARM_RVA32
+#undef  ARM_26
+#undef  ARM_THUMB12
+#undef  ARM_26D
+
+#define ARM_32       1
+#define ARM_RVA32    2
+#define ARM_26      3
+#define ARM_THUMB12  4
+#define ARM_26D      5
+#define ARM_SECTION  14
+#define ARM_SECREL   15
+#endif
+
+static reloc_howto_type aoutarm_std_reloc_howto[] =
+  {
+#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),
+#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)
+#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. */
+   appear in the output .reloc section.  */
 
-static boolean in_reloc_p (abfd, howto)
-     bfd * abfd;
-     reloc_howto_type *howto;
+static boolean
+in_reloc_p (abfd, howto)
+     bfd * abfd ATTRIBUTE_UNUSED;
+     reloc_howto_type * howto;
 {
-  return !howto->pc_relative && howto->type != 11;
-}     
+  return !howto->pc_relative && howto->type != ARM_RVA32;
+}
 #endif
 
-
-#define RTYPE2HOWTO(cache_ptr, dst) \
-           (cache_ptr)->howto = aoutarm_std_reloc_howto + (dst)->r_type;
+#define RTYPE2HOWTO(cache_ptr, dst)            \
+  (cache_ptr)->howto =                         \
+    (dst)->r_type < NUM_RELOCS                 \
+    ? aoutarm_std_reloc_howto + (dst)->r_type  \
+    : NULL
 
 #define coff_rtype_to_howto coff_arm_rtype_to_howto
 
 static reloc_howto_type *
 coff_arm_rtype_to_howto (abfd, sec, rel, h, sym, addendp)
-     bfd *abfd;
+     bfd *abfd ATTRIBUTE_UNUSED;
      asection *sec;
      struct internal_reloc *rel;
-     struct coff_link_hash_entry *h;
-     struct internal_syment *sym;
+     struct coff_link_hash_entry *h ATTRIBUTE_UNUSED;
+     struct internal_syment *sym ATTRIBUTE_UNUSED;
      bfd_vma *addendp;
 {
-  reloc_howto_type *howto;
+  reloc_howto_type * howto;
+
+  if (rel->r_type >= NUM_RELOCS)
+    return NULL;
 
   howto = aoutarm_std_reloc_howto + rel->r_type;
 
   if (rel->r_type == ARM_RVA32)
-    {
-      *addendp -= pe_data(sec->output_section->owner)->pe_opthdr.ImageBase;
-    }
-
-  /* The relocation_section function will skip pcrel_offset relocs
-     when doing a relocateable link.  However, we want to convert
-     ARM26 to ARM26D relocs if possible.  We return a fake howto in
-     this case without pcrel_offset set, and adjust the addend to
-     compensate.  */
-  if (rel->r_type == ARM_26
-      && h != NULL
-      && (h->root.type == bfd_link_hash_defined
-         || h->root.type == bfd_link_hash_defweak)
-      && h->root.u.def.section->output_section == sec->output_section)
-    {
-      static reloc_howto_type fake_arm26_reloc = 
-       HOWTO (ARM_26,
-              2,
-              2,
-              24,
-              true,
-              0,
-              complain_overflow_signed,
-              aoutarm_fix_pcrel_26 ,
-              "ARM_26",
-              false,
-              0x00ffffff,
-              0x00ffffff, 
-              false);
-
-      *addendp -= rel->r_vaddr - sec->vma;
-      return & fake_arm26_reloc;
-    }
+    *addendp -= pe_data (sec->output_section->owner)->pe_opthdr.ImageBase;
 
   return howto;
-
 }
-/* Used by the assembler. */
+
+/* Used by the assembler.  */
 
 static bfd_reloc_status_type
 aoutarm_fix_pcrel_26_done (abfd, reloc_entry, symbol, data, input_section,
                          output_bfd, error_message)
-     bfd *abfd;
-     arelent *reloc_entry;
-     asymbol *symbol;
-     PTR data;
-     asection *input_section;
-     bfd *output_bfd;
-     char **error_message;
+     bfd *abfd ATTRIBUTE_UNUSED;
+     arelent *reloc_entry ATTRIBUTE_UNUSED;
+     asymbol *symbol ATTRIBUTE_UNUSED;
+     PTR data ATTRIBUTE_UNUSED;
+     asection *input_section ATTRIBUTE_UNUSED;
+     bfd *output_bfd ATTRIBUTE_UNUSED;
+     char **error_message ATTRIBUTE_UNUSED;
 {
   /* This is dead simple at present.  */
   return bfd_reloc_ok;
 }
 
-/* Used by the assembler. */
+/* Used by the assembler.  */
 
 static bfd_reloc_status_type
 aoutarm_fix_pcrel_26 (abfd, reloc_entry, symbol, data, input_section,
@@ -462,14 +622,14 @@ aoutarm_fix_pcrel_26 (abfd, reloc_entry, symbol, data, input_section,
      PTR data;
      asection *input_section;
      bfd *output_bfd;
-     char **error_message;
+     char **error_message ATTRIBUTE_UNUSED;
 {
   bfd_vma relocation;
   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;
-  
-  /* 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;
@@ -481,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;
@@ -489,22 +649,22 @@ aoutarm_fix_pcrel_26 (abfd, reloc_entry, symbol, data, input_section,
   relocation -= input_section->output_section->vma;
   relocation -= input_section->output_offset;
   relocation -= addr;
-  
+
   if (relocation & 3)
     return bfd_reloc_overflow;
 
-  /* Check for overflow */
+  /* 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)
+  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.  */
@@ -513,8 +673,6 @@ aoutarm_fix_pcrel_26 (abfd, reloc_entry, symbol, data, input_section,
   return flag;
 }
 
-typedef enum {bunknown, b9, b12, b23} thumb_pcrel_branchtype;
-
 static bfd_reloc_status_type
 coff_thumb_pcrel_common (abfd, reloc_entry, symbol, data, input_section,
                     output_bfd, error_message, btype)
@@ -524,10 +682,10 @@ coff_thumb_pcrel_common (abfd, reloc_entry, symbol, data, input_section,
      PTR data;
      asection *input_section;
      bfd *output_bfd;
-     char **error_message;
+     char **error_message ATTRIBUTE_UNUSED;
      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;
@@ -561,8 +719,8 @@ coff_thumb_pcrel_common (abfd, reloc_entry, symbol, data, input_section,
     default:
       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;
@@ -586,9 +744,12 @@ 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 */
+  relocation = (relocation ^ signbit) - signbit; /* Sign extend */
   relocation += symbol->value;
   relocation += symbol->section->output_section->vma;
   relocation += symbol->section->output_offset;
@@ -600,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)
@@ -619,19 +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;
 }
 
@@ -647,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
@@ -662,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
@@ -677,31 +845,40 @@ 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 *
-arm_reloc_type_lookup(abfd,code)
-      bfd *abfd;
+static const struct reloc_howto_struct *
+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]
+#define ASTD(i,j)       case i: return aoutarm_std_reloc_howto + j
+
   if (code == BFD_RELOC_CTOR)
     switch (bfd_get_arch_info (abfd)->bits_per_address)
       {
       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)
     {
+#ifdef ARM_WINCE
+      ASTD (BFD_RELOC_32,                   ARM_32);
+      ASTD (BFD_RELOC_RVA,                  ARM_RVA32);
+      ASTD (BFD_RELOC_ARM_PCREL_BRANCH,     ARM_26);
+      ASTD (BFD_RELOC_THUMB_PCREL_BRANCH12, ARM_THUMB12);
+#else
       ASTD (BFD_RELOC_8,                    ARM_8);
       ASTD (BFD_RELOC_16,                   ARM_16);
       ASTD (BFD_RELOC_32,                   ARM_32);
       ASTD (BFD_RELOC_ARM_PCREL_BRANCH,     ARM_26);
+      ASTD (BFD_RELOC_ARM_PCREL_BLX,        ARM_26);
       ASTD (BFD_RELOC_8_PCREL,              ARM_DISP8);
       ASTD (BFD_RELOC_16_PCREL,             ARM_DISP16);
       ASTD (BFD_RELOC_32_PCREL,             ARM_DISP32);
@@ -709,13 +886,12 @@ arm_reloc_type_lookup(abfd,code)
       ASTD (BFD_RELOC_THUMB_PCREL_BRANCH9,  ARM_THUMB9);
       ASTD (BFD_RELOC_THUMB_PCREL_BRANCH12, ARM_THUMB12);
       ASTD (BFD_RELOC_THUMB_PCREL_BRANCH23, ARM_THUMB23);
-    default: return (CONST struct reloc_howto_struct *) 0;
+      ASTD (BFD_RELOC_THUMB_PCREL_BLX,      ARM_THUMB23);
+#endif
+    default: return (const struct reloc_howto_struct *) 0;
     }
 }
 
-
-#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,47 +900,88 @@ 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, plus a BFD to hang the glue sections onto.
-   Note:  These variable should not be made static, since in a *-pe
-   build there are two versions of this file compiled, one for pe
-   objects and one for pei objects, and they want to share these
-   variables.  */
-#if defined COFF_IMAGE_WITH_PE
-extern long int global_thumb_glue_size;
-#else
-long int global_thumb_glue_size = 0;
-#endif
+/* 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;
 
-#if defined COFF_IMAGE_WITH_PE
-extern long int global_arm_glue_size;
-#else
-long int global_arm_glue_size = 0;
-#endif
+    /* The size in bytes of the section containg the Thumb-to-ARM glue.  */
+    bfd_size_type              thumb_glue_size;
 
-#if defined COFF_IMAGE_WITH_PE 
-extern bfd * bfd_of_glue_owner;
-#else
-bfd * bfd_of_glue_owner = NULL;
-#endif
+    /* The size in bytes of the section containg the ARM-to-Thumb glue.  */
+    bfd_size_type              arm_glue_size;
 
-/* some typedefs for holding instructions */
-typedef unsigned long int insn32;
-typedef unsigned short int insn16;
+    /* 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;
+};
+
+/* 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;
+  bfd_size_type amt = 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;
+
+  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;
+    }
+
+  ret->thumb_glue_size   = 0;
+  ret->arm_glue_size     = 0;
+  ret->bfd_of_glue_owner = NULL;
+
+  return & ret->root.root;
+}
+
+static void
+arm_emit_base_file_entry (info, output_bfd, input_section, reloc_offset)
+      struct bfd_link_info *info;
+      bfd *output_bfd;
+      asection *input_section;
+      bfd_vma reloc_offset;
+{
+  bfd_vma addr = reloc_offset
+                - input_section->vma
+                + 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);
+
+}
 \f
 /* The thumb form of a long branch is a bit finicky, because the offset
    encoding is split over two fields, each in it's own instruction. They
-   can occur in any order. So given a thumb form of long branch, and an 
+   can occur in any order. So given a thumb form of long branch, and an
    offset, insert the offset into the thumb branch and return finished
-   instruction. 
+   instruction.
 
-   It takes two thumb instructions to encode the target address. Each has 
+   It takes two thumb instructions to encode the target address. Each has
    11 bits to invest. The upper 11 bits are stored in one (identifed by
-   H-0.. see below), the lower 11 bits are stored in the other (identified 
-   by H-1). 
+   H-0.. see below), the lower 11 bits are stored in the other (identified
+   by H-1).
 
-   Combine together and shifted left by 1 (it's a half word address) and 
+   Combine together and shifted left by 1 (it's a half word address) and
    there you have it.
 
      Op: 1111 = F,
@@ -772,14 +989,14 @@ typedef unsigned short int insn16;
      Op: 1111 = F,
      H-1, lower address-0 = 800
 
-   They can be ordered either way, but the arm tools I've seen always put 
+   They can be ordered either way, but the arm tools I've seen always put
    the lower one first. It probably doesn't matter. krk@cygnus.com
 
    XXX:  Actually the order does matter.  The second instruction (H-1)
    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
@@ -792,51 +1009,48 @@ insert_thumb_branch (br_insn, rel_off)
   unsigned int low_bits;
   unsigned int high_bits;
 
-
   BFD_ASSERT((rel_off & 1) != 1);
 
   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;
   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;
-     char *                 name;
-     bfd *                  input_bfd;
+     struct bfd_link_info *info;
+     const 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;
+  bfd_size_type amt = strlen (name) + strlen (THUMB2ARM_GLUE_ENTRY_NAME) + 1;
 
-  tmp_name = ((char *)
-        bfd_malloc (strlen (name) + strlen (THUMB2ARM_GLUE_ENTRY_NAME)));
+  tmp_name = (char *) bfd_malloc (amt);
 
   BFD_ASSERT (tmp_name);
 
   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);
-    }
-  
+    /* xgettext:c-format */
+    _bfd_error_handler (_("%s: unable to find THUMB glue '%s' for `%s'"),
+                       bfd_archive_filename (input_bfd), tmp_name, name);
+
   free (tmp_name);
 
   return myh;
@@ -844,30 +1058,28 @@ 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;
-     char *                 name;
-     bfd *                  input_bfd;
+     struct bfd_link_info *info;
+     const 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;
+  bfd_size_type amt = strlen (name) + strlen (ARM2THUMB_GLUE_ENTRY_NAME) + 1;
 
-  tmp_name = ((char *)
-             bfd_malloc (strlen (name) + strlen (ARM2THUMB_GLUE_ENTRY_NAME)));
+  tmp_name = (char *) bfd_malloc (amt);
 
   BFD_ASSERT (tmp_name);
 
   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);
-    }
-  
+    /* xgettext:c-format */
+    _bfd_error_handler (_("%s: unable to find ARM glue '%s' for `%s'"),
+                       bfd_archive_filename (input_bfd), tmp_name, name);
+
   free (tmp_name);
 
   return myh;
@@ -882,44 +1094,42 @@ find_arm_glue (info, name, input_bfd)
             bx  r12
        __func_addr:
             .word func    @ behave as if you saw a ARM_32 reloc
-
-   ldr ip,8 <__func_addr> e59fc000
-   bx  ip                 e12fff1c
-   .word func             00000001
-
 */
 
 #define ARM2THUMB_GLUE_SIZE 12
-static const insn32 
-        a2t1_ldr_insn = 0xe59fc000,
-       a2t2_bx_r12_insn = 0xe12fff1c,
-       a2t3_func_addr_insn = 0x00000001;
+static const insn32 a2t1_ldr_insn       = 0xe59fc000;
+static const insn32 a2t2_bx_r12_insn    = 0xe12fff1c;
+static const insn32 a2t3_func_addr_insn = 0x00000001;
 
 /*
-   Thumb->ARM:
-
-   .thumb
-   .align 2
-      __func_from_thumb:
-          bx pc
-          nop
-   .arm
-      __func_change_to_arm:
-           b func
-
-
-   bx  pc    4778
-   nop       0000
-   b   func  eafffffe
-
+   Thumb->ARM:                         Thumb->(non-interworking aware) ARM
+
+   .thumb                              .thumb
+   .align 2                            .align 2
+      __func_from_thumb:                  __func_from_thumb:
+          bx pc                                push {r6, lr}
+          nop                                  ldr  r6, __func_addr
+   .arm                                                mov  lr, pc
+      __func_change_to_arm:                    bx   r6
+          b func                       .arm
+                                          __func_back_to_thumb:
+                                               ldmia r13! {r6, lr}
+                                               bx    lr
+                                          __func_addr:
+                                               .word   func
 */
 
-#define THUMB2ARM_GLUE_SIZE 8
-static const insn16
-        t2a1_bx_pc_insn    = 0x4778,
-        t2a2_noop_insn     = 0x46c0;
-static const insn32
-        t2a3_b_insn        = 0xea000000;
+#define THUMB2ARM_GLUE_SIZE (globals->support_old_code ? 20 : 8)
+static const insn16 t2a1_bx_pc_insn = 0x4778;
+static const insn16 t2a2_noop_insn  = 0x46c0;
+static const insn32 t2a3_b_insn     = 0xea000000;
+
+static const insn16 t2a1_push_insn  = 0xb540;
+static const insn16 t2a2_ldr_insn   = 0x4e03;
+static const insn16 t2a3_mov_insn   = 0x46fe;
+static const insn16 t2a4_bx_insn    = 0x4730;
+static const insn32 t2a5_pop_insn   = 0xe8bd4040;
+static const insn32 t2a6_bx_insn    = 0xe12fff1e;
 
 /* TODO:
      We should really create new local (static) symbols in destination
@@ -949,10 +1159,10 @@ static const insn32
      instruction we should generate a trampoline stub (needed to
      address the complete 32bit address space).  */
 
-/* The standard COFF backend linker does not cope with the special 
+/* The standard COFF backend linker does not cope with the special
    Thumb BRANCH23 relocation.  The alternative would be to split the
    BRANCH23 into seperate HI23 and LO23 relocations. However, it is a
-   bit simpler simply providing our own relocation driver. */
+   bit simpler simply providing our own relocation driver.  */
 
 /* The reloc processing routine for the ARM/Thumb COFF linker.  NOTE:
    This code is a very slightly modified copy of
@@ -982,15 +1192,15 @@ coff_arm_relocate_section (output_bfd, info, input_bfd, input_section,
 
   for (; rel < relend; rel++)
     {
-      int done = 0;
-      long symndx;
-      struct coff_link_hash_entry *h;
-      struct internal_syment *sym;
-      bfd_vma addend;
-      bfd_vma val;
-      reloc_howto_type *howto;
-      bfd_reloc_status_type rstat;
-      bfd_vma h_val;
+      int                            done = 0;
+      long                           symndx;
+      struct coff_link_hash_entry *  h;
+      struct internal_syment *       sym;
+      bfd_vma                        addend;
+      bfd_vma                        val;
+      reloc_howto_type *             howto;
+      bfd_reloc_status_type          rstat;
+      bfd_vma                        h_val;
 
       symndx = rel->r_symndx;
 
@@ -1000,7 +1210,7 @@ coff_arm_relocate_section (output_bfd, info, input_bfd, input_section,
          sym = NULL;
        }
       else
-       {    
+       {
          h = obj_coff_sym_hashes (input_bfd)[symndx];
          sym = syms + symndx;
        }
@@ -1015,12 +1225,50 @@ coff_arm_relocate_section (output_bfd, info, input_bfd, input_section,
       else
        addend = 0;
 
-
-      howto = bfd_coff_rtype_to_howto (input_bfd, input_section, rel, h,
+      howto = coff_rtype_to_howto (input_bfd, input_section, rel, h,
                                       sym, &addend);
       if (howto == NULL)
        return false;
 
+      /* The relocation_section function will skip pcrel_offset relocs
+         when doing a relocateable link.  However, we want to convert
+         ARM26 to ARM26D relocs if possible.  We return a fake howto in
+         this case without pcrel_offset set, and adjust the addend to
+         compensate.  */
+      if (rel->r_type == ARM_26
+          && h != NULL
+          && 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))
+        {
+          static reloc_howto_type fake_arm26_reloc =
+           HOWTO (ARM_26,
+              2,
+              2,
+              24,
+              true,
+              0,
+              complain_overflow_signed,
+              aoutarm_fix_pcrel_26 ,
+              "ARM_26",
+              false,
+              0x00ffffff,
+              0x00ffffff,
+              false);
+
+          addend -= rel->r_vaddr - input_section->vma;
+          howto = &fake_arm26_reloc;
+        }
+
+#ifdef ARM_WINCE
+      /* MS ARM-CE makes the reloc relative to the opcode's pc, not
+        the next opcode's pc, so is off by one.  */
+      if (howto->pc_relative && !info->relocateable)
+       addend -= 8;
+#endif
+
       /* If we are doing a relocateable link, then we can just ignore
          a PC relative reloc that is pcrel_offset.  It will already
          have the correct value.  If this is not a relocateable link,
@@ -1029,18 +1277,23 @@ coff_arm_relocate_section (output_bfd, info, input_bfd, input_section,
         {
           if (info->relocateable)
             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 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
+            the beginning of the symbol's section, so we must not cancel
+            out the symbol's value, otherwise we'll be adding it in
+            twice.  */
           if (sym != NULL && sym->n_scnum != 0)
             addend += sym->n_value;
+#endif
         }
 
-      /* If we are doing a relocateable link, then we can just ignore
-         a PC relative reloc that is pcrel_offset.  It will already
-         have the correct value.  */
-      if (info->relocateable
-          && howto->pc_relative
-          && howto->pcrel_offset)
-        continue;
-
       val = 0;
 
       if (h == NULL)
@@ -1068,15 +1321,14 @@ coff_arm_relocate_section (output_bfd, info, input_bfd, input_section,
              relocatable output file, since we may as well leave the
              stub generation to the final linker pass. If we fail to
             verify that the name is defined, we'll try to build stubs
-            for an undefined name... */
+            for an undefined name...  */
           if (! info->relocateable
              && (   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;
-             
+
              /* h locates the symbol referenced in the reloc.  */
              h_val = (h->root.u.def.value
                       + h_sec->output_section->vma
@@ -1087,27 +1339,30 @@ 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 */
-                     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; 
+                     /* Arm code calling a Thumb function.  */
+                     unsigned long int                 tmp;
+                     bfd_vma                           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;
 
+                     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,
                                                  ARM2THUMB_GLUE_SECTION_NAME);
                      BFD_ASSERT (s != NULL);
                      BFD_ASSERT (s->contents != NULL);
+                     BFD_ASSERT (s->output_section != NULL);
 
                      if ((my_offset & 0x01) == 0x01)
                        {
@@ -1116,140 +1371,199 @@ coff_arm_relocate_section (output_bfd, info, input_bfd, input_section,
                              && ! INTERWORK_FLAG (h_sec->owner))
                            {
                              _bfd_error_handler
-                               ("%s(%s): warning: interworking not enabled.",
-                                bfd_get_filename (h_sec->owner), name);
+                               /* xgettext:c-format */
+                               (_("%s(%s): warning: interworking not enabled."),
+                                bfd_archive_filename (h_sec->owner), name);
                              _bfd_error_handler
-                               ("  first occurrence: %s: arm call to thumb",
-                                bfd_get_filename (input_bfd));
+                               /* xgettext:c-format */
+                               (_("  first occurrence: %s: arm call to thumb"),
+                                bfd_archive_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, a2t2_bx_r12_insn, s->contents + my_offset + 4);
-                         
+                         bfd_put_32 (output_bfd, (bfd_vma) a2t1_ldr_insn,
+                                     s->contents + my_offset);
+
+                         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.  */
-                         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);
+
+                          if (info->base_file)
+                            arm_emit_base_file_entry (info, output_bfd, s,
+                                                      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. */
+
+                     /* Somehow these are both 4 too far, so subtract 8.  */
                      ret_offset =
                        s->output_offset
-                       + my_offset 
+                       + my_offset
                        + s->output_section->vma
                        - (input_section->output_offset
-                          + input_section->output_section->vma 
+                          + input_section->output_section->vma
                           + rel->r_vaddr)
                        - 8;
 
                      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;
                    }
                 }
-             
-             /* Note: We used to check for ARM_THUMB9 and ARM_THUMB12 */
+
+#ifndef ARM_WINCE
+             /* Note: We used to check for ARM_THUMB9 and ARM_THUMB12.  */
               else if (howto->type == ARM_THUMB23)
                 {
-                  if (   h->class == C_EXT 
+                  if (   h->class == C_EXT
                      || h->class == C_STAT
                      || 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;
+                     bfd_vma                            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 (h_sec->owner != NULL
                              && INTERWORK_SET (h_sec->owner)
-                             && ! INTERWORK_FLAG (h_sec->owner))
+                             && ! INTERWORK_FLAG (h_sec->owner)
+                             && ! globals->support_old_code)
                            {
                              _bfd_error_handler
-                               ("%s(%s): warning: interworking not enabled.",
-                                bfd_get_filename (h_sec->owner), name);
+                               /* xgettext:c-format */
+                               (_("%s(%s): warning: interworking not enabled."),
+                                bfd_archive_filename (h_sec->owner), name);
+                             _bfd_error_handler
+                               /* xgettext:c-format */
+                               (_("  first occurrence: %s: thumb call to arm"),
+                                bfd_archive_filename (input_bfd));
                              _bfd_error_handler
-                               ("  first occurrence: %s: thumb call to arm",
-                                bfd_get_filename (input_bfd));
+                               (_("  consider relinking with --support-old-code enabled"));
                            }
-                         
+
                          -- my_offset;
                          myh->root.u.def.value = my_offset;
 
-                         bfd_put_16 (output_bfd, t2a1_bx_pc_insn,
-                                    s->contents + my_offset);
-                     
-                         bfd_put_16 (output_bfd, t2a2_noop_insn,
-                                    s->contents + my_offset + 2);
-                     
-                         ret_offset =
-                           ((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);
+                         if (globals->support_old_code)
+                           {
+                             bfd_put_16 (output_bfd, (bfd_vma) t2a1_push_insn,
+                                         s->contents + my_offset);
+
+                             bfd_put_16 (output_bfd, (bfd_vma) t2a2_ldr_insn,
+                                         s->contents + my_offset + 2);
+
+                             bfd_put_16 (output_bfd, (bfd_vma) t2a3_mov_insn,
+                                         s->contents + my_offset + 4);
+
+                             bfd_put_16 (output_bfd, (bfd_vma) t2a4_bx_insn,
+                                         s->contents + my_offset + 6);
+
+                             bfd_put_32 (output_bfd, (bfd_vma) t2a5_pop_insn,
+                                         s->contents + my_offset + 8);
+
+                             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.  */
+                             bfd_put_32 (output_bfd, h_val,
+                                         s->contents + my_offset + 16);
+
+                              if (info->base_file)
+                                arm_emit_base_file_entry (info, output_bfd, s,
+                                                         my_offset + 16);
+                           }
+                         else
+                           {
+                             bfd_put_16 (output_bfd, (bfd_vma) t2a1_bx_pc_insn,
+                                         s->contents + my_offset);
+
+                             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)
+                                  (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,
+                                         (bfd_vma) 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
                        - (input_section->output_offset
                           + 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),
+                                 (bfd_vma) insert_thumb_branch (tmp,
+                                                                ret_offset),
                                  contents + rel->r_vaddr - input_section->vma);
-                     
+
                      done = 1;
                     }
                 }
+#endif
             }
-         
+
           /* If the relocation type and destination symbol does not
              fall into one of the above categories, then we can just
-             perform a direct link. */
+             perform a direct link.  */
 
          if (done)
            rstat = bfd_reloc_ok;
-         else 
+         else
 #endif /* THUMBEXTENSION */
            if (   h->root.type == bfd_link_hash_defined
                || h->root.type == bfd_link_hash_defweak)
@@ -1266,33 +1580,23 @@ coff_arm_relocate_section (output_bfd, info, input_bfd, input_section,
            {
              if (! ((*info->callbacks->undefined_symbol)
                     (info, h->root.root.string, input_bfd, input_section,
-                     rel->r_vaddr - input_section->vma)))
+                     rel->r_vaddr - input_section->vma, true)))
                return false;
            }
        }
 
       if (info->base_file)
        {
-         /* Emit a reloc if the backend thinks it needs it. */
+         /* Emit a reloc if the backend thinks it needs it.  */
          if (sym && pe_data(output_bfd)->in_reloc_p(output_bfd, howto))
-           {
-             /* relocation to a symbol in a section which
-                isn't absolute - we output the address here 
-                to a file */
-             bfd_vma addr = rel->r_vaddr 
-               - input_section->vma 
-               + input_section->output_offset 
-                 + input_section->output_section->vma;
-             if (coff_data(output_bfd)->pe)
-               addr -= pe_data(output_bfd)->pe_opthdr.ImageBase;
-             /* FIXME: Shouldn't 4 be sizeof (addr)?  */
-             fwrite (&addr, 1,4, (FILE *) info->base_file);
-           }
+            arm_emit_base_file_entry (info, output_bfd, input_section,
+                                     rel->r_vaddr);
        }
-  
+
 #if 1 /* THUMBEXTENSION */
       if (done)
-       ;
+       rstat = bfd_reloc_ok;
+#ifndef ARM_WINCE
       /* Only perform this fix during the final link, not a relocatable link.  nickc@cygnus.com  */
       else if (! info->relocateable
               && howto->type == ARM_THUMB23)
@@ -1304,11 +1608,11 @@ coff_arm_relocate_section (output_bfd, info, input_bfd, input_section,
              instruction.  Again, it would probably be simpler adding a
              ThumbBRANCH23 specific macro expansion into the default
              code.  */
-         
+
           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;
@@ -1325,21 +1629,21 @@ coff_arm_relocate_section (output_bfd, info, input_bfd, input_section,
              bfd_signed_vma  signed_add;
 
              BFD_ASSERT (size == 4);
-             
-              /* howto->pc_relative should be TRUE for type 14 BRANCH23 */
+
+              /* howto->pc_relative should be TRUE for type 14 BRANCH23 */
               relocation -= (input_section->output_section->vma
                              + input_section->output_offset);
-             
-              /* howto->pcrel_offset should be TRUE for type 14 BRANCH23 */
+
+              /* howto->pcrel_offset should be TRUE for type 14 BRANCH23 */
               relocation -= address;
-             
-             /* No need to negate the relocation with BRANCH23. */
+
+             /* No need to negate the relocation with BRANCH23.  */
              /* howto->complain_on_overflow == complain_overflow_signed for BRANCH23.  */
              /* howto->rightshift == 1 */
+
              /* Drop unwanted bits from the value we are relocating to.  */
-             
              check = relocation >> howto->rightshift;
-               
+
              /* If this is a signed value, the rightshift just dropped
                 leading 1 bits (assuming twos complement).  */
              if ((bfd_signed_vma) relocation >= 0)
@@ -1348,16 +1652,12 @@ coff_arm_relocate_section (output_bfd, info, input_bfd, input_section,
                signed_check = (check
                                | ((bfd_vma) - 1
                                   & ~((bfd_vma) - 1 >> howto->rightshift)));
-             
+
              /* Get the value from the object file.  */
              if (bfd_big_endian (input_bfd))
-               {
-                 add = (((x) & 0x07ff0000) >> 4) | (((x) & 0x7ff) << 1);
-               }
+               add = (((x) & 0x07ff0000) >> 4) | (((x) & 0x7ff) << 1);
              else
-               {
-                 add = ((((x) & 0x7ff) << 12) | (((x) & 0x07ff0000) >> 15));
-               }
+               add = ((((x) & 0x7ff) << 12) | (((x) & 0x07ff0000) >> 15));
 
              /* Get the value from the object file with an appropriate sign.
                 The expression involving howto->src_mask isolates the upper
@@ -1367,18 +1667,16 @@ coff_arm_relocate_section (output_bfd, info, input_bfd, input_section,
                 can not get the upper bit, but that does not matter since
                 signed_add needs no adjustment to become negative in that
                 case.  */
-             
              signed_add = add;
-             
+
              if ((add & (((~ src_mask) >> 1) & src_mask)) != 0)
                signed_add -= (((~ src_mask) >> 1) & src_mask) << 1;
-             
+
+             /* howto->bitpos == 0 */
              /* Add the value from the object file, shifted so that it is a
                 straight number.  */
-             /* howto->bitpos == 0 */
-             
              signed_check += signed_add;
-             relocation += signed_add;
+             relocation   += signed_add;
 
              BFD_ASSERT (howto->complain_on_overflow == complain_overflow_signed);
 
@@ -1386,27 +1684,33 @@ coff_arm_relocate_section (output_bfd, info, input_bfd, input_section,
              if (   signed_check > reloc_signed_max
                  || signed_check < reloc_signed_min)
                overflow = true;
-             
-             /* Put RELOCATION into the correct bits:  */
-             
+
+             /* For the BLX(1) instruction remove bit 0 of the adjusted offset.
+                Bit 0 can only be set if the upper insn is at a half-word boundary,
+                since the destination address, an ARM instruction, must always be
+                on a word boundary.  The semantics of the BLX (1) instruction,
+                however, are that bit 0 in the offset must always be 0, and the
+                corresponding bit 1 in the target address will be set from bit
+                1 of the source address.  */
+             if ((x & 0x18000000) == 0x08000000)
+               relocation &= ~0x2;
+
+             /* Put the relocation into the correct bits.  */
              if (bfd_big_endian (input_bfd))
-               {
-                 relocation = (((relocation & 0xffe) >> 1)  | ((relocation << 4) & 0x07ff0000));
-               }
+               relocation = (((relocation & 0xffe) >> 1)  | ((relocation << 4) & 0x07ff0000));
              else
-               {
-                 relocation = (((relocation & 0xffe) << 15) | ((relocation >> 12) & 0x7ff));
-               }
-             
-             /* Add RELOCATION to the correct bits of X:  */
+               relocation = (((relocation & 0xffe) << 15) | ((relocation >> 12) & 0x7ff));
+
+             /* Add the relocation to the correct bits of X.  */
              x = ((x & ~howto->dst_mask) | relocation);
 
-             /* Put the relocated value back in the object file:  */
+             /* Put the relocated value back in the object file.  */
              bfd_put_32 (input_bfd, x, location);
 
              rstat = overflow ? bfd_reloc_overflow : bfd_reloc_ok;
             }
         }
+#endif
       else
 #endif /* THUMBEXTENSION */
         rstat = _bfd_final_link_relocate (howto, input_bfd, input_section,
@@ -1414,18 +1718,18 @@ coff_arm_relocate_section (output_bfd, info, input_bfd, input_section,
                                           rel->r_vaddr - input_section->vma,
                                           val, addend);
 #if 1 /* THUMBEXTENSION */
-      /* FIXME: 
+      /* FIXME:
         Is this the best way to fix up thumb addresses? krk@cygnus.com
         Probably not, but it works, and if it works it don't need fixing!  nickc@cygnus.com */
       /* Only perform this fix during the final link, not a relocatable link.  nickc@cygnus.com  */
       if (! info->relocateable
-         && rel->r_type == ARM_32)
+         && (rel->r_type == ARM_32 || rel->r_type == ARM_RVA32))
        {
          /* Determine if we need to set the bottom bit of a relocated address
             because the address is the address of a Thumb code symbol.  */
-            
+
          int patchit = false;
-         
+
          if (h != NULL
              && (   h->class == C_THUMBSTATFUNC
                  || h->class == C_THUMBEXTFUNC))
@@ -1450,8 +1754,8 @@ coff_arm_relocate_section (output_bfd, info, input_bfd, input_section,
              bfd_put_32 (input_bfd, x | 1, location);
            }
        }
-#endif /* THUMBEXTENSION */      
-      
+#endif /* THUMBEXTENSION */
+
       switch (rstat)
        {
        default:
@@ -1460,8 +1764,8 @@ coff_arm_relocate_section (output_bfd, info, input_bfd, input_section,
          break;
        case bfd_reloc_outofrange:
          (*_bfd_error_handler)
-           ("%s: bad reloc address 0x%lx in section `%s'",
-            bfd_get_filename (input_bfd),
+           (_("%s: bad reloc address 0x%lx in section `%s'"),
+            bfd_archive_filename (input_bfd),
             (unsigned long) rel->r_vaddr,
             bfd_get_section_name (input_bfd, input_section));
          return false;
@@ -1492,48 +1796,58 @@ coff_arm_relocate_section (output_bfd, info, input_bfd, input_section,
   return true;
 }
 
-#if defined COFF_IMAGE_WITH_PE || ! defined COFF_WITH_PE
+#ifndef COFF_IMAGE_WITH_PE
+
 boolean
-arm_allocate_interworking_sections (info) 
-     struct bfd_link_info *info;
+bfd_arm_allocate_interworking_sections (info)
+     struct bfd_link_info * info;
 {
-  asection * s;
-  bfd_byte * foo;
+  asection *                        s;
+  bfd_byte *                        foo;
+  struct coff_arm_link_hash_table * globals;
 #if 0
-  static char test_char = '1';
+  static char                       test_char = '1';
 #endif
 
-  if (global_arm_glue_size != 0)
+  globals = coff_arm_hash_table (info);
+
+  BFD_ASSERT (globals != NULL);
+
+  if (globals->arm_glue_size != 0)
     {
-      BFD_ASSERT (bfd_of_glue_owner != NULL);
-      
-      s = bfd_get_section_by_name (bfd_of_glue_owner, ARM2THUMB_GLUE_SECTION_NAME);
+      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);
-      
-      foo = (bfd_byte *) bfd_alloc (bfd_of_glue_owner, global_arm_glue_size);
+
+      foo = (bfd_byte *) bfd_alloc (globals->bfd_of_glue_owner,
+                                   globals->arm_glue_size);
 #if 0
-      memset (foo, test_char, global_arm_glue_size);
+      memset (foo, test_char, (size_t) 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);
-      
-      s = bfd_get_section_by_name (bfd_of_glue_owner, THUMB2ARM_GLUE_SECTION_NAME);
+      BFD_ASSERT (globals->bfd_of_glue_owner != NULL);
+
+      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);
+
+      foo = (bfd_byte *) bfd_alloc (globals->bfd_of_glue_owner,
+                                   globals->thumb_glue_size);
 #if 0
-      memset (foo, test_char, global_thumb_glue_size);
+      memset (foo, test_char, (size_t) 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;
     }
 
@@ -1545,51 +1859,53 @@ 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_vma val;
+  bfd_size_type amt;
 
+  globals = coff_arm_hash_table (info);
 
-  BFD_ASSERT (bfd_of_glue_owner != NULL);
+  BFD_ASSERT (globals != 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);
 
-  tmp_name = ((char *)
-             bfd_malloc (strlen (name) + strlen (ARM2THUMB_GLUE_ENTRY_NAME)));
+  amt = strlen (name) + strlen (ARM2THUMB_GLUE_ENTRY_NAME) + 1;
+  tmp_name = (char *) bfd_malloc (amt);
 
   BFD_ASSERT (tmp_name);
 
   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)
     {
       free (tmp_name);
       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,
-                               BSF_EXPORT | BSF_GLOBAL, 
-                               s, global_arm_glue_size + 1,
-                               NULL, true, false, 
+  val = globals->arm_glue_size + 1;
+  bfd_coff_link_add_one_symbol (info, globals->bfd_of_glue_owner, tmp_name,
+                               BSF_GLOBAL, s, val, NULL, true, false,
                                (struct bfd_link_hash_entry **) & myh);
-  
-  global_arm_glue_size += ARM2THUMB_GLUE_SIZE;
 
   free (tmp_name);
 
+  globals->arm_glue_size += ARM2THUMB_GLUE_SIZE;
+
   return;
 }
 
@@ -1598,86 +1914,160 @@ 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_vma val;
+  bfd_size_type amt;
+
+  globals = coff_arm_hash_table (info);
 
-  BFD_ASSERT (bfd_of_glue_owner != NULL);
+  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);
 
-  tmp_name = (char *) bfd_malloc (strlen (name) + strlen (THUMB2ARM_GLUE_ENTRY_NAME));
+  amt = strlen (name) + strlen (THUMB2ARM_GLUE_ENTRY_NAME) + 1;
+  tmp_name = (char *) bfd_malloc (amt);
 
   BFD_ASSERT (tmp_name);
 
   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)
     {
       free (tmp_name);
       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,
-                               NULL, true, false, 
-                               (struct bfd_link_hash_entry **) & myh);
-  
+  val = globals->thumb_glue_size + 1;
+  bfd_coff_link_add_one_symbol (info, globals->bfd_of_glue_owner, tmp_name,
+                               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;
 
   free (tmp_name);
 
+  /* Allocate another symbol to mark where we switch to arm mode.  */
+
 #define CHANGE_TO_ARM "__%s_change_to_arm"
-  
-  tmp_name = ((char *)
-             bfd_malloc (strlen (name) + strlen (CHANGE_TO_ARM)));
+#define BACK_FROM_ARM "__%s_back_from_arm"
+
+  amt = strlen (name) + strlen (CHANGE_TO_ARM) + 1;
+  tmp_name = (char *) bfd_malloc (amt);
 
   BFD_ASSERT (tmp_name);
 
-  sprintf (tmp_name, CHANGE_TO_ARM, name);
+  sprintf (tmp_name, globals->support_old_code ? BACK_FROM_ARM : CHANGE_TO_ARM, name);
 
   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,
-                               NULL, true, false, 
+  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, val, NULL, true, false,
                                (struct bfd_link_hash_entry **) & myh);
 
-  free (tmp_name);  
+  free (tmp_name);
 
-  global_thumb_glue_size += THUMB2ARM_GLUE_SIZE;
+  globals->thumb_glue_size += THUMB2ARM_GLUE_SIZE;
 
   return;
 }
 
+/* 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
-arm_process_before_allocation (abfd, info)
+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_CODE | SEC_READONLY;
+
+      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_CODE | SEC_READONLY;
+
+      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, support_old_code)
      bfd *                   abfd;
      struct bfd_link_info *  info;
+     int                    support_old_code;
 {
   asection * sec;
+  struct coff_arm_link_hash_table * globals;
 
   /* If we are only performing a partial link do not bother
      to construct any glue.  */
   if (info->relocateable)
     return true;
-  
+
   /* Here we have a bfd that is to be included on the link.  We have a hook
      to do reloc rummaging, before section sizes are nailed down.  */
 
   _bfd_coff_get_external_symbols (abfd);
 
-  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);
+
+  globals->support_old_code = support_old_code;
+
   /* Rummage around all the relocs and map the glue vectors.  */
   sec = abfd->sections;
 
@@ -1689,17 +2079,17 @@ arm_process_before_allocation (abfd, info)
       struct internal_reloc * i;
       struct internal_reloc * rel;
 
-      if (sec->reloc_count == 0) 
+      if (sec->reloc_count == 0)
        continue;
 
       /* Load the relocs.  */
-      /* FIXME: there may be a storage leak here. */
-      
+      /* FIXME: there may be a storage leak here.  */
+
       i = _bfd_coff_read_internal_relocs (abfd, sec, 1, 0, 0, 0);
-    
+
       BFD_ASSERT (i != 0);
 
-      for (rel = i; rel < i + sec->reloc_count; ++rel) 
+      for (rel = i; rel < i + sec->reloc_count; ++rel)
        {
          unsigned short                 r_type  = rel->r_type;
          long                           symndx;
@@ -1711,6 +2101,14 @@ arm_process_before_allocation (abfd, info)
          if (symndx == -1)
            continue;
 
+         /* If the index is outside of the range of our table, something has gone wrong.  */
+         if (symndx >= obj_conv_table_size (abfd))
+           {
+             _bfd_error_handler (_("%s: illegal symbol index in reloc: %d"),
+                                 bfd_archive_filename (abfd), symndx);
+             continue;
+           }
+
          h = obj_coff_sym_hashes (abfd)[symndx];
 
          /* If the relocation is against a static symbol it must be within
@@ -1724,11 +2122,12 @@ arm_process_before_allocation (abfd, info)
              /* This one is a call from arm code.  We need to look up
                 the target of the call. If it is a thumb target, we
                 insert glue.  */
-             
+
              if (h->class == C_THUMBEXTFUNC)
                record_arm_to_thumb_glue (info, h);
              break;
-             
+
+#ifndef ARM_WINCE
            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
@@ -1749,7 +2148,8 @@ arm_process_before_allocation (abfd, info)
                  ;
                }
              break;
-             
+#endif
+
            default:
              break;
            }
@@ -1758,17 +2158,28 @@ arm_process_before_allocation (abfd, info)
 
   return true;
 }
-#endif /* COFF_IMAGE_WITH_PE or not COFF_WITH_PE */
 
-#define coff_relocate_section coff_arm_relocate_section
+#endif /* ! defined (COFF_IMAGE_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
 
 /* When doing a relocateable link, we want to convert ARM26 relocs
    into ARM26D relocs.  */
 
 static boolean
 coff_arm_adjust_symndx (obfd, info, ibfd, sec, irel, adjustedp)
-     bfd *obfd;
-     struct bfd_link_info *info;
+     bfd *obfd ATTRIBUTE_UNUSED;
+     struct bfd_link_info *info ATTRIBUTE_UNUSED;
      bfd *ibfd;
      asection *sec;
      struct internal_reloc *irel;
@@ -1789,14 +2200,13 @@ coff_arm_adjust_symndx (obfd, info, ibfd, sec, irel, adjustedp)
   return true;
 }
 
-#if defined COFF_IMAGE_WITH_PE || ! defined COFF_WITH_PE
 /* 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.     */
 
-boolean
-coff_arm_bfd_merge_private_bfd_data (ibfd, obfd)
+static boolean
+coff_arm_merge_private_bfd_data (ibfd, obfd)
      bfd *   ibfd;
      bfd *   obfd;
 {
@@ -1821,34 +2231,46 @@ coff_arm_bfd_merge_private_bfd_data (ibfd, obfd)
          if (APCS_26_FLAG (obfd) != APCS_26_FLAG (ibfd))
            {
              _bfd_error_handler
-               ("%s: ERROR: compiled for APCS-%d whereas target %s uses APCS-%d",
-                bfd_get_filename (ibfd), APCS_26_FLAG (ibfd) ? 26 : 32,
+               /* xgettext: c-format */
+               (_("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
                 );
 
              bfd_set_error (bfd_error_wrong_format);
              return false;
            }
-         
+
          if (APCS_FLOAT_FLAG (obfd) != APCS_FLOAT_FLAG (ibfd))
            {
-             _bfd_error_handler
-               ("%s: ERROR: passes floats in %s registers whereas target %s uses %s registers",
-                bfd_get_filename (ibfd), APCS_FLOAT_FLAG (ibfd) ? "float" : "integer",
-                bfd_get_filename (obfd), APCS_FLOAT_FLAG (obfd) ? "float" : "integer"
-                );
+             const char *msg;
+
+             if (APCS_FLOAT_FLAG (ibfd))
+               /* xgettext: c-format */
+               msg = _("ERROR: %s passes floats in float registers, whereas %s passes them in integer registers");
+             else
+               /* xgettext: c-format */
+               msg = _("ERROR: %s passes floats in integer registers, whereas %s passes them in float registers");
+
+             _bfd_error_handler (msg, bfd_archive_filename (ibfd),
+                                 bfd_get_filename (obfd));
 
              bfd_set_error (bfd_error_wrong_format);
              return false;
            }
-         
+
          if (PIC_FLAG (obfd) != PIC_FLAG (ibfd))
            {
-             _bfd_error_handler
-               ("%s: ERROR: compiled as %s code, whereas target %s is %s",
-                bfd_get_filename (ibfd), PIC_FLAG (ibfd) ? "position independent" : "absoluste position",
-                bfd_get_filename (obfd), PIC_FLAG (obfd) ? "position independent" : "absoluste position"
-                );
+             const char * msg;
+
+             if (PIC_FLAG (ibfd))
+               /* xgettext: c-format */
+               msg = _("ERROR: %s is compiled as position independent code, whereas target %s is absolute position");
+             else
+               /* xgettext: c-format */
+               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);
              return false;
@@ -1857,7 +2279,7 @@ coff_arm_bfd_merge_private_bfd_data (ibfd, obfd)
       else
        {
          SET_APCS_FLAGS (obfd, APCS_26_FLAG (ibfd) | APCS_FLOAT_FLAG (ibfd) | PIC_FLAG (ibfd));
-         
+
          /* Set up the arch and fields as well as these are probably wrong.  */
          bfd_set_arch_mach (obfd, bfd_get_arch (ibfd), bfd_get_mach (ibfd));
        }
@@ -1871,13 +2293,17 @@ coff_arm_bfd_merge_private_bfd_data (ibfd, obfd)
          /* If the src and dest differ in their interworking issue a warning.  */
          if (INTERWORK_FLAG (obfd) != INTERWORK_FLAG (ibfd))
            {
-             _bfd_error_handler
-               ("Warning: input file %s %s interworking, whereas %s does%s",
-                bfd_get_filename (ibfd),
-                INTERWORK_FLAG (ibfd) ? "supports" : "does not support",
-                bfd_get_filename (obfd),
-                INTERWORK_FLAG (obfd) ? "." : " not."
-                );
+             const char * msg;
+
+             if (INTERWORK_FLAG (ibfd))
+               /* xgettext: c-format */
+               msg = _("Warning: %s supports interworking, whereas %s does not");
+             else
+               /* xgettext: c-format */
+               msg = _("Warning: %s does not support interworking, whereas %s does");
+
+             _bfd_error_handler (msg, bfd_archive_filename (ibfd),
+                                 bfd_get_filename (obfd));
            }
        }
       else
@@ -1889,56 +2315,68 @@ coff_arm_bfd_merge_private_bfd_data (ibfd, obfd)
   return true;
 }
 
-
 /* Display the flags field.  */
 
-boolean
-coff_arm_bfd_print_private_bfd_data (abfd, ptr)
+static boolean
+coff_arm_print_private_bfd_data (abfd, ptr)
      bfd *   abfd;
      PTR     ptr;
 {
   FILE * file = (FILE *) ptr;
-  
-  BFD_ASSERT (abfd != NULL && ptr != NULL)
-  
-  fprintf (file, "private flags = %x", coff_data (abfd)->flags);
-  
+
+  BFD_ASSERT (abfd != NULL && ptr != NULL);
+
+  /* xgettext:c-format */
+  fprintf (file, _("private flags = %x:"), coff_data (abfd)->flags);
+
   if (APCS_SET (abfd))
-    fprintf (file, ": [APCS-%d] [floats passed in %s registers] [%s]",
-            APCS_26_FLAG (abfd) ? 26 : 32,
-            APCS_FLOAT_FLAG (abfd) ? "float" : "integer",
-            PIC_FLAG (abfd) ? "position independent" : "absolute position"
-            );
-  
-  if (INTERWORK_SET (abfd))
-    fprintf (file, " [interworking %ssupported]",
-            INTERWORK_FLAG (abfd) ? "" : "not " );
+    {
+      /* xgettext: APCS is ARM Prodecure Call Standard, it should not be translated.  */
+      fprintf (file, " [APCS-%d]", APCS_26_FLAG (abfd) ? 26 : 32);
+
+      if (APCS_FLOAT_FLAG (abfd))
+       fprintf (file, _(" [floats passed in float registers]"));
+      else
+       fprintf (file, _(" [floats passed in integer registers]"));
+
+      if (PIC_FLAG (abfd))
+       fprintf (file, _(" [position independent]"));
+      else
+       fprintf (file, _(" [absolute position]"));
+    }
+
+  if (! INTERWORK_SET (abfd))
+    fprintf (file, _(" [interworking flag not initialised]"));
+  else if (INTERWORK_FLAG (abfd))
+    fprintf (file, _(" [interworking supported]"));
   else
-    fprintf (file, " [interworking flag not initialised]");
-  
+    fprintf (file, _(" [interworking not supported]"));
+
   fputc ('\n', file);
-  
+
   return true;
 }
 
-
 /* 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.  */
 
-boolean
-coff_arm_bfd_set_private_flags (abfd, flags)
+static boolean
+_bfd_coff_arm_set_private_flags (abfd, flags)
        bfd *      abfd;
        flagword   flags;
 {
-  int flag;
+  flagword flag;
 
   BFD_ASSERT (abfd != NULL);
 
   flag = (flags & F_APCS26) ? F_APCS_26 : 0;
-  
-  /* Make sure that the APCS field has not been initialised to the opposite value.  */
+
+  /* Make sure that the APCS field has not been initialised to the opposite
+     value.  */
   if (APCS_SET (abfd)
       && (   (APCS_26_FLAG    (abfd) != flag)
          || (APCS_FLOAT_FLAG (abfd) != (flags & F_APCS_FLOAT))
@@ -1947,35 +2385,48 @@ coff_arm_bfd_set_private_flags (abfd, flags)
     return false;
 
   flag |= (flags & (F_APCS_FLOAT | F_PIC));
-  
+
   SET_APCS_FLAGS (abfd, flag);
 
   flag = (flags & F_INTERWORK);
-  
-  /* If either the flags or the BFD do support interworking then do not set the interworking flag.  */
+
+  /* If the BFD has already had its interworking flag set, but it
+     is different from the value that we have been asked to set,
+     then assume that that merged code will not support interworking
+     and set the flag accordingly.  */
   if (INTERWORK_SET (abfd) && (INTERWORK_FLAG (abfd) != flag))
-    flag = 0;
+    {
+      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_archive_filename (abfd));
+      else
+       /* xgettext: c-format */
+       _bfd_error_handler (_("Warning: Clearing the interworking flag of %s due to outside request"),
+                           bfd_archive_filename (abfd));
+      flag = 0;
+    }
 
   SET_INTERWORK_FLAG (abfd, flag);
 
   return true;
 }
 
-
 /* Copy the important parts of the target specific data
    from one instance of a BFD to another.  */
 
-boolean
-coff_arm_bfd_copy_private_bfd_data (src, dest)
+static boolean
+coff_arm_copy_private_bfd_data (src, dest)
      bfd *  src;
      bfd *  dest;
 {
   BFD_ASSERT (src != NULL && dest != NULL);
+
   if (src == dest)
     return true;
 
-  /* If the destination is not in the same format as the source, do not do the copy.  */
+  /* If the destination is not in the same format as the source, do not do
+     the copy.  */
   if (src->xvec != dest->xvec)
     return true;
 
@@ -1987,24 +2438,37 @@ coff_arm_bfd_copy_private_bfd_data (src, dest)
          /* If the src and dest have different APCS flag bits set, fail.  */
          if (APCS_26_FLAG (dest) != APCS_26_FLAG (src))
            return false;
-         
+
          if (APCS_FLOAT_FLAG (dest) != APCS_FLOAT_FLAG (src))
            return false;
-         
+
          if (PIC_FLAG (dest) != PIC_FLAG (src))
            return false;
        }
       else
-       SET_APCS_FLAGS (dest, APCS_26_FLAG (src) | APCS_FLOAT_FLAG (src) | PIC_FLAG (src));
+       SET_APCS_FLAGS (dest, APCS_26_FLAG (src) | APCS_FLOAT_FLAG (src)
+                       | PIC_FLAG (src));
     }
 
   if (INTERWORK_SET (src))
     {
       if (INTERWORK_SET (dest))
        {
-         /* If the src and dest have different interworking flags then turn off the interworking bit.  */
+         /* If the src and dest have different interworking flags then turn
+            off the interworking bit.  */
          if (INTERWORK_FLAG (dest) != INTERWORK_FLAG (src))
-           SET_INTERWORK_FLAG (dest, 0);
+           {
+             if (INTERWORK_FLAG (dest))
+               {
+                 /* xgettext:c-format */
+                 _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_archive_filename (src));
+               }
+
+             SET_INTERWORK_FLAG (dest, 0);
+           }
        }
       else
        {
@@ -2016,25 +2480,22 @@ coff_arm_bfd_copy_private_bfd_data (src, dest)
 }
 
 /* Note:  the definitions here of LOCAL_LABEL_PREFIX and USER_LABEL_PREIFX
- *must* match the definitions on gcc/config/arm/semi.h.  */
-#define LOCAL_LABEL_PREFIX "."
+   *must* match the definitions in gcc/config/arm/{coff|semi|aout}.h.  */
+#define LOCAL_LABEL_PREFIX ""
+#ifndef USER_LABEL_PREFIX
 #define USER_LABEL_PREFIX "_"
+#endif
 
-boolean
+/* Like _bfd_coff_is_local_label_name, but
+   a) test against USER_LABEL_PREFIX, to avoid stripping labels known to be
+      non-local.
+   b) Allow other prefixes than ".", e.g. an empty prefix would cause all
+      labels of the form Lxxx to be stripped.  */
+static boolean
 coff_arm_is_local_label_name (abfd, name)
-     bfd *        abfd;
+     bfd *        abfd ATTRIBUTE_UNUSED;
      const char * name;
 {
-#ifdef LOCAL_LABEL_PREFIX
-  /* If there is a prefix for local labels then look for this.
-     If the prefix exists, but it is empty, then ignore the test. */
-  
-  if (LOCAL_LABEL_PREFIX[0] != 0)
-    {
-      if (strncmp (name, LOCAL_LABEL_PREFIX, strlen (LOCAL_LABEL_PREFIX)) == 0)
-       return true;
-    }
-#endif
 #ifdef USER_LABEL_PREFIX
   if (USER_LABEL_PREFIX[0] != 0)
     {
@@ -2042,251 +2503,95 @@ coff_arm_is_local_label_name (abfd, name)
        return false;
     }
 #endif
-  
-  /* devo/gcc/config/dbxcoff.h defines ASM_OUTPUT_SOURCE_LINE to generate local line numbers as .LM<number>, so treat these as local.  */
-  
-  switch (name[0])
+
+#ifdef LOCAL_LABEL_PREFIX
+  /* If there is a prefix for local labels then look for this.
+     If the prefix exists, but it is empty, then ignore the test.  */
+
+  if (LOCAL_LABEL_PREFIX[0] != 0)
     {
-    case 'L': return true;
-    case '.': return (name[1] == 'L' && name[2] == 'M') ? true : false;
-    default:  return false;     /* Cannot make our minds up - default to false so that it will not be stripped by accident.  */ 
+      size_t len = strlen (LOCAL_LABEL_PREFIX);
+
+      if (strncmp (name, LOCAL_LABEL_PREFIX, len) != 0)
+       return false;
+
+      /* Perform the checks below for the rest of the name.  */
+      name += len;
     }
-}
-#endif /* COFF_IMAGE_WITH_PE or not COFF_WITH_PE */
+#endif
 
-#define coff_bfd_is_local_label_name           coff_arm_is_local_label_name
-#define coff_adjust_symndx                     coff_arm_adjust_symndx
+  return name[0] == 'L';
+}
 
-#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_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
-
-extern boolean coff_arm_final_link_postscript ();
-extern boolean coff_arm_bfd_set_private_flags ();
-extern boolean coff_arm_bfd_merge_private_bfd_data ();
-extern boolean coff_arm_link_output_has_begun ();
-extern boolean coff_arm_is_local_label_name ();
-
-#if defined COFF_IMAGE_WITH_PE || ! defined COFF_WITH_PE
 /* This piece of machinery exists only to guarantee that the bfd that holds
-   the glue section is written last. 
+   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. 
+   end of the section list for the bfd.
 
    krk@cygnus.com  */
 
-boolean
-coff_arm_link_output_has_begun (sub)
+static boolean
+coff_arm_link_output_has_begun (sub, info)
      bfd * sub;
+     struct coff_final_link_info * info;
 {
-  return (sub->output_has_begun || sub == bfd_of_glue_owner);
+  return (sub->output_has_begun
+         || sub == coff_arm_hash_table (info->info)->bfd_of_glue_owner);
 }
 
-boolean
+static boolean
 coff_arm_final_link_postscript (abfd, pfinfo)
-     bfd * abfd;
+     bfd * abfd ATTRIBUTE_UNUSED;
      struct coff_final_link_info * pfinfo;
 {
-  if (bfd_of_glue_owner != NULL)
-    {
-      if (! _bfd_coff_link_input_bfd (pfinfo, bfd_of_glue_owner))
-       return false;
-      bfd_of_glue_owner->output_has_begun = true;
-    }
-  return true;
-}
-#endif /* COFF_IMAGE_WITH_PE or not COFF_WITH_PE */
-
-#define coff_SWAP_sym_in  arm_bfd_coff_swap_sym_in
-
-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;
-{
-  flagword flags;
-  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;
+  struct coff_arm_link_hash_table * globals;
 
-  /* Save the bfd for later allocation.  */
-  bfd_of_glue_owner = abfd;
+  globals = coff_arm_hash_table (pfinfo->info);
 
-  s = bfd_get_section_by_name (bfd_of_glue_owner , 
-                              ARM2THUMB_GLUE_SECTION_NAME);
+  BFD_ASSERT (globals != NULL);
 
-  if (s == NULL) 
+  if (globals->bfd_of_glue_owner != NULL)
     {
-      flags = SEC_ALLOC | SEC_LOAD | SEC_HAS_CONTENTS | SEC_IN_MEMORY ;
-      
-      s = bfd_make_section (bfd_of_glue_owner, ARM2THUMB_GLUE_SECTION_NAME);
+      if (! _bfd_coff_link_input_bfd (pfinfo, globals->bfd_of_glue_owner))
+       return false;
 
-      if (s == NULL
-         || !bfd_set_section_flags (bfd_of_glue_owner, s, flags)
-         || !bfd_set_section_alignment (bfd_of_glue_owner, s, 2))
-       {
-         /* FIXME: set appropriate bfd error */
-         abort();
-       }
+      globals->bfd_of_glue_owner->output_has_begun = true;
     }
 
-  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);
-      
-      if (s == NULL
-         || !bfd_set_section_flags (bfd_of_glue_owner, s, flags)
-         || !bfd_set_section_alignment (bfd_of_glue_owner, s, 2))
-       {
-         /* FIXME: set appropriate bfd error krk@cygnus.com */
-         abort();
-       }
-    }
-  
-  return;
+  return true;
 }
 
 #include "coffcode.h"
 
-const bfd_target
-#ifdef TARGET_LITTLE_SYM
-TARGET_LITTLE_SYM =
-#else
-armcoff_little_vec =
+#ifndef TARGET_LITTLE_SYM
+#define TARGET_LITTLE_SYM armcoff_little_vec
 #endif
-{
-#ifdef TARGET_LITTLE_NAME
-  TARGET_LITTLE_NAME,
-#else
-  "coff-arm-little",
+#ifndef TARGET_LITTLE_NAME
+#define TARGET_LITTLE_NAME "coff-arm-little"
 #endif
-  bfd_target_coff_flavour,
-  BFD_ENDIAN_LITTLE,           /* data byte order is little */
-  BFD_ENDIAN_LITTLE,           /* header byte order is little */
-
-  (HAS_RELOC | EXEC_P |                /* object flags */
-   HAS_LINENO | HAS_DEBUG |
-   HAS_SYMS | HAS_LOCALS | WP_TEXT | D_PAGED),
-
-#ifndef COFF_WITH_PE
-  (SEC_HAS_CONTENTS | SEC_ALLOC | SEC_LOAD | SEC_RELOC), /* section flags */
-#else
-  (SEC_HAS_CONTENTS | SEC_ALLOC | SEC_LOAD | SEC_RELOC /* section flags */
-   | SEC_LINK_ONCE | SEC_LINK_DUPLICATES),
+#ifndef TARGET_BIG_SYM
+#define TARGET_BIG_SYM armcoff_big_vec
+#endif
+#ifndef TARGET_BIG_NAME
+#define TARGET_BIG_NAME "coff-arm-big"
 #endif
 
-#ifdef TARGET_UNDERSCORE
-  TARGET_UNDERSCORE,           /* leading underscore */
-#else
-  0,                           /* leading underscore */
+#ifndef TARGET_UNDERSCORE
+#define TARGET_UNDERSCORE 0
 #endif
-  '/',                         /* ar_pad_char */
-  15,                          /* ar_max_namelen */
-
-  bfd_getl64, bfd_getl_signed_64, bfd_putl64,
-     bfd_getl32, bfd_getl_signed_32, bfd_putl32,
-     bfd_getl16, bfd_getl_signed_16, bfd_putl16, /* data */
-  bfd_getl64, bfd_getl_signed_64, bfd_putl64,
-     bfd_getl32, bfd_getl_signed_32, bfd_putl32,
-     bfd_getl16, bfd_getl_signed_16, bfd_putl16, /* hdrs */
-
-/* Note that we allow an object file to be treated as a core file as well. */
-    {_bfd_dummy_target, coff_object_p, /* bfd_check_format */
-       bfd_generic_archive_p, coff_object_p},
-    {bfd_false, coff_mkobject, _bfd_generic_mkarchive, /* bfd_set_format */
-       bfd_false},
-    {bfd_false, coff_write_object_contents, /* bfd_write_contents */
-       _bfd_write_archive_contents, bfd_false},
-
-     BFD_JUMP_TABLE_GENERIC (coff),
-     BFD_JUMP_TABLE_COPY (coff),
-     BFD_JUMP_TABLE_CORE (_bfd_nocore),
-     BFD_JUMP_TABLE_ARCHIVE (_bfd_archive_coff),
-     BFD_JUMP_TABLE_SYMBOLS (coff),
-     BFD_JUMP_TABLE_RELOCS (coff),
-     BFD_JUMP_TABLE_WRITE (coff),
-     BFD_JUMP_TABLE_LINK (coff),
-     BFD_JUMP_TABLE_DYNAMIC (_bfd_nodynamic),
-
-  (PTR) & bfd_coff_std_swap_table,
-};
 
-const bfd_target
-#ifdef TARGET_BIG_SYM
-TARGET_BIG_SYM =
+#ifndef EXTRA_S_FLAGS
+#ifdef COFF_WITH_PE
+#define EXTRA_S_FLAGS (SEC_LINK_ONCE | SEC_LINK_DUPLICATES)
 #else
-armcoff_big_vec =
+#define EXTRA_S_FLAGS 0
 #endif
-{
-#ifdef TARGET_BIG_NAME
-  TARGET_BIG_NAME,
-#else
-  "coff-arm-big",
 #endif
-  bfd_target_coff_flavour,
-  BFD_ENDIAN_BIG,              /* data byte order is big */
-  BFD_ENDIAN_BIG,              /* header byte order is big */
 
-  (HAS_RELOC | EXEC_P |                /* object flags */
-   HAS_LINENO | HAS_DEBUG |
-   HAS_SYMS | HAS_LOCALS | WP_TEXT | D_PAGED),
+/* Forward declaration for use initialising alternative_target field.  */
+extern const bfd_target TARGET_BIG_SYM ;
 
-#ifndef COFF_WITH_PE
-  (SEC_HAS_CONTENTS | SEC_ALLOC | SEC_LOAD | SEC_RELOC), /* section flags */
-#else
-  (SEC_HAS_CONTENTS | SEC_ALLOC | SEC_LOAD | SEC_RELOC /* section flags */
-   | SEC_LINK_ONCE | SEC_LINK_DUPLICATES),
-#endif
-
-#ifdef TARGET_UNDERSCORE
-  TARGET_UNDERSCORE,           /* leading underscore */
-#else
-  0,                           /* leading underscore */
-#endif
-  '/',                         /* ar_pad_char */
-  15,                          /* ar_max_namelen */
-
-  bfd_getb64, bfd_getb_signed_64, bfd_putb64,
-     bfd_getb32, bfd_getb_signed_32, bfd_putb32,
-     bfd_getb16, bfd_getb_signed_16, bfd_putb16, /* data */
-  bfd_getb64, bfd_getb_signed_64, bfd_putb64,
-     bfd_getb32, bfd_getb_signed_32, bfd_putb32,
-     bfd_getb16, bfd_getb_signed_16, bfd_putb16, /* hdrs */
-
-/* Note that we allow an object file to be treated as a core file as well. */
-    {_bfd_dummy_target, coff_object_p, /* bfd_check_format */
-       bfd_generic_archive_p, coff_object_p},
-    {bfd_false, coff_mkobject, _bfd_generic_mkarchive, /* bfd_set_format */
-       bfd_false},
-    {bfd_false, coff_write_object_contents, /* bfd_write_contents */
-       _bfd_write_archive_contents, bfd_false},
-
-     BFD_JUMP_TABLE_GENERIC (coff),
-     BFD_JUMP_TABLE_COPY (coff),
-     BFD_JUMP_TABLE_CORE (_bfd_nocore),
-     BFD_JUMP_TABLE_ARCHIVE (_bfd_archive_coff),
-     BFD_JUMP_TABLE_SYMBOLS (coff),
-     BFD_JUMP_TABLE_RELOCS (coff),
-     BFD_JUMP_TABLE_WRITE (coff),
-     BFD_JUMP_TABLE_LINK (coff),
-     BFD_JUMP_TABLE_DYNAMIC (_bfd_nodynamic),
-
-  (PTR) & bfd_coff_std_swap_table,
-};
+/* Target vectors.  */
+CREATE_LITTLE_COFF_TARGET_VEC (TARGET_LITTLE_SYM, TARGET_LITTLE_NAME, D_PAGED, EXTRA_S_FLAGS, TARGET_UNDERSCORE, & TARGET_BIG_SYM)
+CREATE_BIG_COFF_TARGET_VEC (TARGET_BIG_SYM, TARGET_BIG_NAME, D_PAGED, EXTRA_S_FLAGS, TARGET_UNDERSCORE, & TARGET_LITTLE_SYM)
This page took 0.057949 seconds and 4 git commands to generate.