OSDN Git Service

* arm.c (arm_print_operand): Use output_operand_lossage where possible
[pf3gnuchains/gcc-fork.git] / gcc / config / arm / arm.c
index 5731a95..3fe1d28 100644 (file)
@@ -21,7 +21,7 @@
    along with GCC; see the file COPYING.  If not, write to
    the Free Software Foundation, 59 Temple Place - Suite 330,
    Boston, MA 02111-1307, USA.  */
-    
+
 #include "config.h"
 #include "system.h"
 #include "coretypes.h"
@@ -70,6 +70,8 @@ static int arm_legitimate_index_p (enum machine_mode, rtx, RTX_CODE, int);
 static int thumb_base_register_rtx_p (rtx, enum machine_mode, int);
 inline static int thumb_index_register_rtx_p (rtx, int);
 static int thumb_far_jump_used_p (void);
+static bool thumb_force_lr_save (void);
+static unsigned long thumb_compute_save_reg_mask (void);
 static int const_ok_for_op (HOST_WIDE_INT, enum rtx_code);
 static rtx emit_multi_reg_push (int);
 static rtx emit_sfm (int, int);
@@ -125,6 +127,7 @@ static void arm_internal_label (FILE *, const char *, unsigned long);
 static void arm_output_mi_thunk (FILE *, tree, HOST_WIDE_INT, HOST_WIDE_INT,
                                 tree);
 static int arm_rtx_costs_1 (rtx, enum rtx_code, enum rtx_code);
+static bool arm_size_rtx_costs (rtx, int, int, int *);
 static bool arm_slowmul_rtx_costs (rtx, int, int, int *);
 static bool arm_fastmul_rtx_costs (rtx, int, int, int *);
 static bool arm_xscale_rtx_costs (rtx, int, int, int *);
@@ -142,9 +145,6 @@ static rtx arm_expand_unop_builtin (enum insn_code, tree, rtx, int);
 static rtx arm_expand_builtin (tree, rtx, rtx, enum machine_mode, int);
 static void emit_constant_insn (rtx cond, rtx pattern);
 
-#ifdef OBJECT_FORMAT_ELF
-static void arm_elf_asm_named_section (const char *, unsigned int);
-#endif
 #ifndef ARM_PE
 static void arm_encode_section_info (tree, rtx, int);
 #endif
@@ -169,8 +169,10 @@ static bool arm_cxx_guard_mask_bit (void);
 static tree arm_get_cookie_size (tree);
 static bool arm_cookie_has_size (void);
 static bool arm_cxx_cdtor_returns_this (void);
+static bool arm_cxx_key_method_may_be_inline (void);
+static bool arm_cxx_export_class_data (void);
 static void arm_init_libfuncs (void);
-
+static unsigned HOST_WIDE_INT arm_shift_truncation_mask (enum machine_mode);
 \f
 /* Initialize the GCC target structure.  */
 #if TARGET_DLLIMPORT_DECL_ATTRIBUTES
@@ -243,6 +245,11 @@ static void arm_init_libfuncs (void);
 #undef  TARGET_ADDRESS_COST
 #define TARGET_ADDRESS_COST arm_address_cost
 
+#undef TARGET_SHIFT_TRUNCATION_MASK
+#define TARGET_SHIFT_TRUNCATION_MASK arm_shift_truncation_mask
+#undef TARGET_VECTOR_MODE_SUPPORTED_P
+#define TARGET_VECTOR_MODE_SUPPORTED_P arm_vector_mode_supported_p
+
 #undef  TARGET_MACHINE_DEPENDENT_REORG
 #define TARGET_MACHINE_DEPENDENT_REORG arm_reorg
 
@@ -290,6 +297,12 @@ static void arm_init_libfuncs (void);
 #undef TARGET_CXX_CDTOR_RETURNS_THIS
 #define TARGET_CXX_CDTOR_RETURNS_THIS arm_cxx_cdtor_returns_this
 
+#undef TARGET_CXX_KEY_METHOD_MAY_BE_INLINE
+#define TARGET_CXX_KEY_METHOD_MAY_BE_INLINE arm_cxx_key_method_may_be_inline
+
+#undef TARGET_CXX_EXPORT_CLASS_DATA
+#define TARGET_CXX_EXPORT_CLASS_DATA arm_cxx_export_class_data
+
 struct gcc_target targetm = TARGET_INITIALIZER;
 \f
 /* Obstack for minipool constant handling.  */
@@ -374,6 +387,9 @@ int    arm_structure_size_boundary = DEFAULT_STRUCTURE_SIZE_BOUNDARY;
 #define FL_FOR_ARCH5TEJ        FL_FOR_ARCH5TE
 #define FL_FOR_ARCH6   (FL_FOR_ARCH5TE | FL_ARCH6)
 #define FL_FOR_ARCH6J  FL_FOR_ARCH6
+#define FL_FOR_ARCH6K  FL_FOR_ARCH6
+#define FL_FOR_ARCH6Z  FL_FOR_ARCH6
+#define FL_FOR_ARCH6ZK FL_FOR_ARCH6
 
 /* The bits in this mask specify which
    instructions we are allowed to generate.  */
@@ -429,7 +445,7 @@ int arm_is_6_or_7 = 0;
 int thumb_code = 0;
 
 /* Nonzero if we should define __THUMB_INTERWORK__ in the
-   preprocessor.  
+   preprocessor.
    XXX This is a bit of a hack, it's intended to help work around
    problems in GLD which doesn't understand that armv5t code is
    interworking clean.  */
@@ -486,8 +502,8 @@ struct processors
 static const struct processors all_cores[] =
 {
   /* ARM Cores */
-#define ARM_CORE(NAME, ARCH, FLAGS, COSTS) \
-  {#NAME, arm_none, #ARCH, FLAGS | FL_FOR_ARCH##ARCH, arm_##COSTS##_rtx_costs},
+#define ARM_CORE(NAME, IDENT, ARCH, FLAGS, COSTS) \
+  {NAME, arm_none, #ARCH, FLAGS | FL_FOR_ARCH##ARCH, arm_##COSTS##_rtx_costs},
 #include "arm-cores.def"
 #undef ARM_CORE
   {NULL, arm_none, NULL, 0, NULL}
@@ -498,7 +514,7 @@ static const struct processors all_architectures[] =
   /* ARM Architectures */
   /* We don't specify rtx_costs here as it will be figured out
      from the core.  */
-  
+
   {"armv2",   arm2,       "2",   FL_CO_PROC | FL_MODE26 | FL_FOR_ARCH2, NULL},
   {"armv2a",  arm2,       "2",   FL_CO_PROC | FL_MODE26 | FL_FOR_ARCH2, NULL},
   {"armv3",   arm6,       "3",   FL_CO_PROC | FL_MODE26 | FL_FOR_ARCH3, NULL},
@@ -513,6 +529,9 @@ static const struct processors all_architectures[] =
   {"armv5te", arm1026ejs, "5TE", FL_CO_PROC |             FL_FOR_ARCH5TE, NULL},
   {"armv6",   arm1136js,  "6",   FL_CO_PROC |             FL_FOR_ARCH6, NULL},
   {"armv6j",  arm1136js,  "6J",  FL_CO_PROC |             FL_FOR_ARCH6J, NULL},
+  {"armv6k",  mpcore,    "6K",  FL_CO_PROC |             FL_FOR_ARCH6K, NULL},
+  {"armv6z",  arm1176jzs, "6Z",  FL_CO_PROC |             FL_FOR_ARCH6Z, NULL},
+  {"armv6zk", arm1176jzs, "6ZK", FL_CO_PROC |             FL_FOR_ARCH6ZK, NULL},
   {"ep9312",  ep9312,     "4T",  FL_LDSCHED | FL_CIRRUS | FL_FOR_ARCH4, NULL},
   {"iwmmxt",  iwmmxt,     "5TE", FL_LDSCHED | FL_STRONG | FL_FOR_ARCH5TE | FL_XSCALE | FL_IWMMXT , NULL},
   {NULL, arm_none, NULL, 0 , NULL}
@@ -524,7 +543,7 @@ static const struct processors all_architectures[] =
 
 struct arm_cpu_select arm_select[] =
 {
-  /* string      name            processors  */        
+  /* string      name            processors  */
   { NULL,      "-mcpu=",       all_cores  },
   { NULL,      "-march=",      all_architectures },
   { NULL,      "-mtune=",      all_cores }
@@ -608,7 +627,7 @@ static unsigned
 bit_count (unsigned long value)
 {
   unsigned long count = 0;
-  
+
   while (value)
     {
       count++;
@@ -618,7 +637,7 @@ bit_count (unsigned long value)
   return count;
 }
 
-/* Set up library functions uqniue to ARM.  */
+/* Set up library functions unique to ARM.  */
 
 static void
 arm_init_libfuncs (void)
@@ -638,7 +657,7 @@ arm_init_libfuncs (void)
   set_optab_libfunc (neg_optab, DFmode, "__aeabi_dneg");
   set_optab_libfunc (sub_optab, DFmode, "__aeabi_dsub");
 
-  /* Double-precision comparisions.  Table 3.  */
+  /* Double-precision comparisons.  Table 3.  */
   set_optab_libfunc (eq_optab, DFmode, "__aeabi_dcmpeq");
   set_optab_libfunc (ne_optab, DFmode, NULL);
   set_optab_libfunc (lt_optab, DFmode, "__aeabi_dcmplt");
@@ -653,8 +672,8 @@ arm_init_libfuncs (void)
   set_optab_libfunc (smul_optab, SFmode, "__aeabi_fmul");
   set_optab_libfunc (neg_optab, SFmode, "__aeabi_fneg");
   set_optab_libfunc (sub_optab, SFmode, "__aeabi_fsub");
-  
-  /* Single-precision comparisions.  Table 5.  */
+
+  /* Single-precision comparisons.  Table 5.  */
   set_optab_libfunc (eq_optab, SFmode, "__aeabi_fcmpeq");
   set_optab_libfunc (ne_optab, SFmode, NULL);
   set_optab_libfunc (lt_optab, SFmode, "__aeabi_fcmplt");
@@ -677,7 +696,7 @@ arm_init_libfuncs (void)
   set_conv_libfunc (trunc_optab, SFmode, DFmode, "__aeabi_d2f");
   set_conv_libfunc (sext_optab, DFmode, SFmode, "__aeabi_f2d");
 
-  /* Integer to floating-point converisons.  Table 8.  */
+  /* Integer to floating-point conversions.  Table 8.  */
   set_conv_libfunc (sfloat_optab, DFmode, SImode, "__aeabi_i2d");
   set_conv_libfunc (ufloat_optab, DFmode, SImode, "__aeabi_ui2d");
   set_conv_libfunc (sfloat_optab, DFmode, DImode, "__aeabi_l2d");
@@ -727,7 +746,7 @@ arm_override_options (void)
   for (i = ARRAY_SIZE (arm_select); i--;)
     {
       struct arm_cpu_select * ptr = arm_select + i;
-      
+
       if (ptr->string != NULL && ptr->string[0] != '\0')
         {
          const struct processors * sel;
@@ -760,10 +779,10 @@ arm_override_options (void)
                    if (insn_flags != 0 && (insn_flags ^ sel->flags))
                      warning ("switch -mcpu=%s conflicts with -march= switch",
                               ptr->string);
-                   
+
                    insn_flags = sel->flags;
                  }
-               
+
                 break;
               }
 
@@ -771,7 +790,7 @@ arm_override_options (void)
             error ("bad value (%s) for %s switch", ptr->string, ptr->name);
         }
     }
-  
+
   /* If the user did not specify a processor, choose one for them.  */
   if (insn_flags == 0)
     {
@@ -798,18 +817,18 @@ arm_override_options (void)
       /* Now check to see if the user has specified some command line
         switch that require certain abilities from the cpu.  */
       sought = 0;
-      
+
       if (TARGET_INTERWORK || TARGET_THUMB)
        {
          sought |= (FL_THUMB | FL_MODE32);
-         
+
          /* There are no ARM processors that support both APCS-26 and
             interworking.  Therefore we force FL_MODE26 to be removed
             from insn_flags here (if it was set), so that the search
             below will always be able to find a compatible processor.  */
          insn_flags &= ~FL_MODE26;
        }
-      
+
       if (sought != 0 && ((sought & insn_flags) != sought))
        {
          /* Try to locate a CPU type that supports all of the abilities
@@ -823,7 +842,7 @@ arm_override_options (void)
            {
              unsigned current_bit_count = 0;
              const struct processors * best_fit = NULL;
-             
+
              /* Ideally we would like to issue an error message here
                 saying that it was not possible to find a CPU compatible
                 with the default CPU, but which also supports the command
@@ -861,14 +880,17 @@ arm_override_options (void)
       if (arm_tune == arm_none)
        arm_tune = (enum processor_type) (sel - all_cores);
     }
-  
+
   /* The processor for which we should tune should now have been
      chosen.  */
   if (arm_tune == arm_none)
     abort ();
-  
+
   tune_flags = all_cores[(int)arm_tune].flags;
-  targetm.rtx_costs = all_cores[(int)arm_tune].rtx_costs;
+  if (optimize_size)
+    targetm.rtx_costs = arm_size_rtx_costs;
+  else
+    targetm.rtx_costs = all_cores[(int)arm_tune].rtx_costs;
 
   /* Make sure that the processor choice does not conflict with any of the
      other command line choices.  */
@@ -877,7 +899,7 @@ arm_override_options (void)
       warning ("target CPU does not support interworking" );
       target_flags &= ~ARM_FLAG_INTERWORK;
     }
-  
+
   if (TARGET_THUMB && !(insn_flags & FL_THUMB))
     {
       warning ("target CPU does not support THUMB instructions");
@@ -907,16 +929,16 @@ arm_override_options (void)
       warning ("-mapcs-stack-check incompatible with -mno-apcs-frame");
       target_flags |= ARM_FLAG_APCS_FRAME;
     }
-  
+
   if (TARGET_POKE_FUNCTION_NAME)
     target_flags |= ARM_FLAG_APCS_FRAME;
-  
+
   if (TARGET_APCS_REENT && flag_pic)
     error ("-fpic and -mapcs-reent are incompatible");
-  
+
   if (TARGET_APCS_REENT)
     warning ("APCS reentrant code not supported.  Ignored");
-  
+
   /* If this target is normally configured to use APCS frames, warn if they
      are turned off and debugging is turned on.  */
   if (TARGET_ARM
@@ -924,15 +946,15 @@ arm_override_options (void)
       && !TARGET_APCS_FRAME
       && (TARGET_DEFAULT & ARM_FLAG_APCS_FRAME))
     warning ("-g with -mno-apcs-frame may not give sensible debugging");
-  
+
   /* If stack checking is disabled, we can use r10 as the PIC register,
      which keeps r9 available.  */
   if (flag_pic)
     arm_pic_register = TARGET_APCS_STACK ? 9 : 10;
-  
+
   if (TARGET_APCS_FLOAT)
     warning ("passing floating point arguments in fp regs not yet supported");
-  
+
   /* Initialize boolean versions of the flags, for use in the arm.md file.  */
   arm_arch3m = (insn_flags & FL_ARCH3M) != 0;
   arm_arch4 = (insn_flags & FL_ARCH4) != 0;
@@ -1062,12 +1084,13 @@ arm_override_options (void)
        arm_float_abi = ARM_FLOAT_ABI_HARD;
     }
 
-  if (arm_float_abi == ARM_FLOAT_ABI_SOFTFP)
-    sorry ("-mfloat-abi=softfp");
+  if (arm_float_abi == ARM_FLOAT_ABI_HARD && TARGET_VFP)
+    sorry ("-mfloat-abi=hard and VFP");
+
   /* If soft-float is specified then don't use FPU.  */
   if (TARGET_SOFT_FLOAT)
     arm_fpu_arch = FPUTYPE_NONE;
-  
+
   /* For arm2/3 there is no need to do any scheduling if there is only
      a floating point emulator, or we are doing software floating-point.  */
   if ((TARGET_SOFT_FLOAT
@@ -1075,7 +1098,7 @@ arm_override_options (void)
        || arm_fpu_tune == FPUTYPE_FPA_EMU3)
       && (tune_flags & FL_MODE32) == 0)
     flag_schedule_insns = flag_schedule_insns_after_reload = 0;
-  
+
   /* Override the default structure alignment for AAPCS ABI.  */
   if (arm_abi == ARM_ABI_AAPCS)
     arm_structure_size_boundary = 8;
@@ -1222,7 +1245,7 @@ arm_compute_func_type (void)
   unsigned long type = ARM_FT_UNKNOWN;
   tree a;
   tree attr;
-  
+
   if (TREE_CODE (current_function_decl) != FUNCTION_DECL)
     abort ();
 
@@ -1234,12 +1257,12 @@ arm_compute_func_type (void)
       && TREE_NOTHROW (current_function_decl)
       && TREE_THIS_VOLATILE (current_function_decl))
     type |= ARM_FT_VOLATILE;
-  
+
   if (cfun->static_chain_decl != NULL)
     type |= ARM_FT_NESTED;
 
   attr = DECL_ATTRIBUTES (current_function_decl);
-  
+
   a = lookup_attribute ("naked", attr);
   if (a != NULL_TREE)
     type |= ARM_FT_NAKED;
@@ -1247,12 +1270,12 @@ arm_compute_func_type (void)
   a = lookup_attribute ("isr", attr);
   if (a == NULL_TREE)
     a = lookup_attribute ("interrupt", attr);
-  
+
   if (a == NULL_TREE)
     type |= TARGET_INTERWORK ? ARM_FT_INTERWORKED : ARM_FT_NORMAL;
   else
     type |= arm_isr_value (TREE_VALUE (a));
-  
+
   return type;
 }
 
@@ -1267,7 +1290,7 @@ arm_current_func_type (void)
   return cfun->machine->func_type;
 }
 \f
-/* Return 1 if it is possible to return using a single instruction.  
+/* Return 1 if it is possible to return using a single instruction.
    If SIBLING is non-null, this is a test for a return before a sibling
    call.  SIBLING is the call insn, so we can examine its register usage.  */
 
@@ -1320,7 +1343,7 @@ use_return_insn (int iscond, rtx sibling)
      pointer won't be correctly restored if the instruction takes a
      page fault.  We work around this problem by popping r3 along with
      the other registers, since that is never slower than executing
-     another instruction.  
+     another instruction.
 
      We test for !arm_arch5 here, because code for any architecture
      less than this could potentially be run on one of the buggy
@@ -1359,7 +1382,7 @@ use_return_insn (int iscond, rtx sibling)
      taken and multiple registers have been stacked.  */
   if (iscond && arm_is_strong)
     {
-      /* Conditional return when just the LR is stored is a simple 
+      /* Conditional return when just the LR is stored is a simple
         conditional-load instruction, that's not expensive.  */
       if (saved_int_regs != 0 && saved_int_regs != (1 << LR_REGNUM))
        return 0;
@@ -1401,14 +1424,14 @@ const_ok_for_arm (HOST_WIDE_INT i)
 {
   unsigned HOST_WIDE_INT mask = ~(unsigned HOST_WIDE_INT)0xFF;
 
-  /* For machines with >32 bit HOST_WIDE_INT, the bits above bit 31 must 
+  /* For machines with >32 bit HOST_WIDE_INT, the bits above bit 31 must
      be all zero, or all one.  */
   if ((i & ~(unsigned HOST_WIDE_INT) 0xffffffff) != 0
       && ((i & ~(unsigned HOST_WIDE_INT) 0xffffffff)
          != ((~(unsigned HOST_WIDE_INT) 0)
              & ~(unsigned HOST_WIDE_INT) 0xffffffff)))
     return FALSE;
-  
+
   /* Fast return for 0 and powers of 2 */
   if ((i & (i - 1)) == 0)
     return TRUE;
@@ -1487,7 +1510,7 @@ arm_split_constant (enum rtx_code code, enum machine_mode mode, rtx insn,
       */
       if (!after_arm_reorg
          && !cond
-         && (arm_gen_constant (code, mode, NULL_RTX, val, target, source, 
+         && (arm_gen_constant (code, mode, NULL_RTX, val, target, source,
                                1, 0)
              > arm_constant_limit + (code != SET)))
        {
@@ -1516,7 +1539,7 @@ arm_split_constant (enum rtx_code code, enum machine_mode mode, rtx insn,
        }
     }
 
-  return arm_gen_constant (code, mode, cond, val, target, source, subtargets, 
+  return arm_gen_constant (code, mode, cond, val, target, source, subtargets,
                           1);
 }
 
@@ -1528,7 +1551,7 @@ count_insns_for_constant (HOST_WIDE_INT remainder, int i)
   do
     {
       int end;
-         
+
       if (i <= 0)
        i += 32;
       if (remainder & (3 << (i - 2)))
@@ -1674,7 +1697,7 @@ arm_gen_constant (enum rtx_code code, enum machine_mode mode, rtx cond,
        {
          if (generate)
            emit_constant_insn (cond,
-                               gen_rtx_SET (VOIDmode, target, 
+                               gen_rtx_SET (VOIDmode, target,
                                             gen_rtx_MINUS (mode, GEN_INT (val),
                                                            source)));
          return 1;
@@ -1695,7 +1718,7 @@ arm_gen_constant (enum rtx_code code, enum machine_mode mode, rtx cond,
       if (generate)
        emit_constant_insn (cond,
                            gen_rtx_SET (VOIDmode, target,
-                                        (source 
+                                        (source
                                          ? gen_rtx_fmt_ee (code, mode, source,
                                                            GEN_INT (val))
                                          : GEN_INT (val))));
@@ -1745,17 +1768,17 @@ arm_gen_constant (enum rtx_code code, enum machine_mode mode, rtx cond,
       if (set_sign_bit_copies > 1)
        {
          if (const_ok_for_arm
-             (temp1 = ARM_SIGN_EXTEND (remainder 
+             (temp1 = ARM_SIGN_EXTEND (remainder
                                        << (set_sign_bit_copies - 1))))
            {
              if (generate)
                {
                  rtx new_src = subtargets ? gen_reg_rtx (mode) : target;
                  emit_constant_insn (cond,
-                                     gen_rtx_SET (VOIDmode, new_src, 
+                                     gen_rtx_SET (VOIDmode, new_src,
                                                   GEN_INT (temp1)));
                  emit_constant_insn (cond,
-                                     gen_ashrsi3 (target, new_src, 
+                                     gen_ashrsi3 (target, new_src,
                                                   GEN_INT (set_sign_bit_copies - 1)));
                }
              return 2;
@@ -1772,7 +1795,7 @@ arm_gen_constant (enum rtx_code code, enum machine_mode mode, rtx cond,
                                      gen_rtx_SET (VOIDmode, new_src,
                                                   GEN_INT (temp1)));
                  emit_constant_insn (cond,
-                                     gen_ashrsi3 (target, new_src, 
+                                     gen_ashrsi3 (target, new_src,
                                                   GEN_INT (set_sign_bit_copies - 1)));
                }
              return 2;
@@ -1802,7 +1825,7 @@ arm_gen_constant (enum rtx_code code, enum machine_mode mode, rtx cond,
                                            source, subtargets, generate);
                  source = new_src;
                  if (generate)
-                   emit_constant_insn 
+                   emit_constant_insn
                      (cond,
                       gen_rtx_SET
                       (VOIDmode, target,
@@ -1857,10 +1880,10 @@ arm_gen_constant (enum rtx_code code, enum machine_mode mode, rtx cond,
                  rtx sub = subtargets ? gen_reg_rtx (mode) : target;
 
                  emit_constant_insn (cond,
-                                     gen_rtx_SET (VOIDmode, sub, 
+                                     gen_rtx_SET (VOIDmode, sub,
                                                   GEN_INT (val)));
                  emit_constant_insn (cond,
-                                     gen_rtx_SET (VOIDmode, target, 
+                                     gen_rtx_SET (VOIDmode, target,
                                                   gen_rtx_fmt_ee (code, mode,
                                                                   source, sub)));
                }
@@ -1879,14 +1902,14 @@ arm_gen_constant (enum rtx_code code, enum machine_mode mode, rtx cond,
              rtx sub = subtargets ? gen_reg_rtx (mode) : target;
              rtx shift = GEN_INT (set_sign_bit_copies);
 
-             emit_constant_insn 
+             emit_constant_insn
                (cond,
                 gen_rtx_SET (VOIDmode, sub,
-                             gen_rtx_NOT (mode, 
+                             gen_rtx_NOT (mode,
                                           gen_rtx_ASHIFT (mode,
-                                                          source, 
+                                                          source,
                                                           shift))));
-             emit_constant_insn 
+             emit_constant_insn
                (cond,
                 gen_rtx_SET (VOIDmode, target,
                              gen_rtx_NOT (mode,
@@ -1911,7 +1934,7 @@ arm_gen_constant (enum rtx_code code, enum machine_mode mode, rtx cond,
                                           gen_rtx_LSHIFTRT (mode,
                                                             source,
                                                             shift))));
-             emit_constant_insn 
+             emit_constant_insn
                (cond,
                 gen_rtx_SET (VOIDmode, target,
                              gen_rtx_NOT (mode,
@@ -1934,7 +1957,7 @@ arm_gen_constant (enum rtx_code code, enum machine_mode mode, rtx cond,
                sub = gen_reg_rtx (mode);
              emit_constant_insn (cond,
                                  gen_rtx_SET (VOIDmode, sub,
-                                              gen_rtx_AND (mode, source, 
+                                              gen_rtx_AND (mode, source,
                                                            GEN_INT (temp1))));
              emit_constant_insn (cond,
                                  gen_rtx_SET (VOIDmode, target,
@@ -1957,7 +1980,7 @@ arm_gen_constant (enum rtx_code code, enum machine_mode mode, rtx cond,
              if (generate)
                {
                  rtx new_src = subtargets ? gen_reg_rtx (mode) : target;
-                 insns = arm_gen_constant (AND, mode, cond, 
+                 insns = arm_gen_constant (AND, mode, cond,
                                            remainder | shift_mask,
                                            new_src, source, subtargets, 1);
                  source = new_src;
@@ -1986,7 +2009,7 @@ arm_gen_constant (enum rtx_code code, enum machine_mode mode, rtx cond,
       if (clear_zero_bit_copies >= 16 && clear_zero_bit_copies < 24)
        {
          HOST_WIDE_INT shift_mask = (1 << clear_zero_bit_copies) - 1;
-         
+
          if ((remainder | shift_mask) != 0xffffffff)
            {
              if (generate)
@@ -2090,12 +2113,12 @@ arm_gen_constant (enum rtx_code code, enum machine_mode mode, rtx cond,
        rather than having to synthesize both large constants from scratch.
 
        Therefore, we calculate how many insns would be required to emit
-       the constant starting from `best_start', and also starting from 
-       zero (ie with bit 31 first to be output).  If `best_start' doesn't 
+       the constant starting from `best_start', and also starting from
+       zero (i.e. with bit 31 first to be output).  If `best_start' doesn't
        yield a shorter sequence, we may as well use zero.  */
     if (best_start != 0
        && ((((unsigned HOST_WIDE_INT) 1) << best_start) < remainder)
-       && (count_insns_for_constant (remainder, 0) <= 
+       && (count_insns_for_constant (remainder, 0) <=
            count_insns_for_constant (remainder, best_start)))
       best_start = 0;
 
@@ -2149,7 +2172,7 @@ arm_gen_constant (enum rtx_code code, enum machine_mode mode, rtx cond,
                  temp1_rtx = gen_rtx_fmt_ee (code, mode, source, temp1_rtx);
 
                emit_constant_insn (cond,
-                                   gen_rtx_SET (VOIDmode, new_src, 
+                                   gen_rtx_SET (VOIDmode, new_src,
                                                 temp1_rtx));
                source = new_src;
              }
@@ -2244,7 +2267,7 @@ rtx arm_function_value(tree type, tree func ATTRIBUTE_UNUSED)
   int unsignedp ATTRIBUTE_UNUSED;
   rtx r ATTRIBUTE_UNUSED;
 
-  
+
   mode = TYPE_MODE (type);
   /* Promote integer types.  */
   if (INTEGRAL_TYPE_P (type))
@@ -2273,7 +2296,7 @@ arm_return_in_memory (tree type)
         larger than a word (or are variable size).  */
       return (size < 0 || size > UNITS_PER_WORD);
     }
-  
+
   /* For the arm-wince targets we choose to be compatible with Microsoft's
      ARM and Thumb compilers, which always return aggregates in memory.  */
 #ifndef ARM_WINCE
@@ -2283,7 +2306,7 @@ arm_return_in_memory (tree type)
      we will want to return it via memory and not in a register.  */
   if (size < 0 || size > UNITS_PER_WORD)
     return 1;
-  
+
   if (TREE_CODE (type) == RECORD_TYPE)
     {
       tree field;
@@ -2293,14 +2316,14 @@ arm_return_in_memory (tree type)
         has an offset of zero.  For practical purposes this means
         that the structure can have at most one non bit-field element
         and that this element must be the first one in the structure.  */
-      
+
       /* Find the first field, ignoring non FIELD_DECL things which will
         have been created by C++.  */
       for (field = TYPE_FIELDS (type);
           field && TREE_CODE (field) != FIELD_DECL;
           field = TREE_CHAIN (field))
        continue;
-      
+
       if (field == NULL)
        return 0; /* An empty structure.  Allowed by an extension to ANSI C.  */
 
@@ -2323,14 +2346,14 @@ arm_return_in_memory (tree type)
        {
          if (TREE_CODE (field) != FIELD_DECL)
            continue;
-         
+
          if (!DECL_BIT_FIELD_TYPE (field))
            return 1;
        }
 
       return 0;
     }
-  
+
   if (TREE_CODE (type) == UNION_TYPE)
     {
       tree field;
@@ -2346,15 +2369,15 @@ arm_return_in_memory (tree type)
 
          if (FLOAT_TYPE_P (TREE_TYPE (field)))
            return 1;
-         
+
          if (RETURN_IN_MEMORY (TREE_TYPE (field)))
            return 1;
        }
-      
+
       return 0;
     }
-#endif /* not ARM_WINCE */  
-  
+#endif /* not ARM_WINCE */
+
   /* Return all other types in memory.  */
   return 1;
 }
@@ -2385,7 +2408,7 @@ arm_float_words_big_endian (void)
    for a call to a function whose data type is FNTYPE.
    For a library call, FNTYPE is NULL.  */
 void
-arm_init_cumulative_args (CUMULATIVE_ARGS *pcum, tree fntype, 
+arm_init_cumulative_args (CUMULATIVE_ARGS *pcum, tree fntype,
                          rtx libname  ATTRIBUTE_UNUSED,
                          tree fndecl ATTRIBUTE_UNUSED)
 {
@@ -2393,12 +2416,12 @@ arm_init_cumulative_args (CUMULATIVE_ARGS *pcum, tree fntype,
   pcum->nregs = ((fntype && aggregate_value_p (TREE_TYPE (fntype), fntype)) ? 1 : 0);
   pcum->iwmmxt_nregs = 0;
   pcum->can_split = true;
-  
+
   pcum->call_cookie = CALL_NORMAL;
 
   if (TARGET_LONG_CALLS)
     pcum->call_cookie = CALL_LONG;
-    
+
   /* Check for long call/short call attributes.  The attributes
      override any command line option.  */
   if (fntype)
@@ -2460,7 +2483,7 @@ arm_function_arg (CUMULATIVE_ARGS *pcum, enum machine_mode mode,
   /* Varargs vectors are treated the same as long long.
      named_count avoids having to change the way arm handles 'named' */
   if (TARGET_IWMMXT_ABI
-      && VECTOR_MODE_SUPPORTED_P (mode)
+      && arm_vector_mode_supported_p (mode)
       && pcum->named_count > pcum->nargs + 1)
     {
       if (pcum->iwmmxt_nregs <= 9)
@@ -2492,7 +2515,7 @@ arm_function_arg (CUMULATIVE_ARGS *pcum, enum machine_mode mode,
 
   if (!named || pcum->nregs + nregs > NUM_ARG_REGS)
     return NULL_RTX;
-  
+
   return gen_rtx_REG (mode, pcum->nregs);
 }
 
@@ -2546,7 +2569,7 @@ const struct attribute_spec arm_attribute_table[] =
   /* Whereas these functions are always known to reside within the 26 bit
      addressing range.  */
   { "short_call",   0, 0, false, true,  true,  NULL },
-  /* Interrupt Service Routines have special prologue and epilogue requirements.  */ 
+  /* Interrupt Service Routines have special prologue and epilogue requirements.  */
   { "isr",          0, 1, false, false, false, arm_handle_isr_attribute },
   { "interrupt",    0, 1, false, false, false, arm_handle_isr_attribute },
   { "naked",        0, 0, true,  false, false, arm_handle_fndecl_attribute },
@@ -2619,7 +2642,7 @@ arm_handle_isr_attribute (tree *node, tree name, tree args, int flags,
                   || TREE_CODE (TREE_TYPE (*node)) == METHOD_TYPE)
               && arm_isr_value (args) != ARM_FT_UNKNOWN)
        {
-         *node = build_type_copy (*node);
+         *node = build_variant_type_copy (*node);
          TREE_TYPE (*node) = build_type_attribute_variant
            (TREE_TYPE (*node),
             tree_cons (name, args, TYPE_ATTRIBUTES (TREE_TYPE (*node))));
@@ -2652,7 +2675,7 @@ static int
 arm_comp_type_attributes (tree type1, tree type2)
 {
   int l1, l2, s1, s2;
-  
+
   /* Check for mismatch of non-default calling convention.  */
   if (TREE_CODE (type1) != FUNCTION_TYPE)
     return 1;
@@ -2674,7 +2697,7 @@ arm_comp_type_attributes (tree type1, tree type2)
       if ((l1 & s2) || (l2 & s1))
        return 0;
     }
-  
+
   /* Check for mismatched ISR attribute.  */
   l1 = lookup_attribute ("isr", TYPE_ATTRIBUTES (type1)) != NULL;
   if (! l1)
@@ -2770,11 +2793,11 @@ current_file_function_operand (rtx sym_ref)
             or 3. the target function has __attribute__ ((section))
 
    However we do not generate a long call if the function:
-   
+
         d.  has an __attribute__ ((short_call))
      or e.  is inside the scope of a #pragma no_long_calls
      or f.  is defined within the current compilation unit.
-   
+
    This function will be called by C fragments contained in the machine
    description file.  SYM_REF and CALL_COOKIE correspond to the matched
    rtl operands.  CALL_SYMBOL is used to distinguish between
@@ -2803,14 +2826,14 @@ arm_is_longcall_p (rtx sym_ref, int call_cookie, int call_symbol)
     {
       if (flag_function_sections
          || DECL_SECTION_NAME (current_function_decl))
-       /* c.3 is handled by the defintion of the
+       /* c.3 is handled by the definition of the
           ARM_DECLARE_FUNCTION_SIZE macro.  */
        return 1;
     }
 
   if (current_file_function_operand (sym_ref))
     return 0;
-  
+
   return (call_cookie & CALL_LONG)
     || ENCODED_LONG_CALL_ATTR_P (XSTR (sym_ref, 0))
     || TARGET_LONG_CALLS;
@@ -2843,7 +2866,7 @@ arm_function_ok_for_sibcall (tree decl, tree exp ATTRIBUTE_UNUSED)
     return false;
 
   /* If we are interworking and the function is not declared static
-     then we can't tail-call it unless we know that it exists in this 
+     then we can't tail-call it unless we know that it exists in this
      compilation unit (since it might be a Thumb routine).  */
   if (TARGET_INTERWORK && TREE_PUBLIC (decl) && !TREE_ASM_WRITTEN (decl))
     return false;
@@ -2913,16 +2936,15 @@ legitimize_pic_address (rtx orig, enum machine_mode mode, rtx reg)
        emit_insn (gen_pic_load_addr_thumb (address, orig));
 
       if ((GET_CODE (orig) == LABEL_REF
-          || (GET_CODE (orig) == SYMBOL_REF && 
+          || (GET_CODE (orig) == SYMBOL_REF &&
               SYMBOL_REF_LOCAL_P (orig)))
          && NEED_GOT_RELOC)
        pic_ref = gen_rtx_PLUS (Pmode, pic_offset_table_rtx, address);
       else
        {
-         pic_ref = gen_rtx_MEM (Pmode,
-                                gen_rtx_PLUS (Pmode, pic_offset_table_rtx,
-                                              address));
-         RTX_UNCHANGING_P (pic_ref) = 1;
+         pic_ref = gen_const_mem (Pmode,
+                                  gen_rtx_PLUS (Pmode, pic_offset_table_rtx,
+                                                address));
        }
 
       insn = emit_move_insn (reg, pic_ref);
@@ -2989,16 +3011,35 @@ legitimize_pic_address (rtx orig, enum machine_mode mode, rtx reg)
   return orig;
 }
 
-/* Generate code to load the PIC register.  PROLOGUE is true if
-   called from arm_expand_prologue (in which case we want the 
-   generated insns at the start of the function);  false if called
-   by an exception receiver that needs the PIC register reloaded
-   (in which case the insns are just dumped at the current location).  */
+
+/* Find a spare low register.  */
+
+static int
+thumb_find_work_register (int live_regs_mask)
+{
+  int reg;
+
+  /* Use a spare arg register.  */
+  if (!regs_ever_live[LAST_ARG_REGNUM])
+    return LAST_ARG_REGNUM;
+
+  /* Look for a pushed register.  */
+  for (reg = LAST_LO_REGNUM; reg >=0; reg--)
+    if (live_regs_mask & (1 << reg))
+      return reg;
+
+  /* Something went wrong.  */
+  abort ();
+}
+
+
+/* Generate code to load the PIC register.  */
+
 void
-arm_finalize_pic (int prologue ATTRIBUTE_UNUSED)
+arm_load_pic_register (void)
 {
 #ifndef AOF_ASSEMBLER
-  rtx l1, pic_tmp, pic_tmp2, seq, pic_rtx;
+  rtx l1, pic_tmp, pic_tmp2, pic_rtx;
   rtx global_offset_table;
 
   if (current_function_uses_pic_offset_table == 0 || TARGET_SINGLE_PIC_BASE)
@@ -3007,7 +3048,6 @@ arm_finalize_pic (int prologue ATTRIBUTE_UNUSED)
   if (!flag_pic)
     abort ();
 
-  start_sequence ();
   l1 = gen_label_rtx ();
 
   global_offset_table = gen_rtx_SYMBOL_REF (Pmode, "_GLOBAL_OFFSET_TABLE_");
@@ -3021,7 +3061,7 @@ arm_finalize_pic (int prologue ATTRIBUTE_UNUSED)
     pic_tmp2 = gen_rtx_CONST (VOIDmode, global_offset_table);
 
   pic_rtx = gen_rtx_CONST (Pmode, gen_rtx_MINUS (Pmode, pic_tmp2, pic_tmp));
-  
+
   if (TARGET_ARM)
     {
       emit_insn (gen_pic_load_addr_arm (pic_offset_table_rtx, pic_rtx));
@@ -3029,23 +3069,29 @@ arm_finalize_pic (int prologue ATTRIBUTE_UNUSED)
     }
   else
     {
-      emit_insn (gen_pic_load_addr_thumb (pic_offset_table_rtx, pic_rtx));
+      if (REGNO (pic_offset_table_rtx) > LAST_LO_REGNUM)
+       {
+         int reg;
+
+         /* We will have pushed the pic register, so should always be
+            able to find a work register.  */
+         reg = thumb_find_work_register (thumb_compute_save_reg_mask ());
+         pic_tmp = gen_rtx_REG (SImode, reg);
+         emit_insn (gen_pic_load_addr_thumb (pic_tmp, pic_rtx));
+         emit_insn (gen_movsi (pic_offset_table_rtx, pic_tmp));
+       }
+      else
+       emit_insn (gen_pic_load_addr_thumb (pic_offset_table_rtx, pic_rtx));
       emit_insn (gen_pic_add_dot_plus_four (pic_offset_table_rtx, l1));
     }
 
-  seq = get_insns ();
-  end_sequence ();
-  if (prologue)
-    emit_insn_after (seq, get_insns ());
-  else
-    emit_insn (seq);
-
   /* Need to emit this whether or not we obey regdecls,
      since setjmp/longjmp can cause life info to screw up.  */
   emit_insn (gen_rtx_USE (VOIDmode, pic_offset_table_rtx));
 #endif /* AOF_ASSEMBLER */
 }
 
+
 /* Return nonzero if X is valid as an ARM state addressing register.  */
 static int
 arm_address_register_rtx_p (rtx x, int strict_p)
@@ -3073,7 +3119,7 @@ arm_legitimate_address_p (enum machine_mode mode, rtx x, RTX_CODE outer,
 {
   bool use_ldrd;
   enum rtx_code code = GET_CODE (x);
-  
+
   if (arm_address_register_rtx_p (x, strict_p))
     return 1;
 
@@ -3093,7 +3139,7 @@ arm_legitimate_address_p (enum machine_mode mode, rtx x, RTX_CODE outer,
     {
       rtx addend = XEXP (XEXP (x, 1), 1);
 
-      /* Don't allow ldrd post increment by register becuase it's hard
+      /* Don't allow ldrd post increment by register because it's hard
         to fixup invalid register choices.  */
       if (use_ldrd
          && GET_CODE (x) == POST_MODIFY
@@ -3270,7 +3316,7 @@ thumb_index_register_rtx_p (rtx x, int strict_p)
 }
 
 /* Return nonzero if x is a legitimate Thumb-state address.
+
    The AP may be eliminated to either the SP or the FP, so we use the
    least common denominator, e.g. SImode, and offsets from 0 to 64.
 
@@ -3570,60 +3616,60 @@ thumb_rtx_costs (rtx x, enum rtx_code code, enum rtx_code outer)
     case ASHIFT:
     case ASHIFTRT:
     case LSHIFTRT:
-    case ROTATERT:     
+    case ROTATERT:
     case PLUS:
     case MINUS:
     case COMPARE:
     case NEG:
-    case NOT:  
+    case NOT:
       return COSTS_N_INSNS (1);
-      
-    case MULT:                                                 
-      if (GET_CODE (XEXP (x, 1)) == CONST_INT)                 
-       {                                                               
-         int cycles = 0;                                               
+
+    case MULT:
+      if (GET_CODE (XEXP (x, 1)) == CONST_INT)
+       {
+         int cycles = 0;
          unsigned HOST_WIDE_INT i = INTVAL (XEXP (x, 1));
-         
-         while (i)                                             
-           {                                                   
-             i >>= 2;                                          
-             cycles++;                                         
-           }                                                   
-         return COSTS_N_INSNS (2) + cycles;                    
+
+         while (i)
+           {
+             i >>= 2;
+             cycles++;
+           }
+         return COSTS_N_INSNS (2) + cycles;
        }
       return COSTS_N_INSNS (1) + 16;
-      
-    case SET:                                                  
-      return (COSTS_N_INSNS (1)                                        
-             + 4 * ((GET_CODE (SET_SRC (x)) == MEM)            
+
+    case SET:
+      return (COSTS_N_INSNS (1)
+             + 4 * ((GET_CODE (SET_SRC (x)) == MEM)
                     + GET_CODE (SET_DEST (x)) == MEM));
-      
-    case CONST_INT:                                            
-      if (outer == SET)                                                
-       {                                                       
-         if ((unsigned HOST_WIDE_INT) INTVAL (x) < 256)                
-           return 0;                                           
-         if (thumb_shiftable_const (INTVAL (x)))                       
-           return COSTS_N_INSNS (2);                           
-         return COSTS_N_INSNS (3);                             
-       }                                                               
+
+    case CONST_INT:
+      if (outer == SET)
+       {
+         if ((unsigned HOST_WIDE_INT) INTVAL (x) < 256)
+           return 0;
+         if (thumb_shiftable_const (INTVAL (x)))
+           return COSTS_N_INSNS (2);
+         return COSTS_N_INSNS (3);
+       }
       else if ((outer == PLUS || outer == COMPARE)
-              && INTVAL (x) < 256 && INTVAL (x) > -256)                
+              && INTVAL (x) < 256 && INTVAL (x) > -256)
        return 0;
       else if (outer == AND
               && INTVAL (x) < 256 && INTVAL (x) >= -256)
        return COSTS_N_INSNS (1);
-      else if (outer == ASHIFT || outer == ASHIFTRT            
-              || outer == LSHIFTRT)                            
-       return 0;                                                       
+      else if (outer == ASHIFT || outer == ASHIFTRT
+              || outer == LSHIFTRT)
+       return 0;
       return COSTS_N_INSNS (2);
-      
-    case CONST:                                                        
-    case CONST_DOUBLE:                                         
-    case LABEL_REF:                                            
-    case SYMBOL_REF:                                           
+
+    case CONST:
+    case CONST_DOUBLE:
+    case LABEL_REF:
+    case SYMBOL_REF:
       return COSTS_N_INSNS (3);
-      
+
     case UDIV:
     case UMOD:
     case DIV:
@@ -3635,7 +3681,7 @@ thumb_rtx_costs (rtx x, enum rtx_code code, enum rtx_code outer)
 
     case AND:
     case XOR:
-    case IOR: 
+    case IOR:
       /* XXX guess.  */
       return 8;
 
@@ -3660,18 +3706,18 @@ thumb_rtx_costs (rtx x, enum rtx_code code, enum rtx_code outer)
        case QImode:
          return (1 + (mode == DImode ? 4 : 0)
                  + (GET_CODE (XEXP (x, 0)) == MEM ? 10 : 0));
-         
+
        case HImode:
          return (4 + (mode == DImode ? 4 : 0)
                  + (GET_CODE (XEXP (x, 0)) == MEM ? 10 : 0));
-         
+
        case SImode:
          return (1 + (GET_CODE (XEXP (x, 0)) == MEM ? 10 : 0));
-      
+
        default:
          return 99;
        }
-      
+
     default:
       return 99;
     }
@@ -3712,7 +3758,7 @@ arm_rtx_costs_1 (rtx x, enum rtx_code code, enum rtx_code outer)
     case ASHIFT: case LSHIFTRT: case ASHIFTRT:
       if (mode == DImode)
        return (8 + (GET_CODE (XEXP (x, 1)) == CONST_INT ? 0 : 8)
-               + ((GET_CODE (XEXP (x, 0)) == REG 
+               + ((GET_CODE (XEXP (x, 0)) == REG
                    || (GET_CODE (XEXP (x, 0)) == SUBREG
                        && GET_CODE (SUBREG_REG (XEXP (x, 0))) == REG))
                   ? 0 : 8));
@@ -3761,7 +3807,7 @@ arm_rtx_costs_1 (rtx x, enum rtx_code code, enum rtx_code outer)
        return 1;
       /* Fall through */
 
-    case PLUS: 
+    case PLUS:
       if (GET_MODE_CLASS (mode) == MODE_FLOAT)
        return (2 + (REG_OR_SUBREG_REG (XEXP (x, 0)) ? 0 : 8)
                + ((REG_OR_SUBREG_REG (XEXP (x, 1))
@@ -3770,7 +3816,7 @@ arm_rtx_costs_1 (rtx x, enum rtx_code code, enum rtx_code outer)
                   ? 0 : 8));
 
       /* Fall through */
-    case AND: case XOR: case IOR: 
+    case AND: case XOR: case IOR:
       extra_cost = 0;
 
       /* Normally the frame registers will be spilt into reg+const during
@@ -3882,37 +3928,258 @@ arm_rtx_costs_1 (rtx x, enum rtx_code code, enum rtx_code outer)
        }
       abort ();
 
-    case CONST_INT:                                            
-      if (const_ok_for_arm (INTVAL (x)))                       
-       return outer == SET ? 2 : -1;                   
-      else if (outer == AND                            
-              && const_ok_for_arm (~INTVAL (x)))               
-       return -1;                                              
-      else if ((outer == COMPARE                       
-               || outer == PLUS || outer == MINUS)     
-              && const_ok_for_arm (-INTVAL (x)))               
-       return -1;                                              
-      else                                                     
+    case CONST_INT:
+      if (const_ok_for_arm (INTVAL (x)))
+       return outer == SET ? 2 : -1;
+      else if (outer == AND
+              && const_ok_for_arm (~INTVAL (x)))
+       return -1;
+      else if ((outer == COMPARE
+               || outer == PLUS || outer == MINUS)
+              && const_ok_for_arm (-INTVAL (x)))
+       return -1;
+      else
        return 5;
-      
-    case CONST:                                                        
-    case LABEL_REF:                                            
-    case SYMBOL_REF:                                           
+
+    case CONST:
+    case LABEL_REF:
+    case SYMBOL_REF:
       return 6;
-      
-    case CONST_DOUBLE:                                         
+
+    case CONST_DOUBLE:
       if (arm_const_double_rtx (x))
-       return outer == SET ? 2 : -1;                   
-      else if ((outer == COMPARE || outer == PLUS)     
-              && neg_const_double_rtx_ok_for_fpa (x))          
-       return -1;                                              
+       return outer == SET ? 2 : -1;
+      else if ((outer == COMPARE || outer == PLUS)
+              && neg_const_double_rtx_ok_for_fpa (x))
+       return -1;
       return 7;
-      
+
     default:
       return 99;
     }
 }
 
+/* RTX costs when optimizing for size.  */
+static bool
+arm_size_rtx_costs (rtx x, int code, int outer_code, int *total)
+{
+  enum machine_mode mode = GET_MODE (x);
+
+  if (TARGET_THUMB)
+    {
+      /* XXX TBD.  For now, use the standard costs.  */
+      *total = thumb_rtx_costs (x, code, outer_code);
+      return true;
+    }
+
+  switch (code)
+    {
+    case MEM:
+      /* A memory access costs 1 insn if the mode is small, or the address is
+        a single register, otherwise it costs one insn per word.  */
+      if (REG_P (XEXP (x, 0)))
+       *total = COSTS_N_INSNS (1);
+      else
+       *total = COSTS_N_INSNS (ARM_NUM_REGS (mode));
+      return true;
+
+    case DIV:
+    case MOD:
+    case UDIV:
+    case UMOD:
+      /* Needs a libcall, so it costs about this.  */
+      *total = COSTS_N_INSNS (2);
+      return false;
+
+    case ROTATE:
+      if (mode == SImode && GET_CODE (XEXP (x, 1)) == REG)
+       {
+         *total = COSTS_N_INSNS (2) + rtx_cost (XEXP (x, 0), code);
+         return true;
+       }
+      /* Fall through */
+    case ROTATERT:
+    case ASHIFT:
+    case LSHIFTRT:
+    case ASHIFTRT:
+      if (mode == DImode && GET_CODE (XEXP (x, 1)) == CONST_INT)
+       {
+         *total = COSTS_N_INSNS (3) + rtx_cost (XEXP (x, 0), code);
+         return true;
+       }
+      else if (mode == SImode)
+       {
+         *total = COSTS_N_INSNS (1) + rtx_cost (XEXP (x, 0), code);
+         /* Slightly disparage register shifts, but not by much.  */
+         if (GET_CODE (XEXP (x, 1)) != CONST_INT)
+           *total += 1 + rtx_cost (XEXP (x, 1), code);
+         return true;
+       }
+
+      /* Needs a libcall.  */
+      *total = COSTS_N_INSNS (2);
+      return false;
+
+    case MINUS:
+      if (TARGET_HARD_FLOAT && GET_MODE_CLASS (mode) == MODE_FLOAT)
+       {
+         *total = COSTS_N_INSNS (1);
+         return false;
+       }
+
+      if (mode == SImode)
+       {
+         enum rtx_code subcode0 = GET_CODE (XEXP (x, 0));
+         enum rtx_code subcode1 = GET_CODE (XEXP (x, 1));
+
+         if (subcode0 == ROTATE || subcode0 == ROTATERT || subcode0 == ASHIFT
+             || subcode0 == LSHIFTRT || subcode0 == ASHIFTRT
+             || subcode1 == ROTATE || subcode1 == ROTATERT
+             || subcode1 == ASHIFT || subcode1 == LSHIFTRT
+             || subcode1 == ASHIFTRT)
+           {
+             /* It's just the cost of the two operands.  */
+             *total = 0;
+             return false;
+           }
+
+         *total = COSTS_N_INSNS (1);
+         return false;
+       }
+
+      *total = COSTS_N_INSNS (ARM_NUM_REGS (mode));
+      return false;
+
+    case PLUS:
+      if (TARGET_HARD_FLOAT && GET_MODE_CLASS (mode) == MODE_FLOAT)
+       {
+         *total = COSTS_N_INSNS (1);
+         return false;
+       }
+
+      /* Fall through */
+    case AND: case XOR: case IOR:
+      if (mode == SImode)
+       {
+         enum rtx_code subcode = GET_CODE (XEXP (x, 0));
+
+         if (subcode == ROTATE || subcode == ROTATERT || subcode == ASHIFT
+             || subcode == LSHIFTRT || subcode == ASHIFTRT
+             || (code == AND && subcode == NOT))
+           {
+             /* It's just the cost of the two operands.  */
+             *total = 0;
+             return false;
+           }
+       }
+
+      *total = COSTS_N_INSNS (ARM_NUM_REGS (mode));
+      return false;
+
+    case MULT:
+      *total = COSTS_N_INSNS (ARM_NUM_REGS (mode));
+      return false;
+
+    case NEG:
+      if (TARGET_HARD_FLOAT && GET_MODE_CLASS (mode) == MODE_FLOAT)
+       *total = COSTS_N_INSNS (1);
+      /* Fall through */
+    case NOT:
+      *total = COSTS_N_INSNS (ARM_NUM_REGS (mode));
+
+      return false;
+
+    case IF_THEN_ELSE:
+      *total = 0;
+      return false;
+
+    case COMPARE:
+      if (cc_register (XEXP (x, 0), VOIDmode))
+       * total = 0;
+      else
+       *total = COSTS_N_INSNS (1);
+      return false;
+
+    case ABS:
+      if (TARGET_HARD_FLOAT && GET_MODE_CLASS (mode) == MODE_FLOAT)
+       *total = COSTS_N_INSNS (1);
+      else
+       *total = COSTS_N_INSNS (1 + ARM_NUM_REGS (mode));
+      return false;
+
+    case SIGN_EXTEND:
+      *total = 0;
+      if (GET_MODE_SIZE (GET_MODE (XEXP (x, 0))) < 4)
+       {
+         if (!(arm_arch4 && MEM_P (XEXP (x, 0))))
+           *total += COSTS_N_INSNS (arm_arch6 ? 1 : 2);
+       }
+      if (mode == DImode)
+       *total += COSTS_N_INSNS (1);
+      return false;
+
+    case ZERO_EXTEND:
+      *total = 0;
+      if (!(arm_arch4 && MEM_P (XEXP (x, 0))))
+       {
+         switch (GET_MODE (XEXP (x, 0)))
+           {
+           case QImode:
+             *total += COSTS_N_INSNS (1);
+             break;
+
+           case HImode:
+             *total += COSTS_N_INSNS (arm_arch6 ? 1 : 2);
+
+           case SImode:
+             break;
+
+           default:
+             *total += COSTS_N_INSNS (2);
+           }
+       }
+
+      if (mode == DImode)
+       *total += COSTS_N_INSNS (1);
+
+      return false;
+
+    case CONST_INT:
+      if (const_ok_for_arm (INTVAL (x)))
+       *total = COSTS_N_INSNS (outer_code == SET ? 1 : 0);
+      else if (const_ok_for_arm (~INTVAL (x)))
+       *total = COSTS_N_INSNS (outer_code == AND ? 0 : 1);
+      else if (const_ok_for_arm (-INTVAL (x)))
+       {
+         if (outer_code == COMPARE || outer_code == PLUS
+             || outer_code == MINUS)
+           *total = 0;
+         else
+           *total = COSTS_N_INSNS (1);
+       }
+      else
+       *total = COSTS_N_INSNS (2);
+      return true;
+
+    case CONST:
+    case LABEL_REF:
+    case SYMBOL_REF:
+      *total = COSTS_N_INSNS (2);
+      return true;
+
+    case CONST_DOUBLE:
+      *total = COSTS_N_INSNS (4);
+      return true;
+
+    default:
+      if (mode != VOIDmode)
+       *total = COSTS_N_INSNS (ARM_NUM_REGS (mode));
+      else
+       *total = COSTS_N_INSNS (4); /* How knows?  */
+      return false;
+    }
+}
+
 /* RTX costs for cores with a slow MUL implementation.  */
 
 static bool
@@ -3925,7 +4192,7 @@ arm_slowmul_rtx_costs (rtx x, int code, int outer_code, int *total)
       *total = thumb_rtx_costs (x, code, outer_code);
       return true;
     }
-  
+
   switch (code)
     {
     case MULT:
@@ -3943,7 +4210,7 @@ arm_slowmul_rtx_costs (rtx x, int code, int outer_code, int *total)
          int cost, const_ok = const_ok_for_arm (i);
          int j, booth_unit_size;
 
-         /* Tune as appropriate.  */ 
+         /* Tune as appropriate.  */
          cost = const_ok ? 4 : 8;
          booth_unit_size = 2;
          for (j = 0; i && j < 32; j += booth_unit_size)
@@ -3959,7 +4226,7 @@ arm_slowmul_rtx_costs (rtx x, int code, int outer_code, int *total)
       *total = 30 + (REG_OR_SUBREG_REG (XEXP (x, 0)) ? 0 : 4)
                  + (REG_OR_SUBREG_REG (XEXP (x, 1)) ? 0 : 4);
       return true;
-  
+
     default:
       *total = arm_rtx_costs_1 (x, code, outer_code);
       return true;
@@ -3979,7 +4246,7 @@ arm_fastmul_rtx_costs (rtx x, int code, int outer_code, int *total)
       *total = thumb_rtx_costs (x, code, outer_code);
       return true;
     }
-  
+
   switch (code)
     {
     case MULT:
@@ -3993,7 +4260,7 @@ arm_fastmul_rtx_costs (rtx x, int code, int outer_code, int *total)
          *total = 8;
          return true;
        }
-      
+
 
       if (GET_MODE_CLASS (mode) == MODE_FLOAT
          || mode == DImode)
@@ -4009,7 +4276,7 @@ arm_fastmul_rtx_costs (rtx x, int code, int outer_code, int *total)
          int cost, const_ok = const_ok_for_arm (i);
          int j, booth_unit_size;
 
-         /* Tune as appropriate.  */ 
+         /* Tune as appropriate.  */
          cost = const_ok ? 4 : 8;
          booth_unit_size = 8;
          for (j = 0; i && j < 32; j += booth_unit_size)
@@ -4025,7 +4292,7 @@ arm_fastmul_rtx_costs (rtx x, int code, int outer_code, int *total)
       *total = 8 + (REG_OR_SUBREG_REG (XEXP (x, 0)) ? 0 : 4)
                 + (REG_OR_SUBREG_REG (XEXP (x, 1)) ? 0 : 4);
       return true;
-  
+
     default:
       *total = arm_rtx_costs_1 (x, code, outer_code);
       return true;
@@ -4045,7 +4312,7 @@ arm_xscale_rtx_costs (rtx x, int code, int outer_code, int *total)
       *total = thumb_rtx_costs (x, code, outer_code);
       return true;
     }
-  
+
   switch (code)
     {
     case MULT:
@@ -4059,7 +4326,7 @@ arm_xscale_rtx_costs (rtx x, int code, int outer_code, int *total)
          *total = 8;
          return true;
        }
-      
+
 
       if (GET_MODE_CLASS (mode) == MODE_FLOAT
          || mode == DImode)
@@ -4097,7 +4364,7 @@ arm_xscale_rtx_costs (rtx x, int code, int outer_code, int *total)
       *total = 8 + (REG_OR_SUBREG_REG (XEXP (x, 0)) ? 0 : 4)
                 + (REG_OR_SUBREG_REG (XEXP (x, 1)) ? 0 : 4);
       return true;
-  
+
     default:
       *total = arm_rtx_costs_1 (x, code, outer_code);
       return true;
@@ -4113,7 +4380,7 @@ arm_9e_rtx_costs (rtx x, int code, int outer_code, int *total)
   enum machine_mode mode = GET_MODE (x);
   int nonreg_cost;
   int cost;
-  
+
   if (TARGET_THUMB)
     {
       switch (code)
@@ -4121,13 +4388,13 @@ arm_9e_rtx_costs (rtx x, int code, int outer_code, int *total)
        case MULT:
          *total = COSTS_N_INSNS (3);
          return true;
-         
+
        default:
          *total = thumb_rtx_costs (x, code, outer_code);
          return true;
        }
     }
-  
+
   switch (code)
     {
     case MULT:
@@ -4141,7 +4408,7 @@ arm_9e_rtx_costs (rtx x, int code, int outer_code, int *total)
          *total = 3;
          return true;
        }
-      
+
 
       if (GET_MODE_CLASS (mode) == MODE_FLOAT)
        {
@@ -4163,7 +4430,7 @@ arm_9e_rtx_costs (rtx x, int code, int outer_code, int *total)
       *total = cost + (REG_OR_SUBREG_REG (XEXP (x, 0)) ? 0 : nonreg_cost)
                    + (REG_OR_SUBREG_REG (XEXP (x, 1)) ? 0 : nonreg_cost);
       return true;
-  
+
     default:
       *total = arm_rtx_costs_1 (x, code, outer_code);
       return true;
@@ -4242,7 +4509,7 @@ arm_adjust_cost (rtx insn, rtx link, rtx dep, int cost)
        {
          rtx shifted_operand;
          int opno;
-         
+
          /* Get the shifted operand.  */
          extract_insn (insn);
          shifted_operand = recog_data.operand[shift_opnum];
@@ -4283,9 +4550,9 @@ arm_adjust_cost (rtx insn, rtx link, rtx dep, int cost)
       rtx src_mem = XEXP (SET_SRC (i_pat), 0);
       /* This is a load after a store, there is no conflict if the load reads
         from a cached area.  Assume that loads from the stack, and from the
-        constant pool are cached, and that others will miss.  This is a 
+        constant pool are cached, and that others will miss.  This is a
         hack.  */
-      
+
       if ((GET_CODE (src_mem) == SYMBOL_REF && CONSTANT_POOL_ADDRESS_P (src_mem))
          || reg_mentioned_p (stack_pointer_rtx, src_mem)
          || reg_mentioned_p (frame_pointer_rtx, src_mem)
@@ -4331,10 +4598,10 @@ arm_const_double_rtx (rtx x)
 {
   REAL_VALUE_TYPE r;
   int i;
-  
+
   if (!fp_consts_inited)
     init_fp_table ();
-  
+
   REAL_VALUE_FROM_CONST_DOUBLE (r, x);
   if (REAL_VALUE_MINUS_ZERO (r))
     return 0;
@@ -4352,10 +4619,10 @@ neg_const_double_rtx_ok_for_fpa (rtx x)
 {
   REAL_VALUE_TYPE r;
   int i;
-  
+
   if (!fp_consts_inited)
     init_fp_table ();
-  
+
   REAL_VALUE_FROM_CONST_DOUBLE (r, x);
   r = REAL_VALUE_NEGATE (r);
   if (REAL_VALUE_MINUS_ZERO (r))
@@ -4370,243 +4637,6 @@ neg_const_double_rtx_ok_for_fpa (rtx x)
 \f
 /* Predicates for `match_operand' and `match_operator'.  */
 
-/* s_register_operand is the same as register_operand, but it doesn't accept
-   (SUBREG (MEM)...).
-
-   This function exists because at the time it was put in it led to better
-   code.  SUBREG(MEM) always needs a reload in the places where
-   s_register_operand is used, and this seemed to lead to excessive
-   reloading.  */
-int
-s_register_operand (rtx op, enum machine_mode mode)
-{
-  if (GET_MODE (op) != mode && mode != VOIDmode)
-    return 0;
-
-  if (GET_CODE (op) == SUBREG)
-    op = SUBREG_REG (op);
-
-  /* We don't consider registers whose class is NO_REGS
-     to be a register operand.  */
-  /* XXX might have to check for lo regs only for thumb ??? */
-  return (GET_CODE (op) == REG
-         && (REGNO (op) >= FIRST_PSEUDO_REGISTER
-             || REGNO_REG_CLASS (REGNO (op)) != NO_REGS));
-}
-
-/* A hard register operand (even before reload.  */
-int
-arm_hard_register_operand (rtx op, enum machine_mode mode)
-{
-  if (GET_MODE (op) != mode && mode != VOIDmode)
-    return 0;
-
-  return (GET_CODE (op) == REG
-         && REGNO (op) < FIRST_PSEUDO_REGISTER);
-}
-    
-/* An arm register operand.  */
-int
-arm_general_register_operand (rtx op, enum machine_mode mode)
-{
-  if (GET_MODE (op) != mode && mode != VOIDmode)
-    return 0;
-
-  if (GET_CODE (op) == SUBREG)
-    op = SUBREG_REG (op);
-
-  return (GET_CODE (op) == REG
-         && (REGNO (op) <= LAST_ARM_REGNUM
-             || REGNO (op) >= FIRST_PSEUDO_REGISTER));
-}
-
-/* Only accept reg, subreg(reg), const_int.  */
-int
-reg_or_int_operand (rtx op, enum machine_mode mode)
-{
-  if (GET_CODE (op) == CONST_INT)
-    return 1;
-
-  if (GET_MODE (op) != mode && mode != VOIDmode)
-    return 0;
-
-  if (GET_CODE (op) == SUBREG)
-    op = SUBREG_REG (op);
-
-  /* We don't consider registers whose class is NO_REGS
-     to be a register operand.  */
-  return (GET_CODE (op) == REG
-         && (REGNO (op) >= FIRST_PSEUDO_REGISTER
-             || REGNO_REG_CLASS (REGNO (op)) != NO_REGS));
-}
-
-/* Return 1 if OP is an item in memory, given that we are in reload.  */
-int
-arm_reload_memory_operand (rtx op, enum machine_mode mode ATTRIBUTE_UNUSED)
-{
-  int regno = true_regnum (op);
-
-  return (!CONSTANT_P (op)
-         && (regno == -1
-             || (GET_CODE (op) == REG
-                 && REGNO (op) >= FIRST_PSEUDO_REGISTER)));
-}
-
-/* Return TRUE for valid operands for the rhs of an ARM instruction.  */
-int
-arm_rhs_operand (rtx op, enum machine_mode mode)
-{
-  return (s_register_operand (op, mode)
-         || (GET_CODE (op) == CONST_INT && const_ok_for_arm (INTVAL (op))));
-}
-
-/* Return TRUE for valid operands for the
-   rhs of an ARM instruction, or a load.  */
-int
-arm_rhsm_operand (rtx op, enum machine_mode mode)
-{
-  return (s_register_operand (op, mode)
-         || (GET_CODE (op) == CONST_INT && const_ok_for_arm (INTVAL (op)))
-         || memory_operand (op, mode));
-}
-
-/* Return TRUE for valid operands for the rhs of an ARM instruction, or if a
-   constant that is valid when negated.  */
-int
-arm_add_operand (rtx op, enum machine_mode mode)
-{
-  if (TARGET_THUMB)
-    return thumb_cmp_operand (op, mode);
-  
-  return (s_register_operand (op, mode)
-         || (GET_CODE (op) == CONST_INT
-             && (const_ok_for_arm (INTVAL (op))
-                 || const_ok_for_arm (-INTVAL (op)))));
-}
-
-/* Return TRUE for valid ARM constants (or when valid if negated).  */
-int
-arm_addimm_operand (rtx op, enum machine_mode mode ATTRIBUTE_UNUSED)
-{
-  return (GET_CODE (op) == CONST_INT
-         && (const_ok_for_arm (INTVAL (op))
-             || const_ok_for_arm (-INTVAL (op))));
-}
-
-int
-arm_not_operand (rtx op, enum machine_mode mode)
-{
-  return (s_register_operand (op, mode)
-         || (GET_CODE (op) == CONST_INT
-             && (const_ok_for_arm (INTVAL (op))
-                 || const_ok_for_arm (~INTVAL (op)))));
-}
-
-/* Return TRUE if the operand is a memory reference which contains an
-   offsettable address.  */
-int
-offsettable_memory_operand (rtx op, enum machine_mode mode)
-{
-  if (mode == VOIDmode)
-    mode = GET_MODE (op);
-
-  return (mode == GET_MODE (op)
-         && GET_CODE (op) == MEM
-         && offsettable_address_p (reload_completed | reload_in_progress,
-                                   mode, XEXP (op, 0)));
-}
-
-/* Return TRUE if the operand is a memory reference which is, or can be
-   made word aligned by adjusting the offset.  */
-int
-alignable_memory_operand (rtx op, enum machine_mode mode)
-{
-  rtx reg;
-
-  if (mode == VOIDmode)
-    mode = GET_MODE (op);
-
-  if (mode != GET_MODE (op) || GET_CODE (op) != MEM)
-    return 0;
-
-  op = XEXP (op, 0);
-
-  return ((GET_CODE (reg = op) == REG
-          || (GET_CODE (op) == SUBREG
-              && GET_CODE (reg = SUBREG_REG (op)) == REG)
-          || (GET_CODE (op) == PLUS
-              && GET_CODE (XEXP (op, 1)) == CONST_INT
-              && (GET_CODE (reg = XEXP (op, 0)) == REG
-                  || (GET_CODE (XEXP (op, 0)) == SUBREG
-                      && GET_CODE (reg = SUBREG_REG (XEXP (op, 0))) == REG))))
-         && REGNO_POINTER_ALIGN (REGNO (reg)) >= 32);
-}
-
-/* Similar to s_register_operand, but does not allow hard integer 
-   registers.  */
-int
-f_register_operand (rtx op, enum machine_mode mode)
-{
-  if (GET_MODE (op) != mode && mode != VOIDmode)
-    return 0;
-
-  if (GET_CODE (op) == SUBREG)
-    op = SUBREG_REG (op);
-
-  /* We don't consider registers whose class is NO_REGS
-     to be a register operand.  */
-  return (GET_CODE (op) == REG
-         && (REGNO (op) >= FIRST_PSEUDO_REGISTER
-             || REGNO_REG_CLASS (REGNO (op)) == FPA_REGS));
-}
-
-/* Return TRUE for valid operands for the rhs of an floating point insns.
-   Allows regs or certain consts on FPA, just regs for everything else.  */
-int
-arm_float_rhs_operand (rtx op, enum machine_mode mode)
-{
-  if (s_register_operand (op, mode))
-    return TRUE;
-
-  if (GET_MODE (op) != mode && mode != VOIDmode)
-    return FALSE;
-
-  if (TARGET_FPA && GET_CODE (op) == CONST_DOUBLE)
-    return arm_const_double_rtx (op);
-
-  return FALSE;
-}
-
-int
-arm_float_add_operand (rtx op, enum machine_mode mode)
-{
-  if (s_register_operand (op, mode))
-    return TRUE;
-
-  if (GET_MODE (op) != mode && mode != VOIDmode)
-    return FALSE;
-
-  if (TARGET_FPA && GET_CODE (op) == CONST_DOUBLE)
-    return (arm_const_double_rtx (op)
-           || neg_const_double_rtx_ok_for_fpa (op));
-
-  return FALSE;
-}
-
-
-/* Return TRUE if OP is suitable for the rhs of a floating point comparison.
-   Depends which fpu we are targeting.  */
-
-int
-arm_float_compare_operand (rtx op, enum machine_mode mode)
-{
-  if (TARGET_VFP)
-    return vfp_compare_operand (op, mode);
-  else
-    return arm_float_rhs_operand (op, mode);
-}
-
-
 /* Return nonzero if OP is a valid Cirrus memory address pattern.  */
 int
 cirrus_memory_offset (rtx op)
@@ -4644,55 +4674,6 @@ cirrus_memory_offset (rtx op)
   return 0;
 }
 
-int
-arm_extendqisi_mem_op (rtx op, enum machine_mode mode)
-{
-  if (!memory_operand (op, mode))
-    return 0;
-
-  return arm_legitimate_address_p (mode, XEXP (op, 0), SIGN_EXTEND, 0);
-}
-
-/* Return nonzero if OP is a Cirrus or general register.  */
-int
-cirrus_register_operand (rtx op, enum machine_mode mode)
-{
-  if (GET_MODE (op) != mode && mode != VOIDmode)
-    return FALSE;
-
-  if (GET_CODE (op) == SUBREG)
-    op = SUBREG_REG (op);
-
-  return (GET_CODE (op) == REG
-         && (REGNO_REG_CLASS (REGNO (op)) == CIRRUS_REGS
-             || REGNO_REG_CLASS (REGNO (op)) == GENERAL_REGS));
-}
-
-/* Return nonzero if OP is a cirrus FP register.  */
-int
-cirrus_fp_register (rtx op, enum machine_mode mode)
-{
-  if (GET_MODE (op) != mode && mode != VOIDmode)
-    return FALSE;
-
-  if (GET_CODE (op) == SUBREG)
-    op = SUBREG_REG (op);
-
-  return (GET_CODE (op) == REG
-         && (REGNO (op) >= FIRST_PSEUDO_REGISTER
-             || REGNO_REG_CLASS (REGNO (op)) == CIRRUS_REGS));
-}
-
-/* Return nonzero if OP is a 6bit constant (0..63).  */
-int
-cirrus_shift_const (rtx op, enum machine_mode mode ATTRIBUTE_UNUSED)
-{
-  return (GET_CODE (op) == CONST_INT
-         && INTVAL (op) >= 0
-         && INTVAL (op) < 64);
-}
-
-
 /* Return TRUE if OP is a valid VFP memory address pattern.
    WB if true if writeback address modes are allowed.  */
 
@@ -4760,18 +4741,6 @@ arm_coproc_mem_operand (rtx op, bool wb)
 }
 
 
-/* Return TRUE if OP is a REG or constant zero.  */
-int
-vfp_compare_operand (rtx op, enum machine_mode mode)
-{
-  if (s_register_operand (op, mode))
-    return TRUE;
-
-  return (GET_CODE (op) == CONST_DOUBLE
-         && arm_const_double_rtx (op));
-}
-
-
 /* Return GENERAL_REGS if a scratch register required to reload x to/from
    VFP registers.  Otherwise return NO_REGS.  */
 
@@ -4957,262 +4926,6 @@ cirrus_reorg (rtx first)
     }
 }
 
-/* Return nonzero if OP is a constant power of two.  */
-int
-power_of_two_operand (rtx op, enum machine_mode mode ATTRIBUTE_UNUSED)
-{
-  if (GET_CODE (op) == CONST_INT)
-    {
-      HOST_WIDE_INT value = INTVAL (op);
-
-      return value != 0  &&  (value & (value - 1)) == 0;
-    }
-
-  return FALSE;
-}
-
-/* Return TRUE for a valid operand of a DImode operation.
-   Either: REG, SUBREG, CONST_DOUBLE or MEM(DImode_address).
-   Note that this disallows MEM(REG+REG), but allows
-   MEM(PRE/POST_INC/DEC(REG)).  */
-int
-di_operand (rtx op, enum machine_mode mode)
-{
-  if (s_register_operand (op, mode))
-    return TRUE;
-
-  if (mode != VOIDmode && GET_MODE (op) != VOIDmode && GET_MODE (op) != DImode)
-    return FALSE;
-
-  if (GET_CODE (op) == SUBREG)
-    op = SUBREG_REG (op);
-
-  switch (GET_CODE (op))
-    {
-    case CONST_DOUBLE:
-    case CONST_INT:
-      return TRUE;
-
-    case MEM:
-      return memory_address_p (DImode, XEXP (op, 0));
-
-    default:
-      return FALSE;
-    }
-}
-
-/* Like di_operand, but don't accept constants.  */
-int
-nonimmediate_di_operand (rtx op, enum machine_mode mode)
-{
-  if (s_register_operand (op, mode))
-    return TRUE;
-
-  if (mode != VOIDmode && GET_MODE (op) != VOIDmode && GET_MODE (op) != DImode)
-    return FALSE;
-
-  if (GET_CODE (op) == SUBREG)
-    op = SUBREG_REG (op);
-
-  if (GET_CODE (op) == MEM)
-    return memory_address_p (DImode, XEXP (op, 0));
-
-  return FALSE;
-}
-
-/* Return TRUE for a valid operand of a DFmode operation when soft-float.
-   Either: REG, SUBREG, CONST_DOUBLE or MEM(DImode_address).
-   Note that this disallows MEM(REG+REG), but allows
-   MEM(PRE/POST_INC/DEC(REG)).  */
-int
-soft_df_operand (rtx op, enum machine_mode mode)
-{
-  if (s_register_operand (op, mode))
-    return TRUE;
-
-  if (mode != VOIDmode && GET_MODE (op) != mode)
-    return FALSE;
-
-  if (GET_CODE (op) == SUBREG && CONSTANT_P (SUBREG_REG (op)))
-    return FALSE;
-  
-  if (GET_CODE (op) == SUBREG)
-    op = SUBREG_REG (op);
-  
-  switch (GET_CODE (op))
-    {
-    case CONST_DOUBLE:
-      return TRUE;
-
-    case MEM:
-      return memory_address_p (DFmode, XEXP (op, 0));
-
-    default:
-      return FALSE;
-    }
-}
-
-/* Like soft_df_operand, but don't accept constants.  */
-int
-nonimmediate_soft_df_operand (rtx op, enum machine_mode mode)
-{
-  if (s_register_operand (op, mode))
-    return TRUE;
-
-  if (mode != VOIDmode && GET_MODE (op) != mode)
-    return FALSE;
-
-  if (GET_CODE (op) == SUBREG)
-    op = SUBREG_REG (op);
-
-  if (GET_CODE (op) == MEM)
-    return memory_address_p (DFmode, XEXP (op, 0));
-  return FALSE;
-}
-
-/* Return TRUE for valid index operands.  */
-int
-index_operand (rtx op, enum machine_mode mode)
-{
-  return (s_register_operand (op, mode)
-         || (immediate_operand (op, mode)
-             && (GET_CODE (op) != CONST_INT
-                 || (INTVAL (op) < 4096 && INTVAL (op) > -4096))));
-}
-
-/* Return TRUE for valid shifts by a constant. This also accepts any
-   power of two on the (somewhat overly relaxed) assumption that the
-   shift operator in this case was a mult.  */
-int
-const_shift_operand (rtx op, enum machine_mode mode)
-{
-  return (power_of_two_operand (op, mode)
-         || (immediate_operand (op, mode)
-             && (GET_CODE (op) != CONST_INT
-                 || (INTVAL (op) < 32 && INTVAL (op) > 0))));
-}
-
-/* Return TRUE for arithmetic operators which can be combined with a multiply
-   (shift).  */
-int
-shiftable_operator (rtx x, enum machine_mode mode)
-{
-  enum rtx_code code;
-
-  if (GET_MODE (x) != mode)
-    return FALSE;
-
-  code = GET_CODE (x);
-
-  return (code == PLUS || code == MINUS
-         || code == IOR || code == XOR || code == AND);
-}
-
-/* Return TRUE for binary logical operators.  */
-int
-logical_binary_operator (rtx x, enum machine_mode mode)
-{
-  enum rtx_code code;
-
-  if (GET_MODE (x) != mode)
-    return FALSE;
-
-  code = GET_CODE (x);
-
-  return (code == IOR || code == XOR || code == AND);
-}
-
-/* Return TRUE for shift operators.  */
-int
-shift_operator (rtx x,enum machine_mode mode)
-{
-  enum rtx_code code;
-
-  if (GET_MODE (x) != mode)
-    return FALSE;
-
-  code = GET_CODE (x);
-
-  if (code == MULT)
-    return power_of_two_operand (XEXP (x, 1), mode);
-
-  return (code == ASHIFT || code == ASHIFTRT || code == LSHIFTRT
-         || code == ROTATERT);
-}
-
-/* Return TRUE if x is EQ or NE.  */
-int
-equality_operator (rtx x, enum machine_mode mode ATTRIBUTE_UNUSED)
-{
-  return GET_CODE (x) == EQ || GET_CODE (x) == NE;
-}
-
-/* Return TRUE if x is a comparison operator other than LTGT or UNEQ.  */
-int
-arm_comparison_operator (rtx x, enum machine_mode mode)
-{
-  return (comparison_operator (x, mode)
-         && GET_CODE (x) != LTGT
-         && GET_CODE (x) != UNEQ);
-}
-
-/* Return TRUE for SMIN SMAX UMIN UMAX operators.  */
-int
-minmax_operator (rtx x, enum machine_mode mode)
-{
-  enum rtx_code code = GET_CODE (x);
-
-  if (GET_MODE (x) != mode)
-    return FALSE;
-
-  return code == SMIN || code == SMAX || code == UMIN || code == UMAX;
-}
-
-/* Return TRUE if this is the condition code register, if we aren't given
-   a mode, accept any class CCmode register.  */
-int
-cc_register (rtx x, enum machine_mode mode)
-{
-  if (mode == VOIDmode)
-    {
-      mode = GET_MODE (x);
-      
-      if (GET_MODE_CLASS (mode) != MODE_CC)
-       return FALSE;
-    }
-
-  if (   GET_MODE (x) == mode
-      && GET_CODE (x) == REG
-      && REGNO    (x) == CC_REGNUM)
-    return TRUE;
-
-  return FALSE;
-}
-
-/* Return TRUE if this is the condition code register, if we aren't given
-   a mode, accept any class CCmode register which indicates a dominance
-   expression.  */
-int
-dominant_cc_register (rtx x, enum machine_mode mode)
-{
-  if (mode == VOIDmode)
-    {
-      mode = GET_MODE (x);
-      
-      if (GET_MODE_CLASS (mode) != MODE_CC)
-       return FALSE;
-    }
-
-  if (mode != CC_DNEmode && mode != CC_DEQmode
-      && mode != CC_DLEmode && mode != CC_DLTmode
-      && mode != CC_DGEmode && mode != CC_DGTmode
-      && mode != CC_DLEUmode && mode != CC_DLTUmode
-      && mode != CC_DGEUmode && mode != CC_DGTUmode)
-    return FALSE;
-
-  return cc_register (x, mode);
-}
-
 /* Return TRUE if X references a SYMBOL_REF.  */
 int
 symbol_mentioned_p (rtx x)
@@ -5224,7 +4937,7 @@ symbol_mentioned_p (rtx x)
     return 1;
 
   fmt = GET_RTX_FORMAT (GET_CODE (x));
-  
+
   for (i = GET_RTX_LENGTH (GET_CODE (x)) - 1; i >= 0; i--)
     {
       if (fmt[i] == 'E')
@@ -5300,7 +5013,7 @@ adjacent_mem_locations (rtx a, rtx b)
     {
       int val0 = 0, val1 = 0;
       int reg0, reg1;
-  
+
       if (GET_CODE (XEXP (a, 0)) == PLUS)
         {
          reg0 = REGNO  (XEXP (XEXP (a, 0), 0));
@@ -5322,130 +5035,12 @@ adjacent_mem_locations (rtx a, rtx b)
         arith_adjacentmem pattern to output an overlong sequence.  */
       if (!const_ok_for_op (PLUS, val0) || !const_ok_for_op (PLUS, val1))
        return 0;
-      
+
       return (reg0 == reg1) && ((val1 - val0) == 4 || (val0 - val1) == 4);
     }
   return 0;
 }
 
-/* Return 1 if OP is a load multiple operation.  It is known to be
-   parallel and the first section will be tested.  */
-int
-load_multiple_operation (rtx op, enum machine_mode mode ATTRIBUTE_UNUSED)
-{
-  HOST_WIDE_INT count = XVECLEN (op, 0);
-  int dest_regno;
-  rtx src_addr;
-  HOST_WIDE_INT i = 1, base = 0;
-  rtx elt;
-
-  if (count <= 1
-      || GET_CODE (XVECEXP (op, 0, 0)) != SET)
-    return 0;
-
-  /* Check to see if this might be a write-back.  */
-  if (GET_CODE (SET_SRC (elt = XVECEXP (op, 0, 0))) == PLUS)
-    {
-      i++;
-      base = 1;
-
-      /* Now check it more carefully.  */
-      if (GET_CODE (SET_DEST (elt)) != REG
-          || GET_CODE (XEXP (SET_SRC (elt), 0)) != REG
-          || GET_CODE (XEXP (SET_SRC (elt), 1)) != CONST_INT
-          || INTVAL (XEXP (SET_SRC (elt), 1)) != (count - 1) * 4)
-        return 0;
-    }
-
-  /* Perform a quick check so we don't blow up below.  */
-  if (count <= i
-      || GET_CODE (XVECEXP (op, 0, i - 1)) != SET
-      || GET_CODE (SET_DEST (XVECEXP (op, 0, i - 1))) != REG
-      || GET_CODE (SET_SRC (XVECEXP (op, 0, i - 1))) != MEM)
-    return 0;
-
-  dest_regno = REGNO (SET_DEST (XVECEXP (op, 0, i - 1)));
-  src_addr = XEXP (SET_SRC (XVECEXP (op, 0, i - 1)), 0);
-
-  for (; i < count; i++)
-    {
-      elt = XVECEXP (op, 0, i);
-
-      if (GET_CODE (elt) != SET
-          || GET_CODE (SET_DEST (elt)) != REG
-          || GET_MODE (SET_DEST (elt)) != SImode
-          || REGNO (SET_DEST (elt)) != (unsigned int)(dest_regno + i - base)
-          || GET_CODE (SET_SRC (elt)) != MEM
-          || GET_MODE (SET_SRC (elt)) != SImode
-          || GET_CODE (XEXP (SET_SRC (elt), 0)) != PLUS
-          || !rtx_equal_p (XEXP (XEXP (SET_SRC (elt), 0), 0), src_addr)
-          || GET_CODE (XEXP (XEXP (SET_SRC (elt), 0), 1)) != CONST_INT
-          || INTVAL (XEXP (XEXP (SET_SRC (elt), 0), 1)) != (i - base) * 4)
-        return 0;
-    }
-
-  return 1;
-}
-
-/* Return 1 if OP is a store multiple operation.  It is known to be
-   parallel and the first section will be tested.  */
-int
-store_multiple_operation (rtx op, enum machine_mode mode ATTRIBUTE_UNUSED)
-{
-  HOST_WIDE_INT count = XVECLEN (op, 0);
-  int src_regno;
-  rtx dest_addr;
-  HOST_WIDE_INT i = 1, base = 0;
-  rtx elt;
-
-  if (count <= 1
-      || GET_CODE (XVECEXP (op, 0, 0)) != SET)
-    return 0;
-
-  /* Check to see if this might be a write-back.  */
-  if (GET_CODE (SET_SRC (elt = XVECEXP (op, 0, 0))) == PLUS)
-    {
-      i++;
-      base = 1;
-
-      /* Now check it more carefully.  */
-      if (GET_CODE (SET_DEST (elt)) != REG
-          || GET_CODE (XEXP (SET_SRC (elt), 0)) != REG
-          || GET_CODE (XEXP (SET_SRC (elt), 1)) != CONST_INT
-          || INTVAL (XEXP (SET_SRC (elt), 1)) != (count - 1) * 4)
-        return 0;
-    }
-
-  /* Perform a quick check so we don't blow up below.  */
-  if (count <= i
-      || GET_CODE (XVECEXP (op, 0, i - 1)) != SET
-      || GET_CODE (SET_DEST (XVECEXP (op, 0, i - 1))) != MEM
-      || GET_CODE (SET_SRC (XVECEXP (op, 0, i - 1))) != REG)
-    return 0;
-
-  src_regno = REGNO (SET_SRC (XVECEXP (op, 0, i - 1)));
-  dest_addr = XEXP (SET_DEST (XVECEXP (op, 0, i - 1)), 0);
-
-  for (; i < count; i++)
-    {
-      elt = XVECEXP (op, 0, i);
-
-      if (GET_CODE (elt) != SET
-          || GET_CODE (SET_SRC (elt)) != REG
-          || GET_MODE (SET_SRC (elt)) != SImode
-          || REGNO (SET_SRC (elt)) != (unsigned int)(src_regno + i - base)
-          || GET_CODE (SET_DEST (elt)) != MEM
-          || GET_MODE (SET_DEST (elt)) != SImode
-          || GET_CODE (XEXP (SET_DEST (elt), 0)) != PLUS
-          || !rtx_equal_p (XEXP (XEXP (SET_DEST (elt), 0), 0), dest_addr)
-          || GET_CODE (XEXP (XEXP (SET_DEST (elt), 0), 1)) != CONST_INT
-          || INTVAL (XEXP (XEXP (SET_DEST (elt), 0), 1)) != (i - base) * 4)
-        return 0;
-    }
-
-  return 1;
-}
-
 int
 load_multiple_sequence (rtx *operands, int nops, int *regs, int *base,
                        HOST_WIDE_INT *load_offset)
@@ -5462,7 +5057,7 @@ load_multiple_sequence (rtx *operands, int nops, int *regs, int *base,
     abort ();
 
   /* Loop over the operands and check that the memory references are
-     suitable (ie immediate offsets from the same base register).  At
+     suitable (i.e. immediate offsets from the same base register).  At
      the same time, extract the target register, and the memory
      offsets.  */
   for (i = 0; i < nops; i++)
@@ -5503,7 +5098,7 @@ load_multiple_sequence (rtx *operands, int nops, int *regs, int *base,
                                  : REGNO (SUBREG_REG (operands[i])));
              order[0] = 0;
            }
-         else 
+         else
            {
              if (base_reg != (int) REGNO (reg))
                /* Not addressed from the same base register.  */
@@ -5589,15 +5184,15 @@ load_multiple_sequence (rtx *operands, int nops, int *regs, int *base,
      scratch register (one of the result regs) and then doing a load
      multiple actually becomes slower (and no smaller in code size).
      That is the transformation
+
        ldr     rd1, [rbase + offset]
        ldr     rd2, [rbase + offset + 4]
+
      to
+
        add     rd1, rbase, offset
        ldmia   rd1, {rd1, rd2}
+
      produces worse code -- '3 cycles + any stalls on rd2' instead of
      '2 cycles + any stalls on rd2'.  On ARMs with only one cache
      access per cycle, the first sequence could never complete in less
@@ -5613,7 +5208,7 @@ load_multiple_sequence (rtx *operands, int nops, int *regs, int *base,
 
   /* Can't do it without setting up the offset, only do this if it takes
      no more than one insn.  */
-  return (const_ok_for_arm (unsorted_offsets[order[0]]) 
+  return (const_ok_for_arm (unsorted_offsets[order[0]])
          || const_ok_for_arm (-unsorted_offsets[order[0]])) ? 5 : 0;
 }
 
@@ -5662,7 +5257,7 @@ emit_ldm_seq (rtx *operands, int nops)
       abort ();
     }
 
-  sprintf (buf + strlen (buf), "%s%s, {%s%s", REGISTER_PREFIX, 
+  sprintf (buf + strlen (buf), "%s%s, {%s%s", REGISTER_PREFIX,
           reg_names[base_reg], REGISTER_PREFIX, reg_names[regs[0]]);
 
   for (i = 1; i < nops; i++)
@@ -5691,7 +5286,7 @@ store_multiple_sequence (rtx *operands, int nops, int *regs, int *base,
     abort ();
 
   /* Loop over the operands and check that the memory references are
-     suitable (ie immediate offsets from the same base register).  At
+     suitable (i.e. immediate offsets from the same base register).  At
      the same time, extract the target register, and the memory
      offsets.  */
   for (i = 0; i < nops; i++)
@@ -5732,7 +5327,7 @@ store_multiple_sequence (rtx *operands, int nops, int *regs, int *base,
                                  : REGNO (SUBREG_REG (operands[i])));
              order[0] = 0;
            }
-         else 
+         else
            {
              if (base_reg != (int) REGNO (reg))
                /* Not addressed from the same base register.  */
@@ -5839,7 +5434,7 @@ emit_stm_seq (rtx *operands, int nops)
       abort ();
     }
 
-  sprintf (buf + strlen (buf), "%s%s, {%s%s", REGISTER_PREFIX, 
+  sprintf (buf + strlen (buf), "%s%s, {%s%s", REGISTER_PREFIX,
           reg_names[base_reg], REGISTER_PREFIX, reg_names[regs[0]]);
 
   for (i = 1; i < nops; i++)
@@ -5852,32 +5447,21 @@ emit_stm_seq (rtx *operands, int nops)
   return "";
 }
 
-int
-multi_register_push (rtx op, enum machine_mode mode ATTRIBUTE_UNUSED)
-{
-  if (GET_CODE (op) != PARALLEL
-      || (GET_CODE (XVECEXP (op, 0, 0)) != SET)
-      || (GET_CODE (SET_SRC (XVECEXP (op, 0, 0))) != UNSPEC)
-      || (XINT (SET_SRC (XVECEXP (op, 0, 0)), 1) != UNSPEC_PUSH_MULT))
-    return 0;
-
-  return 1;
-}
 \f
 /* Routines for use in generating RTL.  */
 
 rtx
 arm_gen_load_multiple (int base_regno, int count, rtx from, int up,
-                      int write_back, int unchanging_p, int in_struct_p,
-                      int scalar_p)
+                      int write_back, rtx basemem, HOST_WIDE_INT *offsetp)
 {
+  HOST_WIDE_INT offset = *offsetp;
   int i = 0, j;
   rtx result;
   int sign = up ? 1 : -1;
-  rtx mem;
+  rtx mem, addr;
 
   /* XScale has load-store double instructions, but they have stricter
-     alignment requirements than load-store multiple, so we can not
+     alignment requirements than load-store multiple, so we cannot
      use them.
 
      For XScale ldm requires 2 + NREGS cycles to complete and blocks
@@ -5907,24 +5491,26 @@ arm_gen_load_multiple (int base_regno, int count, rtx from, int up,
   if (arm_tune_xscale && count <= 2 && ! optimize_size)
     {
       rtx seq;
-      
+
       start_sequence ();
-      
+
       for (i = 0; i < count; i++)
        {
-         mem = gen_rtx_MEM (SImode, plus_constant (from, i * 4 * sign));
-         RTX_UNCHANGING_P (mem) = unchanging_p;
-         MEM_IN_STRUCT_P (mem) = in_struct_p;
-         MEM_SCALAR_P (mem) = scalar_p;
+         addr = plus_constant (from, i * 4 * sign);
+         mem = adjust_automodify_address (basemem, SImode, addr, offset);
          emit_move_insn (gen_rtx_REG (SImode, base_regno + i), mem);
+         offset += 4 * sign;
        }
 
       if (write_back)
-       emit_move_insn (from, plus_constant (from, count * 4 * sign));
+       {
+         emit_move_insn (from, plus_constant (from, count * 4 * sign));
+         *offsetp = offset;
+       }
 
       seq = get_insns ();
       end_sequence ();
-      
+
       return seq;
     }
 
@@ -5941,50 +5527,54 @@ arm_gen_load_multiple (int base_regno, int count, rtx from, int up,
 
   for (j = 0; i < count; i++, j++)
     {
-      mem = gen_rtx_MEM (SImode, plus_constant (from, j * 4 * sign));
-      RTX_UNCHANGING_P (mem) = unchanging_p;
-      MEM_IN_STRUCT_P (mem) = in_struct_p;
-      MEM_SCALAR_P (mem) = scalar_p;
+      addr = plus_constant (from, j * 4 * sign);
+      mem = adjust_automodify_address_nv (basemem, SImode, addr, offset);
       XVECEXP (result, 0, i)
        = gen_rtx_SET (VOIDmode, gen_rtx_REG (SImode, base_regno + j), mem);
+      offset += 4 * sign;
     }
 
+  if (write_back)
+    *offsetp = offset;
+
   return result;
 }
 
 rtx
 arm_gen_store_multiple (int base_regno, int count, rtx to, int up,
-                       int write_back, int unchanging_p, int in_struct_p,
-                       int scalar_p)
+                       int write_back, rtx basemem, HOST_WIDE_INT *offsetp)
 {
+  HOST_WIDE_INT offset = *offsetp;
   int i = 0, j;
   rtx result;
   int sign = up ? 1 : -1;
-  rtx mem;
+  rtx mem, addr;
 
   /* See arm_gen_load_multiple for discussion of
      the pros/cons of ldm/stm usage for XScale.  */
   if (arm_tune_xscale && count <= 2 && ! optimize_size)
     {
       rtx seq;
-      
+
       start_sequence ();
-      
+
       for (i = 0; i < count; i++)
        {
-         mem = gen_rtx_MEM (SImode, plus_constant (to, i * 4 * sign));
-         RTX_UNCHANGING_P (mem) = unchanging_p;
-         MEM_IN_STRUCT_P (mem) = in_struct_p;
-         MEM_SCALAR_P (mem) = scalar_p;
+         addr = plus_constant (to, i * 4 * sign);
+         mem = adjust_automodify_address (basemem, SImode, addr, offset);
          emit_move_insn (mem, gen_rtx_REG (SImode, base_regno + i));
+         offset += 4 * sign;
        }
 
       if (write_back)
-       emit_move_insn (to, plus_constant (to, count * 4 * sign));
+       {
+         emit_move_insn (to, plus_constant (to, count * 4 * sign));
+         *offsetp = offset;
+       }
 
       seq = get_insns ();
       end_sequence ();
-      
+
       return seq;
     }
 
@@ -6001,15 +5591,16 @@ arm_gen_store_multiple (int base_regno, int count, rtx to, int up,
 
   for (j = 0; i < count; i++, j++)
     {
-      mem = gen_rtx_MEM (SImode, plus_constant (to, j * 4 * sign));
-      RTX_UNCHANGING_P (mem) = unchanging_p;
-      MEM_IN_STRUCT_P (mem) = in_struct_p;
-      MEM_SCALAR_P (mem) = scalar_p;
-
+      addr = plus_constant (to, j * 4 * sign);
+      mem = adjust_automodify_address_nv (basemem, SImode, addr, offset);
       XVECEXP (result, 0, i)
        = gen_rtx_SET (VOIDmode, mem, gen_rtx_REG (SImode, base_regno + j));
+      offset += 4 * sign;
     }
 
+  if (write_back)
+    *offsetp = offset;
+
   return result;
 }
 
@@ -6017,13 +5608,11 @@ int
 arm_gen_movmemqi (rtx *operands)
 {
   HOST_WIDE_INT in_words_to_go, out_words_to_go, last_bytes;
+  HOST_WIDE_INT srcoffset, dstoffset;
   int i;
-  rtx src, dst;
-  rtx st_src, st_dst, fin_src, fin_dst;
+  rtx src, dst, srcbase, dstbase;
   rtx part_bytes_reg = NULL;
   rtx mem;
-  int dst_unchanging_p, dst_in_struct_p, src_unchanging_p, src_in_struct_p;
-  int dst_scalar_p, src_scalar_p;
 
   if (GET_CODE (operands[2]) != CONST_INT
       || GET_CODE (operands[3]) != CONST_INT
@@ -6031,22 +5620,16 @@ arm_gen_movmemqi (rtx *operands)
       || INTVAL (operands[3]) & 3)
     return 0;
 
-  st_dst = XEXP (operands[0], 0);
-  st_src = XEXP (operands[1], 0);
+  dstbase = operands[0];
+  srcbase = operands[1];
 
-  dst_unchanging_p = RTX_UNCHANGING_P (operands[0]);
-  dst_in_struct_p = MEM_IN_STRUCT_P (operands[0]);
-  dst_scalar_p = MEM_SCALAR_P (operands[0]);
-  src_unchanging_p = RTX_UNCHANGING_P (operands[1]);
-  src_in_struct_p = MEM_IN_STRUCT_P (operands[1]);
-  src_scalar_p = MEM_SCALAR_P (operands[1]);
-
-  fin_dst = dst = copy_to_mode_reg (SImode, st_dst);
-  fin_src = src = copy_to_mode_reg (SImode, st_src);
+  dst = copy_to_mode_reg (SImode, XEXP (dstbase, 0));
+  src = copy_to_mode_reg (SImode, XEXP (srcbase, 0));
 
   in_words_to_go = ARM_NUM_INTS (INTVAL (operands[2]));
   out_words_to_go = INTVAL (operands[2]) / 4;
   last_bytes = INTVAL (operands[2]) & 3;
+  dstoffset = srcoffset = 0;
 
   if (out_words_to_go != in_words_to_go && ((in_words_to_go - 1) & 3) != 0)
     part_bytes_reg = gen_rtx_REG (SImode, (in_words_to_go - 1) & 3);
@@ -6055,38 +5638,31 @@ arm_gen_movmemqi (rtx *operands)
     {
       if (in_words_to_go > 4)
        emit_insn (arm_gen_load_multiple (0, 4, src, TRUE, TRUE,
-                                         src_unchanging_p,
-                                         src_in_struct_p,
-                                         src_scalar_p));
+                                         srcbase, &srcoffset));
       else
-       emit_insn (arm_gen_load_multiple (0, in_words_to_go, src, TRUE, 
-                                         FALSE, src_unchanging_p,
-                                         src_in_struct_p, src_scalar_p));
+       emit_insn (arm_gen_load_multiple (0, in_words_to_go, src, TRUE,
+                                         FALSE, srcbase, &srcoffset));
 
       if (out_words_to_go)
        {
          if (out_words_to_go > 4)
            emit_insn (arm_gen_store_multiple (0, 4, dst, TRUE, TRUE,
-                                              dst_unchanging_p,
-                                              dst_in_struct_p,
-                                              dst_scalar_p));
+                                              dstbase, &dstoffset));
          else if (out_words_to_go != 1)
            emit_insn (arm_gen_store_multiple (0, out_words_to_go,
-                                              dst, TRUE, 
+                                              dst, TRUE,
                                               (last_bytes == 0
                                                ? FALSE : TRUE),
-                                              dst_unchanging_p,
-                                              dst_in_struct_p,
-                                              dst_scalar_p));
+                                              dstbase, &dstoffset));
          else
            {
-             mem = gen_rtx_MEM (SImode, dst);
-             RTX_UNCHANGING_P (mem) = dst_unchanging_p;
-             MEM_IN_STRUCT_P (mem) = dst_in_struct_p;
-             MEM_SCALAR_P (mem) = dst_scalar_p;
+             mem = adjust_automodify_address (dstbase, SImode, dst, dstoffset);
              emit_move_insn (mem, gen_rtx_REG (SImode, 0));
              if (last_bytes != 0)
-               emit_insn (gen_addsi3 (dst, dst, GEN_INT (4)));
+               {
+                 emit_insn (gen_addsi3 (dst, dst, GEN_INT (4)));
+                 dstoffset += 4;
+               }
            }
        }
 
@@ -6098,22 +5674,14 @@ arm_gen_movmemqi (rtx *operands)
   if (out_words_to_go)
     {
       rtx sreg;
-      
-      mem = gen_rtx_MEM (SImode, src);
-      RTX_UNCHANGING_P (mem) = src_unchanging_p;
-      MEM_IN_STRUCT_P (mem) = src_in_struct_p;
-      MEM_SCALAR_P (mem) = src_scalar_p;
-      emit_move_insn (sreg = gen_reg_rtx (SImode), mem);
-      emit_move_insn (fin_src = gen_reg_rtx (SImode), plus_constant (src, 4));
-      
-      mem = gen_rtx_MEM (SImode, dst);
-      RTX_UNCHANGING_P (mem) = dst_unchanging_p;
-      MEM_IN_STRUCT_P (mem) = dst_in_struct_p;
-      MEM_SCALAR_P (mem) = dst_scalar_p;
+
+      mem = adjust_automodify_address (srcbase, SImode, src, srcoffset);
+      sreg = copy_to_reg (mem);
+
+      mem = adjust_automodify_address (dstbase, SImode, dst, dstoffset);
       emit_move_insn (mem, sreg);
-      emit_move_insn (fin_dst = gen_reg_rtx (SImode), plus_constant (dst, 4));
       in_words_to_go--;
-      
+
       if (in_words_to_go)      /* Sanity check */
        abort ();
     }
@@ -6123,10 +5691,7 @@ arm_gen_movmemqi (rtx *operands)
       if (in_words_to_go < 0)
        abort ();
 
-      mem = gen_rtx_MEM (SImode, src);
-      RTX_UNCHANGING_P (mem) = src_unchanging_p;
-      MEM_IN_STRUCT_P (mem) = src_in_struct_p;
-      MEM_SCALAR_P (mem) = src_scalar_p;
+      mem = adjust_automodify_address (srcbase, SImode, src, srcoffset);
       part_bytes_reg = copy_to_mode_reg (SImode, mem);
     }
 
@@ -6141,13 +5706,12 @@ arm_gen_movmemqi (rtx *operands)
       emit_insn (gen_lshrsi3 (tmp, part_bytes_reg,
                              GEN_INT (8 * (4 - last_bytes))));
       part_bytes_reg = tmp;
-      
+
       while (last_bytes)
        {
-         mem = gen_rtx_MEM (QImode, plus_constant (dst, last_bytes - 1));
-         RTX_UNCHANGING_P (mem) = dst_unchanging_p;
-         MEM_IN_STRUCT_P (mem) = dst_in_struct_p;
-         MEM_SCALAR_P (mem) = dst_scalar_p;
+         mem = adjust_automodify_address (dstbase, QImode,
+                                          plus_constant (dst, last_bytes - 1),
+                                          dstoffset + last_bytes - 1);
          emit_move_insn (mem, gen_lowpart (QImode, part_bytes_reg));
 
          if (--last_bytes)
@@ -6157,34 +5721,28 @@ arm_gen_movmemqi (rtx *operands)
              part_bytes_reg = tmp;
            }
        }
-         
+
     }
   else
     {
       if (last_bytes > 1)
        {
-         mem = gen_rtx_MEM (HImode, dst);
-         RTX_UNCHANGING_P (mem) = dst_unchanging_p;
-         MEM_IN_STRUCT_P (mem) = dst_in_struct_p;
-         MEM_SCALAR_P (mem) = dst_scalar_p;
+         mem = adjust_automodify_address (dstbase, HImode, dst, dstoffset);
          emit_move_insn (mem, gen_lowpart (HImode, part_bytes_reg));
          last_bytes -= 2;
          if (last_bytes)
            {
              rtx tmp = gen_reg_rtx (SImode);
-
              emit_insn (gen_addsi3 (dst, dst, const2_rtx));
              emit_insn (gen_lshrsi3 (tmp, part_bytes_reg, GEN_INT (16)));
              part_bytes_reg = tmp;
+             dstoffset += 2;
            }
        }
-      
+
       if (last_bytes)
        {
-         mem = gen_rtx_MEM (QImode, dst);
-         RTX_UNCHANGING_P (mem) = dst_unchanging_p;
-         MEM_IN_STRUCT_P (mem) = dst_in_struct_p;
-         MEM_SCALAR_P (mem) = dst_scalar_p;
+         mem = adjust_automodify_address (dstbase, QImode, dst, dstoffset);
          emit_move_insn (mem, gen_lowpart (QImode, part_bytes_reg));
        }
     }
@@ -6222,11 +5780,11 @@ arm_gen_rotated_half_load (rtx memref)
 
 /* Select a dominance comparison mode if possible for a test of the general
    form (OP (COND_OR (X) (Y)) (const_int 0)).  We support three forms.
-   COND_OR == DOM_CC_X_AND_Y => (X && Y) 
+   COND_OR == DOM_CC_X_AND_Y => (X && Y)
    COND_OR == DOM_CC_NX_OR_Y => ((! X) || Y)
-   COND_OR == DOM_CC_X_OR_Y => (X || Y) 
+   COND_OR == DOM_CC_X_OR_Y => (X || Y)
    In all cases OP will be either EQ or NE, but we don't need to know which
-   here.  If we are unable to support a dominance comparison we return 
+   here.  If we are unable to support a dominance comparison we return
    CC mode.  This will then fail to match for the RTL expressions that
    generate this call.  */
 enum machine_mode
@@ -6252,7 +5810,7 @@ arm_select_dominance_cc_mode (rtx x, rtx y, HOST_WIDE_INT cond_or)
 
   /* If the comparisons are not equal, and one doesn't dominate the other,
      then we can't do this.  */
-  if (cond1 != cond2 
+  if (cond1 != cond2
       && !comparison_dominates_p (cond1, cond2)
       && (swapped = 1, !comparison_dominates_p (cond2, cond1)))
     return CCmode;
@@ -6298,7 +5856,7 @@ arm_select_dominance_cc_mode (rtx x, rtx y, HOST_WIDE_INT cond_or)
       if (cond2 == NE)
        return CC_DNEmode;
       break;
-      
+
     case LTU:
       if (cond2 == LTU || cond_or == DOM_CC_X_AND_Y)
        return CC_DLTUmode;
@@ -6374,7 +5932,7 @@ arm_select_cc_mode (enum rtx_code op, rtx x, rtx y)
          abort ();
        }
     }
-  
+
   /* A compare with a shifted operand.  Because of canonicalization, the
      comparison will have to be swapped when we emit the assembler.  */
   if (GET_MODE (y) == SImode && GET_CODE (y) == REG
@@ -6383,7 +5941,7 @@ arm_select_cc_mode (enum rtx_code op, rtx x, rtx y)
          || GET_CODE (x) == ROTATERT))
     return CC_SWPmode;
 
-  /* This is a special case that is used by combine to allow a 
+  /* This is a special case that is used by combine to allow a
      comparison of a shifted byte load to be split into a zero-extend
      followed by a comparison of the shifted integer (only valid for
      equalities and unsigned inequalities).  */
@@ -6407,7 +5965,7 @@ arm_select_cc_mode (enum rtx_code op, rtx x, rtx y)
          || XEXP (x, 2) == const1_rtx)
       && COMPARISON_P (XEXP (x, 0))
       && COMPARISON_P (XEXP (x, 1)))
-    return arm_select_dominance_cc_mode (XEXP (x, 0), XEXP (x, 1), 
+    return arm_select_dominance_cc_mode (XEXP (x, 0), XEXP (x, 1),
                                         INTVAL (XEXP (x, 2)));
 
   /* Alternate canonicalizations of the above.  These are somewhat cleaner.  */
@@ -6579,12 +6137,12 @@ arm_reload_in_hi (rtx *operands)
                                                plus_constant (base,
                                                               offset))));
   emit_insn (gen_zero_extendqisi2 (gen_rtx_SUBREG (SImode, operands[0], 0),
-                                  gen_rtx_MEM (QImode, 
+                                  gen_rtx_MEM (QImode,
                                                plus_constant (base,
                                                               offset + 1))));
   if (!BYTES_BIG_ENDIAN)
     emit_insn (gen_rtx_SET (VOIDmode, gen_rtx_SUBREG (SImode, operands[0], 0),
-                       gen_rtx_IOR (SImode, 
+                       gen_rtx_IOR (SImode,
                                     gen_rtx_ASHIFT
                                     (SImode,
                                      gen_rtx_SUBREG (SImode, operands[0], 0),
@@ -6592,7 +6150,7 @@ arm_reload_in_hi (rtx *operands)
                                     scratch)));
   else
     emit_insn (gen_rtx_SET (VOIDmode, gen_rtx_SUBREG (SImode, operands[0], 0),
-                           gen_rtx_IOR (SImode, 
+                           gen_rtx_IOR (SImode,
                                         gen_rtx_ASHIFT (SImode, scratch,
                                                         GEN_INT (8)),
                                         gen_rtx_SUBREG (SImode, operands[0],
@@ -6741,7 +6299,7 @@ arm_reload_out_hi (rtx *operands)
 
   if (BYTES_BIG_ENDIAN)
     {
-      emit_insn (gen_movqi (gen_rtx_MEM (QImode, 
+      emit_insn (gen_movqi (gen_rtx_MEM (QImode,
                                         plus_constant (base, offset + 1)),
                            gen_lowpart (QImode, outval)));
       emit_insn (gen_lshrsi3 (scratch,
@@ -6951,7 +6509,7 @@ static rtx
 is_jump_table (rtx insn)
 {
   rtx table;
-  
+
   if (GET_CODE (insn) == JUMP_INSN
       && JUMP_LABEL (insn) != NULL
       && ((table = next_real_insn (JUMP_LABEL (insn)))
@@ -7025,7 +6583,7 @@ move_minipool_fix_forward_ref (Mnode *mp, Mnode *max_mp,
       mp->next = max_mp;
       mp->prev = max_mp->prev;
       max_mp->prev = mp;
-      
+
       if (mp->prev != NULL)
        mp->prev->next = mp;
       else
@@ -7057,7 +6615,7 @@ add_minipool_forward_ref (Mfix *fix)
   Mnode *       max_mp = NULL;
   HOST_WIDE_INT max_address = fix->address + fix->forwards;
   Mnode *       mp;
-  
+
   /* If this fix's address is greater than the address of the first
      entry, then we can't put the fix in this pool.  We subtract the
      size of the current fix to ensure that if the table is fully
@@ -7218,10 +6776,10 @@ move_minipool_fix_backward_ref (Mnode *mp, Mnode *min_mp,
     }
 
   return min_mp;
-}      
+}
 
 /* Add a constant to the minipool for a backward reference.  Returns the
-   node added or NULL if the constant will not fit in this pool.  
+   node added or NULL if the constant will not fit in this pool.
 
    Note that the code for insertion for a backwards reference can be
    somewhat confusing because the calculated offsets for each fix do
@@ -7260,8 +6818,8 @@ add_minipool_backward_ref (Mfix *fix)
          && rtx_equal_p (fix->value, mp->value)
          /* Check that there is enough slack to move this entry to the
             end of the table (this is conservative).  */
-         && (mp->max_address 
-             > (minipool_barrier->address 
+         && (mp->max_address
+             > (minipool_barrier->address
                 + minipool_vector_tail->offset
                 + minipool_vector_tail->fix_size)))
        {
@@ -7339,7 +6897,7 @@ add_minipool_backward_ref (Mfix *fix)
       mp->next = min_mp->next;
       mp->prev = min_mp;
       min_mp->next = mp;
-      
+
       if (mp->next != NULL)
        mp->next->prev = mp;
       else
@@ -7382,7 +6940,7 @@ assign_minipool_offsets (Mfix *barrier)
   for (mp = minipool_vector_head; mp != NULL; mp = mp->next)
     {
       mp->offset = offset;
-      
+
       if (mp->refcount > 0)
        offset += mp->fix_size;
     }
@@ -7419,7 +6977,7 @@ dump_minipool (rtx scan)
        {
          if (dump_file)
            {
-             fprintf (dump_file, 
+             fprintf (dump_file,
                       ";;  Offset %u, min %ld, max %ld ",
                       (unsigned) mp->offset, (unsigned long) mp->min_address,
                       (unsigned long) mp->max_address);
@@ -7486,7 +7044,7 @@ arm_barrier_cost (rtx insn)
     case CODE_LABEL:
       /* It will always be better to place the table before the label, rather
         than after it.  */
-      return 50;  
+      return 50;
 
     case INSN:
     case CALL_INSN:
@@ -7557,7 +7115,7 @@ create_fix_barrier (Mfix *fix, HOST_WIDE_INT max_address)
        }
 
       new_cost = arm_barrier_cost (from);
-      
+
       if (count < max_count && new_cost <= selected_cost)
        {
          selected = from;
@@ -7650,7 +7208,7 @@ push_minipool_fix (rtx insn, HOST_WIDE_INT address, rtx *loc,
       fprintf (dump_file,
               ";; %smode fixup for i%d; addr %lu, range (%ld,%ld): ",
               GET_MODE_NAME (mode),
-              INSN_UID (insn), (unsigned long) address, 
+              INSN_UID (insn), (unsigned long) address,
               -1 * (long)fix->backwards, (long)fix->forwards);
       arm_print_value (dump_file, fix->value);
       fprintf (dump_file, "\n");
@@ -7658,7 +7216,7 @@ push_minipool_fix (rtx insn, HOST_WIDE_INT address, rtx *loc,
 
   /* Add it to the chain of fixes.  */
   fix->next = NULL;
-  
+
   if (minipool_fix_head != NULL)
     minipool_fix_tail->next = fix;
   else
@@ -7721,7 +7279,7 @@ note_invalid_constants (rtx insn, HOST_WIDE_INT address, int do_pushes)
                  /* Casting the address of something to a mode narrower
                     than a word can cause avoid_constant_pool_reference()
                     to return the pool reference itself.  That's no good to
-                    us here.  Lets just hope that we can use the 
+                    us here.  Lets just hope that we can use the
                     constant pool value directly.  */
                  if (op == cop)
                    cop = get_pool_constant (XEXP (op, 0));
@@ -7787,7 +7345,7 @@ arm_reorg (void)
     }
 
   fix = minipool_fix_head;
-  
+
   /* Now scan the fixups and perform the required changes.  */
   while (fix)
     {
@@ -7827,7 +7385,7 @@ arm_reorg (void)
         the next mini-pool.  */
       if (last_barrier != NULL)
        {
-         /* Reduce the refcount for those fixes that won't go into this 
+         /* Reduce the refcount for those fixes that won't go into this
             pool after all.  */
          for (fdel = last_barrier->next;
               fdel && fdel != ftmp;
@@ -7883,7 +7441,7 @@ arm_reorg (void)
        if (GET_CODE (this_fix->insn) != BARRIER)
          {
            rtx addr
-             = plus_constant (gen_rtx_LABEL_REF (VOIDmode, 
+             = plus_constant (gen_rtx_LABEL_REF (VOIDmode,
                                                  minipool_vector_label),
                               this_fix->minipool->offset);
            *this_fix->loc = gen_rtx_MEM (this_fix->mode, addr);
@@ -7912,10 +7470,10 @@ fp_immediate_constant (rtx x)
 {
   REAL_VALUE_TYPE r;
   int i;
-  
+
   if (!fp_consts_inited)
     init_fp_table ();
-  
+
   REAL_VALUE_FROM_CONST_DOUBLE (r, x);
   for (i = 0; i < 8; i++)
     if (REAL_VALUES_EQUAL (r, values_fp[i]))
@@ -7953,13 +7511,13 @@ print_multi_reg (FILE *stream, const char *instr, int reg, int mask)
   fputc ('\t', stream);
   asm_fprintf (stream, instr, reg);
   fputs (", {", stream);
-  
+
   for (i = 0; i <= LAST_ARM_REGNUM; i++)
     if (mask & (1 << i))
       {
        if (not_first)
          fprintf (stream, ", ");
-       
+
        asm_fprintf (stream, "%r", i);
        not_first = TRUE;
       }
@@ -8117,14 +7675,14 @@ output_call (rtx *operands)
       operands[0] = gen_rtx_REG (SImode, IP_REGNUM);
       output_asm_insn ("mov%?\t%0, %|lr", operands);
     }
-  
+
   output_asm_insn ("mov%?\t%|lr, %|pc", operands);
-  
+
   if (TARGET_INTERWORK || arm_arch4t)
     output_asm_insn ("bx%?\t%0", operands);
   else
     output_asm_insn ("mov%?\t%|pc, %0", operands);
-  
+
   return "";
 }
 
@@ -8145,7 +7703,7 @@ output_call_mem (rtx *operands)
         load since the call will kill it anyway.  */
       output_asm_insn ("ldr%?\t%|ip, %0", operands);
       if (arm_arch5)
-       output_asm_insn ("blx%?%|ip", operands);
+       output_asm_insn ("blx%?\t%|ip", operands);
       else
        {
          output_asm_insn ("mov%?\t%|lr, %|pc", operands);
@@ -8180,10 +7738,10 @@ output_mov_long_double_fpa_from_arm (rtx *operands)
   ops[0] = gen_rtx_REG (SImode, arm_reg0);
   ops[1] = gen_rtx_REG (SImode, 1 + arm_reg0);
   ops[2] = gen_rtx_REG (SImode, 2 + arm_reg0);
-  
+
   output_asm_insn ("stm%?fd\t%|sp!, {%0, %1, %2}", ops);
   output_asm_insn ("ldf%?e\t%0, [%|sp], #12", operands);
-  
+
   return "";
 }
 
@@ -8254,7 +7812,7 @@ output_mov_double_fpa_from_arm (rtx *operands)
 
   if (arm_reg0 == IP_REGNUM)
     abort ();
-  
+
   ops[0] = gen_rtx_REG (SImode, arm_reg0);
   ops[1] = gen_rtx_REG (SImode, 1 + arm_reg0);
   output_asm_insn ("stm%?fd\t%|sp!, {%0, %1}", ops);
@@ -8296,7 +7854,7 @@ output_move_double (rtx *operands)
       int reg0 = REGNO (operands[0]);
 
       otherops[0] = gen_rtx_REG (SImode, 1 + reg0);
-      
+
       if (code1 == REG)
        {
          int reg1 = REGNO (operands[1]);
@@ -8401,7 +7959,7 @@ output_move_double (rtx *operands)
 
              operands[1] = GEN_INT (hint);
              break;
-             
+
            default:
              abort ();
            }
@@ -8432,7 +7990,7 @@ output_move_double (rtx *operands)
              otherops[1] = GEN_INT (CONST_DOUBLE_HIGH (operands[1]));
              operands[1] = GEN_INT (CONST_DOUBLE_LOW (operands[1]));
            }
-         
+
          output_mov_immediate (operands);
          output_mov_immediate (otherops);
        }
@@ -8561,7 +8119,7 @@ output_move_double (rtx *operands)
                                 avoid a conflict.  */
                              otherops[1] = XEXP (XEXP (operands[1], 0), 1);
                              otherops[2] = XEXP (XEXP (operands[1], 0), 0);
-                             
+
                            }
                          /* If both registers conflict, it will usually
                             have been fixed by a splitter.  */
@@ -8592,7 +8150,7 @@ output_move_double (rtx *operands)
                    }
                  else
                    output_asm_insn ("sub%?\t%0, %1, %2", otherops);
-                 
+
                  return "ldm%?ia\t%0, %M0";
                 }
               else
@@ -8800,7 +8358,7 @@ output_multi_immediate (rtx *operands, const char *instr1, const char *instr2,
            }
        }
     }
-  
+
   return "";
 }
 
@@ -8865,6 +8423,13 @@ shift_op (rtx op, HOST_WIDE_INT *amountp)
       mnem = "lsr";
       break;
 
+    case ROTATE:
+      if (*amountp == -1)
+       abort ();
+      *amountp = 32 - *amountp;
+
+      /* Fall through.  */
+
     case ROTATERT:
       mnem = "ror";
       break;
@@ -8889,7 +8454,7 @@ shift_op (rtx op, HOST_WIDE_INT *amountp)
         shift.  >=32 is not a valid shift for "asl", so we must try and
         output a shift that produces the correct arithmetical result.
         Using lsr #32 is identical except for the fact that the carry bit
-        is not set correctly if we set the flags; but we never use the 
+        is not set correctly if we set the flags; but we never use the
         carry bit from such an operation, so we can ignore that.  */
       if (code == ROTATERT)
        /* Rotate is just modulo 32.  */
@@ -8904,7 +8469,7 @@ shift_op (rtx op, HOST_WIDE_INT *amountp)
       /* Shifts of 0 are no-ops.  */
       if (*amountp == 0)
        return NULL;
-    }    
+    }
 
   return mnem;
 }
@@ -8937,7 +8502,7 @@ output_ascii_pseudo_op (FILE *stream, const unsigned char *p, int len)
   int len_so_far = 0;
 
   fputs ("\t.ascii\t\"", stream);
-  
+
   for (i = 0; i < len; i++)
     {
       int c = p[i];
@@ -8950,26 +8515,26 @@ output_ascii_pseudo_op (FILE *stream, const unsigned char *p, int len)
 
       switch (c)
        {
-       case TARGET_TAB:                
+       case TARGET_TAB:
          fputs ("\\t", stream);
-         len_so_far += 2;                      
+         len_so_far += 2;
          break;
-         
+
        case TARGET_FF:
          fputs ("\\f", stream);
          len_so_far += 2;
          break;
-         
+
        case TARGET_BS:
          fputs ("\\b", stream);
          len_so_far += 2;
          break;
-         
+
        case TARGET_CR:
          fputs ("\\r", stream);
          len_so_far += 2;
          break;
-         
+
        case TARGET_NEWLINE:
          fputs ("\\n", stream);
          c = p [i + 1];
@@ -8980,7 +8545,7 @@ output_ascii_pseudo_op (FILE *stream, const unsigned char *p, int len)
          else
            len_so_far += 2;
          break;
-         
+
        case '\"':
        case '\\':
          putc ('\\', stream);
@@ -9032,7 +8597,7 @@ arm_compute_save_reg0_reg12_mask (void)
        max_reg = 7;
       else
        max_reg = 12;
-       
+
       for (reg = 0; reg <= max_reg; reg++)
        if (regs_ever_live[reg]
            || (! current_function_is_leaf && call_used_regs [reg]))
@@ -9056,7 +8621,7 @@ arm_compute_save_reg0_reg12_mask (void)
       /* If we aren't loading the PIC register,
         don't stack it even though it may be live.  */
       if (flag_pic
-         && ! TARGET_SINGLE_PIC_BASE 
+         && ! TARGET_SINGLE_PIC_BASE
          && regs_ever_live[PIC_OFFSET_TABLE_REGNUM])
        save_reg_mask |= 1 << PIC_OFFSET_TABLE_REGNUM;
     }
@@ -9065,7 +8630,7 @@ arm_compute_save_reg0_reg12_mask (void)
   if (current_function_calls_eh_return)
     {
       unsigned int i;
-      
+
       for (i = 0; ; i++)
        {
          reg = EH_RETURN_DATA_REGNO (i);
@@ -9158,6 +8723,39 @@ arm_compute_save_reg_mask (void)
 }
 
 
+/* Compute a bit mask of which registers need to be
+   saved on the stack for the current function.  */
+static unsigned long
+thumb_compute_save_reg_mask (void)
+{
+  unsigned long mask;
+  int reg;
+
+  mask = 0;
+  for (reg = 0; reg < 12; reg ++)
+    {
+      if (regs_ever_live[reg] && !call_used_regs[reg])
+       mask |= 1 << reg;
+    }
+
+  if (flag_pic && !TARGET_SINGLE_PIC_BASE)
+    mask |= (1 << PIC_OFFSET_TABLE_REGNUM);
+  if (TARGET_SINGLE_PIC_BASE)
+    mask &= ~(1 << arm_pic_register);
+
+  /* lr will also be pushed if any lo regs are pushed.  */
+  if (mask & 0xff || thumb_force_lr_save ())
+    mask |= (1 << LR_REGNUM);
+
+  /* Make sure we have a low work register if we need one.  */
+  if (((mask & 0xff) == 0 && regs_ever_live[LAST_ARG_REGNUM])
+      && ((mask & 0x0f00) || TARGET_BACKTRACE))
+    mask |= 1 << LAST_LO_REGNUM;
+
+  return mask;
+}
+
+
 /* Return the number of bytes required to save VFP registers.  */
 static int
 arm_get_vfp_saved_size (void)
@@ -9226,15 +8824,15 @@ output_return_instruction (rtx operand, int really_return, int reverse)
       if (really_return)
        {
          rtx ops[2];
-      
+
          /* Otherwise, trap an attempted return by aborting.  */
          ops[0] = operand;
-         ops[1] = gen_rtx_SYMBOL_REF (Pmode, NEED_PLT_RELOC ? "abort(PLT)" 
+         ops[1] = gen_rtx_SYMBOL_REF (Pmode, NEED_PLT_RELOC ? "abort(PLT)"
                                       : "abort");
          assemble_external_libcall (ops[1]);
          output_asm_insn (reverse ? "bl%D0\t%a1" : "bl%d0\t%a1", ops);
        }
-      
+
       return "";
     }
 
@@ -9251,8 +8849,8 @@ output_return_instruction (rtx operand, int really_return, int reverse)
     {
       const char * return_reg;
 
-      /* If we do not have any special requirements for function exit 
-        (eg interworking, or ISR) then we can load the return address 
+      /* If we do not have any special requirements for function exit
+        (e.g. interworking, or ISR) then we can load the return address
         directly into the PC.  Otherwise we must load it into LR.  */
       if (really_return
          && ! TARGET_INTERWORK)
@@ -9293,10 +8891,10 @@ output_return_instruction (rtx operand, int really_return, int reverse)
        }
       if (reg <= LAST_ARM_REGNUM
          && (reg != LR_REGNUM
-             || ! really_return 
+             || ! really_return
              || ! IS_INTERRUPT (func_type)))
        {
-         sprintf (instr, "ldr%s\t%%|%s, [%%|sp], #4", conditional, 
+         sprintf (instr, "ldr%s\t%%|%s, [%%|sp], #4", conditional,
                   (reg == LR_REGNUM) ? return_reg : reg_names[reg]);
        }
       else
@@ -9350,7 +8948,7 @@ output_return_instruction (rtx operand, int really_return, int reverse)
                memcpy (p + 2, reg_names[reg], l);
                p += l + 2;
              }
-         
+
          if (live_regs_mask & (1 << LR_REGNUM))
            {
              sprintf (p, "%s%%|%s}", first ? "" : ", ", return_reg);
@@ -9396,7 +8994,7 @@ output_return_instruction (rtx operand, int really_return, int reverse)
        default:
          /* Use bx if it's available.  */
          if (arm_arch5 || arm_arch4t)
-           sprintf (instr, "bx%s\t%%|lr", conditional);            
+           sprintf (instr, "bx%s\t%%|lr", conditional);
          else
            sprintf (instr, "mov%s\t%%|pc, %%|lr", conditional);
          break;
@@ -9443,7 +9041,7 @@ arm_poke_function_name (FILE *stream, const char *name)
 
   length      = strlen (name) + 1;
   alignlength = ROUND_UP_WORD (length);
-  
+
   ASM_OUTPUT_ASCII (stream, name, length);
   ASM_OUTPUT_ALIGN (stream, 2);
   x = GEN_INT ((unsigned HOST_WIDE_INT) 0xff000000 + alignlength);
@@ -9462,13 +9060,13 @@ arm_output_function_prologue (FILE *f, HOST_WIDE_INT frame_size)
       thumb_output_function_prologue (f, frame_size);
       return;
     }
-  
+
   /* Sanity check.  */
   if (arm_ccfsm_state || arm_target_insn)
     abort ();
 
   func_type = arm_current_func_type ();
-  
+
   switch ((int) ARM_FUNC_TYPE (func_type))
     {
     default:
@@ -9487,7 +9085,7 @@ arm_output_function_prologue (FILE *f, HOST_WIDE_INT frame_size)
       asm_fprintf (f, "\t%@ ARM Exception Handler.\n");
       break;
     }
-  
+
   if (IS_NAKED (func_type))
     asm_fprintf (f, "\t%@ Naked Function: prologue and epilogue provided by programmer.\n");
 
@@ -9496,7 +9094,7 @@ arm_output_function_prologue (FILE *f, HOST_WIDE_INT frame_size)
 
   if (IS_NESTED (func_type))
     asm_fprintf (f, "\t%@ Nested: function declared inside another function.\n");
-    
+
   asm_fprintf (f, "\t%@ args = %d, pretend = %d, frame = %wd\n",
               current_function_args_size,
               current_function_pretend_args_size, frame_size);
@@ -9516,7 +9114,7 @@ arm_output_function_prologue (FILE *f, HOST_WIDE_INT frame_size)
     asm_fprintf (f, "\tmov\t%r, %r\n", IP_REGNUM, PIC_OFFSET_TABLE_REGNUM);
 #endif
 
-  return_used_this_function = 0;  
+  return_used_this_function = 0;
 }
 
 const char *
@@ -9525,7 +9123,7 @@ arm_output_epilogue (rtx sibling)
   int reg;
   unsigned long saved_regs_mask;
   unsigned long func_type;
-  /* Floats_offset is the offset from the "virtual" frame.  In an APCS 
+  /* Floats_offset is the offset from the "virtual" frame.  In an APCS
      frame that is $fp + 4 for a non-variadic function.  */
   int floats_offset = 0;
   rtx operands[3];
@@ -9549,12 +9147,12 @@ arm_output_epilogue (rtx sibling)
   if (IS_VOLATILE (func_type) && TARGET_ABORT_NORETURN)
     {
       rtx op;
-         
+
       /* A volatile function should never return.  Call abort.  */
       op = gen_rtx_SYMBOL_REF (Pmode, NEED_PLT_RELOC ? "abort(PLT)" : "abort");
       assemble_external_libcall (op);
       output_asm_insn ("bl\t%a0", &op);
-      
+
       return "";
     }
 
@@ -9563,7 +9161,7 @@ arm_output_epilogue (rtx sibling)
     /* If we are throwing an exception, then we really must
        be doing a return,  so we can't tail-call.  */
     abort ();
-  
+
   offsets = arm_get_frame_offsets ();
   saved_regs_mask = arm_compute_save_reg_mask ();
 
@@ -9575,7 +9173,7 @@ arm_output_epilogue (rtx sibling)
   for (reg = 0; reg <= LAST_ARM_REGNUM; reg++)
     if (saved_regs_mask & (1 << reg))
       floats_offset += 4;
-  
+
   if (frame_pointer_needed)
     {
       /* This variable is for the Virtual Frame Pointer, not VFP regs.  */
@@ -9587,7 +9185,7 @@ arm_output_epilogue (rtx sibling)
            if (regs_ever_live[reg] && !call_used_regs[reg])
              {
                floats_offset += 12;
-               asm_fprintf (f, "\tldfe\t%r, [%r, #-%d]\n", 
+               asm_fprintf (f, "\tldfe\t%r, [%r, #-%d]\n",
                             reg, FP_REGNUM, floats_offset - vfp_offset);
              }
        }
@@ -9600,7 +9198,7 @@ arm_output_epilogue (rtx sibling)
              if (regs_ever_live[reg] && !call_used_regs[reg])
                {
                  floats_offset += 12;
-                 
+
                  /* We can't unstack more than four registers at once.  */
                  if (start_reg - reg == 3)
                    {
@@ -9669,13 +9267,13 @@ arm_output_epilogue (rtx sibling)
             We can ignore floats_offset since that was already included in
             the live_regs_mask.  */
          lrm_count += (lrm_count % 2 ? 2 : 1);
-             
+
          for (reg = LAST_IWMMXT_REGNUM; reg >= FIRST_IWMMXT_REGNUM; reg--)
            if (regs_ever_live[reg] && !call_used_regs[reg])
              {
-               asm_fprintf (f, "\twldrd\t%r, [%r, #-%d]\n", 
+               asm_fprintf (f, "\twldrd\t%r, [%r, #-%d]\n",
                             reg, FP_REGNUM, lrm_count * 4);
-               lrm_count += 2; 
+               lrm_count += 2;
              }
        }
 
@@ -9759,7 +9357,7 @@ arm_output_epilogue (rtx sibling)
                    asm_fprintf (f, "\tlfmfd\t%r, %d, [%r]!\n",
                                 start_reg, reg - start_reg,
                                 SP_REGNUM);
-                 
+
                  start_reg = reg + 1;
                }
            }
@@ -9816,7 +9414,7 @@ arm_output_epilogue (rtx sibling)
        {
          if (saved_regs_mask & (1 << SP_REGNUM))
            /* Note - write back to the stack register is not enabled
-              (ie "ldmfd sp!...").  We know that the stack pointer is
+              (i.e. "ldmfd sp!...").  We know that the stack pointer is
               in the list of registers and if we add writeback the
               instruction becomes UNPREDICTABLE.  */
            print_multi_reg (f, "ldmfd\t%r", SP_REGNUM, saved_regs_mask);
@@ -9839,7 +9437,7 @@ arm_output_epilogue (rtx sibling)
 
   /* Stack adjustment for exception handler.  */
   if (current_function_calls_eh_return)
-    asm_fprintf (f, "\tadd\t%r, %r, %r\n", SP_REGNUM, SP_REGNUM, 
+    asm_fprintf (f, "\tadd\t%r, %r, %r\n", SP_REGNUM, SP_REGNUM,
                 ARM_EH_STACKADJ_REGNUM);
 
   /* Generate the return instruction.  */
@@ -9930,7 +9528,7 @@ emit_multi_reg_push (int mask)
      by the push_multi pattern in the arm.md file.  The insn looks
      something like this:
 
-       (parallel [ 
+       (parallel [
            (set (mem:BLK (pre_dec:BLK (reg:SI sp)))
                (unspec:BLK [(reg:SI r4)] UNSPEC_PUSH_MULT))
            (use (reg:SI 11 fp))
@@ -9946,7 +9544,7 @@ emit_multi_reg_push (int mask)
      stack decrement per instruction.  The RTL we generate for the note looks
      something like this:
 
-      (sequence [ 
+      (sequence [
            (set (reg:SI sp) (plus:SI (reg:SI sp) (const_int -20)))
            (set (mem:SI (reg:SI sp)) (reg:SI r4))
            (set (mem:SI (plus:SI (reg:SI sp) (const_int 4))) (reg:SI fp))
@@ -9956,7 +9554,7 @@ emit_multi_reg_push (int mask)
 
       This sequence is used both by the code to support stack unwinding for
       exceptions handlers and the code to generate dwarf2 frame debugging.  */
-  
+
   par = gen_rtx_PARALLEL (VOIDmode, rtvec_alloc (num_regs));
   dwarf = gen_rtx_SEQUENCE (VOIDmode, rtvec_alloc (num_dwarf_regs + 1));
   dwarf_par_index = 1;
@@ -10014,7 +9612,7 @@ emit_multi_reg_push (int mask)
     }
 
   par = emit_insn (par);
-  
+
   tmp = gen_rtx_SET (SImode,
                     stack_pointer_rtx,
                     gen_rtx_PLUS (SImode,
@@ -10022,7 +9620,7 @@ emit_multi_reg_push (int mask)
                                   GEN_INT (-4 * num_regs)));
   RTX_FRAME_RELATED_P (tmp) = 1;
   XVECEXP (dwarf, 0, 0) = tmp;
-  
+
   REG_NOTES (par) = gen_rtx_EXPR_LIST (REG_FRAME_RELATED_EXPR, dwarf,
                                       REG_NOTES (par));
   return par;
@@ -10042,29 +9640,29 @@ emit_sfm (int base_reg, int count)
   reg = gen_rtx_REG (XFmode, base_reg++);
 
   XVECEXP (par, 0, 0)
-    = gen_rtx_SET (VOIDmode, 
+    = gen_rtx_SET (VOIDmode,
                   gen_rtx_MEM (BLKmode,
                                gen_rtx_PRE_DEC (BLKmode, stack_pointer_rtx)),
                   gen_rtx_UNSPEC (BLKmode,
                                   gen_rtvec (1, reg),
                                   UNSPEC_PUSH_MULT));
-  tmp = gen_rtx_SET (VOIDmode, 
+  tmp = gen_rtx_SET (VOIDmode,
                     gen_rtx_MEM (XFmode, stack_pointer_rtx), reg);
   RTX_FRAME_RELATED_P (tmp) = 1;
-  XVECEXP (dwarf, 0, 1) = tmp;   
-  
+  XVECEXP (dwarf, 0, 1) = tmp;
+
   for (i = 1; i < count; i++)
     {
       reg = gen_rtx_REG (XFmode, base_reg++);
       XVECEXP (par, 0, i) = gen_rtx_USE (VOIDmode, reg);
 
-      tmp = gen_rtx_SET (VOIDmode, 
+      tmp = gen_rtx_SET (VOIDmode,
                         gen_rtx_MEM (XFmode,
                                      plus_constant (stack_pointer_rtx,
                                                     i * 12)),
                         reg);
       RTX_FRAME_RELATED_P (tmp) = 1;
-      XVECEXP (dwarf, 0, i + 1) = tmp;   
+      XVECEXP (dwarf, 0, i + 1) = tmp;
     }
 
   tmp = gen_rtx_SET (VOIDmode,
@@ -10156,7 +9754,7 @@ arm_get_frame_offsets (void)
   HOST_WIDE_INT frame_size;
 
   offsets = &cfun->machine->stack_offsets;
-  
+
   /* We need to know if we are a leaf function.  Unfortunately, it
      is possible to be called after start_sequence has been called,
      which causes get_insns to return the insns for the sequence,
@@ -10216,29 +9814,9 @@ arm_get_frame_offsets (void)
     }
   else /* TARGET_THUMB */
     {
-      int reg;
-      int count_regs;
-
-      saved = 0;
-      count_regs = 0;
-      for (reg = 8; reg < 13; reg ++)
-       if (THUMB_REG_PUSHED_P (reg))
-         count_regs ++;
-      if (count_regs)
-       saved += 4 * count_regs;
-      count_regs = 0;
-      for (reg = 0; reg <= LAST_LO_REGNUM; reg ++)
-       if (THUMB_REG_PUSHED_P (reg))
-         count_regs ++;
-      if (count_regs || thumb_force_lr_save ())
-       saved += 4 * (count_regs + 1);
+      saved = bit_count (thumb_compute_save_reg_mask ()) * 4;
       if (TARGET_BACKTRACE)
-       {
-         if ((count_regs & 0xFF) == 0 && (regs_ever_live[3] != 0))
-           saved += 20;
-         else
-           saved += 16;
-       }
+       saved += 16;
     }
 
   /* Saved registers include the stack frame.  */
@@ -10366,7 +9944,7 @@ arm_expand_prologue (void)
   int fp_offset = 0;
   int saved_pretend_args = 0;
   int saved_regs = 0;
-  unsigned int args_to_push;
+  unsigned HOST_WIDE_INT args_to_push;
   arm_stack_offsets *offsets;
 
   func_type = arm_current_func_type ();
@@ -10377,7 +9955,7 @@ arm_expand_prologue (void)
 
   /* Make a copy of c_f_p_a_s as we may need to modify it locally.  */
   args_to_push = current_function_pretend_args_size;
-  
+
   /* Compute which register we will have to save onto the stack.  */
   live_regs_mask = arm_compute_save_reg_mask ();
 
@@ -10411,7 +9989,7 @@ arm_expand_prologue (void)
             To get around this need to find somewhere to store IP
             whilst the frame is being created.  We try the following
             places in order:
-            
+
               1. The last argument register.
               2. A slot on the stack above the frame.  (This only
                  works if the function is not a varargs function).
@@ -10455,7 +10033,7 @@ arm_expand_prologue (void)
                  ((0xf0 >> (args_to_push / 4)) & 0xf);
              else
                insn = emit_insn
-                 (gen_addsi3 (stack_pointer_rtx, stack_pointer_rtx, 
+                 (gen_addsi3 (stack_pointer_rtx, stack_pointer_rtx,
                               GEN_INT (- args_to_push)));
 
              RTX_FRAME_RELATED_P (insn) = 1;
@@ -10478,7 +10056,7 @@ arm_expand_prologue (void)
        }
       else
        insn = gen_movsi (ip_rtx, stack_pointer_rtx);
-      
+
       insn = emit_insn (insn);
       RTX_FRAME_RELATED_P (insn) = 1;
     }
@@ -10491,7 +10069,7 @@ arm_expand_prologue (void)
          ((0xf0 >> (args_to_push / 4)) & 0xf);
       else
        insn = emit_insn
-         (gen_addsi3 (stack_pointer_rtx, stack_pointer_rtx, 
+         (gen_addsi3 (stack_pointer_rtx, stack_pointer_rtx,
                       GEN_INT (- args_to_push)));
       RTX_FRAME_RELATED_P (insn) = 1;
     }
@@ -10504,7 +10082,7 @@ arm_expand_prologue (void)
   if ((func_type == ARM_FT_ISR || func_type == ARM_FT_FIQ)
       && (live_regs_mask & (1 << LR_REGNUM)) != 0
       && ! frame_pointer_needed)
-    emit_insn (gen_rtx_SET (SImode, 
+    emit_insn (gen_rtx_SET (SImode,
                            gen_rtx_REG (SImode, LR_REGNUM),
                            gen_rtx_PLUS (SImode,
                                          gen_rtx_REG (SImode, LR_REGNUM),
@@ -10610,7 +10188,7 @@ arm_expand_prologue (void)
       insn = GEN_INT (-(4 + args_to_push + fp_offset));
       insn = emit_insn (gen_addsi3 (hard_frame_pointer_rtx, ip_rtx, insn));
       RTX_FRAME_RELATED_P (insn) = 1;
-      
+
       if (IS_NESTED (func_type))
        {
          /* Recover the static chain register.  */
@@ -10657,6 +10235,10 @@ arm_expand_prologue (void)
                                         hard_frame_pointer_rtx));
     }
 
+
+  if (flag_pic)
+    arm_load_pic_register ();
+
   /* If we are profiling, make sure no instructions are scheduled before
      the call to mcount.  Similarly if the user has requested no
      scheduling in the prolog.  */
@@ -10697,7 +10279,7 @@ arm_print_operand (FILE *stream, rtx x, int code)
     case '_':
       fputs (user_label_prefix, stream);
       return;
-         
+
     case '|':
       fputs (REGISTER_PREFIX, stream);
       return;
@@ -10705,8 +10287,17 @@ arm_print_operand (FILE *stream, rtx x, int code)
     case '?':
       if (arm_ccfsm_state == 3 || arm_ccfsm_state == 4)
        {
-         if (TARGET_THUMB || current_insn_predicate != NULL)
-           abort ();
+         if (TARGET_THUMB)
+           {
+             output_operand_lossage ("predicated Thumb instruction");
+             break;
+           }
+         if (current_insn_predicate != NULL)
+           {
+             output_operand_lossage
+               ("predicated instruction in conditional sequence");
+             break;
+           }
 
          fputs (arm_condition_codes[arm_current_cc], stream);
        }
@@ -10715,7 +10306,10 @@ arm_print_operand (FILE *stream, rtx x, int code)
          enum arm_cond_code code;
 
          if (TARGET_THUMB)
-           abort ();
+           {
+             output_operand_lossage ("predicated Thumb instruction");
+             break;
+           }
 
          code = get_arm_condition_code (current_insn_predicate);
          fputs (arm_condition_codes[code], stream);
@@ -10780,19 +10374,19 @@ arm_print_operand (FILE *stream, rtx x, int code)
       return;
 
       /* An explanation of the 'Q', 'R' and 'H' register operands:
-        
+
         In a pair of registers containing a DI or DF value the 'Q'
         operand returns the register number of the register containing
         the least significant part of the value.  The 'R' operand returns
         the register number of the register containing the most
         significant part of the value.
-        
+
         The 'H' operand returns the higher of the two register numbers.
         On a run where WORDS_BIG_ENDIAN is true the 'H' operand is the
         same as the 'Q' operand, since the most significant part of the
         value is held in the lower number register.  The reverse is true
         on systems where WORDS_BIG_ENDIAN is false.
-        
+
         The purpose of these operands is to distinguish between cases
         where the endian-ness of the values is important (for example
         when they are added together), and cases where the endian-ness
@@ -10807,25 +10401,37 @@ arm_print_operand (FILE *stream, rtx x, int code)
         of the memory location is actually held in one of the registers
         being overwritten by the load.  */
     case 'Q':
-      if (REGNO (x) > LAST_ARM_REGNUM)
-       abort ();
+      if (GET_CODE (x) != REG || REGNO (x) > LAST_ARM_REGNUM)
+       {
+         output_operand_lossage ("invalid operand for code '%c'", code);
+         return;
+       }
+
       asm_fprintf (stream, "%r", REGNO (x) + (WORDS_BIG_ENDIAN ? 1 : 0));
       return;
 
     case 'R':
-      if (REGNO (x) > LAST_ARM_REGNUM)
-       abort ();
+      if (GET_CODE (x) != REG || REGNO (x) > LAST_ARM_REGNUM)
+       {
+         output_operand_lossage ("invalid operand for code '%c'", code);
+         return;
+       }
+
       asm_fprintf (stream, "%r", REGNO (x) + (WORDS_BIG_ENDIAN ? 0 : 1));
       return;
 
-    case 'H':
-      if (REGNO (x) > LAST_ARM_REGNUM)
-       abort ();
+    case 'H':
+      if (GET_CODE (x) != REG || REGNO (x) > LAST_ARM_REGNUM)
+       {
+         output_operand_lossage ("invalid operand for code '%c'", code);
+         return;
+       }
+
       asm_fprintf (stream, "%r", REGNO (x) + 1);
       return;
 
     case 'm':
-      asm_fprintf (stream, "%r", 
+      asm_fprintf (stream, "%r",
                   GET_CODE (XEXP (x, 0)) == REG
                   ? REGNO (XEXP (x, 0)) : REGNO (XEXP (XEXP (x, 0), 0)));
       return;
@@ -10840,16 +10446,30 @@ arm_print_operand (FILE *stream, rtx x, int code)
       /* CONST_TRUE_RTX means always -- that's the default.  */
       if (x == const_true_rtx)
        return;
-      
+
+      if (!COMPARISON_P (x))
+       {
+         output_operand_lossage ("invalid operand for code '%c'", code);
+         return;
+       }
+
       fputs (arm_condition_codes[get_arm_condition_code (x)],
             stream);
       return;
 
     case 'D':
-      /* CONST_TRUE_RTX means not always -- ie never.  We shouldn't ever
+      /* CONST_TRUE_RTX means not always -- i.e. never.  We shouldn't ever
         want to do that.  */
       if (x == const_true_rtx)
-       abort ();
+       {
+         output_operand_lossage ("instruction never exectued");
+         return;
+       }
+      if (!COMPARISON_P (x))
+       {
+         output_operand_lossage ("invalid operand for code '%c'", code);
+         return;
+       }
 
       fputs (arm_condition_codes[ARM_INVERSE_CONDITION_CODE
                                 (get_arm_condition_code (x))],
@@ -10881,7 +10501,10 @@ arm_print_operand (FILE *stream, rtx x, int code)
        int mode = GET_MODE (x);
 
        if (GET_CODE (x) != REG || REGNO_REG_CLASS (REGNO (x)) != CIRRUS_REGS)
-         abort ();
+         {
+           output_operand_lossage ("invalid operand for code '%c'", code);
+           return;
+         }
 
        fprintf (stream, "mv%s%s",
                 mode == DFmode ? "d"
@@ -10897,7 +10520,11 @@ arm_print_operand (FILE *stream, rtx x, int code)
          || REGNO (x) < FIRST_IWMMXT_GR_REGNUM
          || REGNO (x) > LAST_IWMMXT_GR_REGNUM)
        /* Bad value for wCG register number.  */
-       abort ();
+       {
+         output_operand_lossage ("invalid operand for code '%c'", code);
+         return;
+       }
+
       else
        fprintf (stream, "%d", REGNO (x) - FIRST_IWMMXT_GR_REGNUM);
       return;
@@ -10908,7 +10535,11 @@ arm_print_operand (FILE *stream, rtx x, int code)
          || INTVAL (x) < 0
          || INTVAL (x) >= 16)
        /* Bad value for wC register number.  */
-       abort ();
+       {
+         output_operand_lossage ("invalid operand for code '%c'", code);
+         return;
+       }
+
       else
        {
          static const char * wc_reg_names [16] =
@@ -10918,7 +10549,7 @@ arm_print_operand (FILE *stream, rtx x, int code)
              "wCGR0", "wCGR1", "wCGR2", "wCGR3",
              "wC12",  "wC13",  "wC14",  "wC15"
            };
-         
+
          fprintf (stream, wc_reg_names [INTVAL (x)]);
        }
       return;
@@ -10930,15 +10561,24 @@ arm_print_operand (FILE *stream, rtx x, int code)
        int num;
 
        if (mode != DImode && mode != DFmode)
-         abort ();
+         {
+           output_operand_lossage ("invalid operand for code '%c'", code);
+           return;
+         }
 
        if (GET_CODE (x) != REG
            || !IS_VFP_REGNUM (REGNO (x)))
-         abort ();
+         {
+           output_operand_lossage ("invalid operand for code '%c'", code);
+           return;
+         }
 
        num = REGNO(x) - FIRST_VFP_REGNUM;
        if (num & 1)
-         abort ();
+         {
+           output_operand_lossage ("invalid operand for code '%c'", code);
+           return;
+         }
 
        fprintf (stream, "d%d", num >> 1);
       }
@@ -10946,7 +10586,10 @@ arm_print_operand (FILE *stream, rtx x, int code)
 
     default:
       if (x == 0)
-       abort ();
+       {
+         output_operand_lossage ("missing operand");
+         return;
+       }
 
       if (GET_CODE (x) == REG)
        asm_fprintf (stream, "%r", REGNO (x));
@@ -10996,7 +10639,7 @@ arm_assemble_integer (rtx x, unsigned int size, int aligned_p)
       return true;
     }
 
-  if (VECTOR_MODE_SUPPORTED_P (GET_MODE (x)))
+  if (arm_vector_mode_supported_p (GET_MODE (x)))
     {
       int i, units;
 
@@ -11170,7 +10813,7 @@ get_arm_condition_code (rtx comparison)
       case GEU: return ARM_CC;
       default: abort ();
       }
-      
+
     case CCmode:
       switch (comp_code)
        {
@@ -11208,10 +10851,10 @@ arm_final_prescan_insn (rtx insn)
      means that we have to grub around within the jump expression to find
      out what the conditions are when the jump isn't taken.  */
   int jump_clobbers = 0;
-  
+
   /* If we start with a return insn, we only succeed if we find another one.  */
   int seeking_return = 0;
-  
+
   /* START_INSN will hold the insn from where we start looking.  This is the
      first insn after the following code_label if REVERSE is true.  */
   rtx start_insn = insn;
@@ -11273,7 +10916,7 @@ arm_final_prescan_insn (rtx insn)
   if (GET_CODE (insn) != JUMP_INSN)
     return;
 
-  /* This jump might be paralleled with a clobber of the condition codes 
+  /* This jump might be paralleled with a clobber of the condition codes
      the jump should always come first */
   if (GET_CODE (body) == PARALLEL && XVECLEN (body, 0) > 0)
     body = XVECEXP (body, 0, 0);
@@ -11288,14 +10931,14 @@ arm_final_prescan_insn (rtx insn)
       int then_not_else = TRUE;
       rtx this_insn = start_insn, label = 0;
 
-      /* If the jump cannot be done with one instruction, we cannot 
+      /* If the jump cannot be done with one instruction, we cannot
         conditionally execute the instruction in the inverse case.  */
       if (get_attr_conds (insn) == CONDS_JUMP_CLOB)
        {
          jump_clobbers = 1;
          return;
        }
-      
+
       /* Register the insn jumped to.  */
       if (reverse)
         {
@@ -11353,8 +10996,8 @@ arm_final_prescan_insn (rtx insn)
 
            case BARRIER:
              /* Succeed if the following insn is the target label.
-                Otherwise fail.  
-                If return insns are used then the last insn in a function 
+                Otherwise fail.
+                If return insns are used then the last insn in a function
                 will be a barrier.  */
              this_insn = next_nonnote_insn (this_insn);
              if (this_insn && this_insn == label)
@@ -11426,7 +11069,7 @@ arm_final_prescan_insn (rtx insn)
                  else if (GET_CODE (SET_SRC (scanbody)) == IF_THEN_ELSE)
                    fail = TRUE;
                }
-             /* Fail if a conditional return is undesirable (eg on a
+             /* Fail if a conditional return is undesirable (e.g. on a
                 StrongARM), but still allow this if optimizing for size.  */
              else if (GET_CODE (scanbody) == RETURN
                       && !use_return_insn (TRUE, NULL)
@@ -11450,7 +11093,7 @@ arm_final_prescan_insn (rtx insn)
                    }
                }
              else
-               fail = TRUE;    /* Unrecognized jump (eg epilogue).  */
+               fail = TRUE;    /* Unrecognized jump (e.g. epilogue).  */
 
              break;
 
@@ -11509,7 +11152,7 @@ arm_final_prescan_insn (rtx insn)
            {
              if (reverse)
                abort ();
-             arm_current_cc = 
+             arm_current_cc =
                  get_arm_condition_code (XEXP (XEXP (XEXP (SET_SRC (body),
                                                            0), 0), 1));
              if (GET_CODE (XEXP (XEXP (SET_SRC (body), 0), 0)) == AND)
@@ -11529,7 +11172,7 @@ arm_final_prescan_insn (rtx insn)
          if (reverse || then_not_else)
            arm_current_cc = ARM_INVERSE_CONDITION_CODE (arm_current_cc);
        }
-      
+
       /* Restore recog_data (getting the attributes of other insns can
         destroy this array, but final.c assumes that it remains intact
         across this call; since the insn has been recognized already we
@@ -11539,13 +11182,13 @@ arm_final_prescan_insn (rtx insn)
 }
 
 /* Returns true if REGNO is a valid register
-   for holding a quantity of tyoe MODE.  */
+   for holding a quantity of type MODE.  */
 int
 arm_hard_regno_mode_ok (unsigned int regno, enum machine_mode mode)
 {
   if (GET_MODE_CLASS (mode) == MODE_CC)
     return regno == CC_REGNUM || regno == VFPCC_REGNUM;
-  
+
   if (TARGET_THUMB)
     /* For the Thumb we only allow values bigger than SImode in
        registers 0 - 6, so that there is always a second low
@@ -11615,7 +11258,7 @@ arm_regno_class (int regno)
       || regno == FRAME_POINTER_REGNUM
       || regno == ARG_POINTER_REGNUM)
     return GENERAL_REGS;
-  
+
   if (regno == CC_REGNUM || regno == VFPCC_REGNUM)
     return NO_REGS;
 
@@ -11653,13 +11296,13 @@ arm_debugger_arg_offset (int value, rtx addr)
      an offset of 0 is correct.  */
   if (REGNO (addr) == (unsigned) HARD_FRAME_POINTER_REGNUM)
     return 0;
-  
+
   /* If we are using the stack pointer to point at the
      argument, then an offset of 0 is correct.  */
   if ((TARGET_THUMB || !frame_pointer_needed)
       && REGNO (addr) == SP_REGNUM)
     return 0;
-  
+
   /* Oh dear.  The argument is pointed to by a register rather
      than being held in a register, or being stored at a known
      offset from the frame pointer.  Since GDB only understands
@@ -11669,12 +11312,12 @@ arm_debugger_arg_offset (int value, rtx addr)
      looking to see where this register gets its value.  If the
      register is initialized from the frame pointer plus an offset
      then we are in luck and we can continue, otherwise we give up.
-     
+
      This code is exercised by producing debugging information
      for a function with arguments like this:
-     
+
            double func (double a, double b, int c, double d) {return d;}
-     
+
      Without this code the stab for parameter 'd' will be set to
      an offset of 0 from the frame pointer, rather than 8.  */
 
@@ -11689,10 +11332,10 @@ arm_debugger_arg_offset (int value, rtx addr)
      a constant integer
 
      then...  */
-  
+
   for (insn = get_insns (); insn; insn = NEXT_INSN (insn))
     {
-      if (   GET_CODE (insn) == INSN 
+      if (   GET_CODE (insn) == INSN
          && GET_CODE (PATTERN (insn)) == SET
          && REGNO    (XEXP (PATTERN (insn), 0)) == REGNO (addr)
          && GET_CODE (XEXP (PATTERN (insn), 1)) == PLUS
@@ -11702,11 +11345,11 @@ arm_debugger_arg_offset (int value, rtx addr)
             )
        {
          value = INTVAL (XEXP (XEXP (PATTERN (insn), 1), 1));
-         
+
          break;
        }
     }
-  
+
   if (value == 0)
     {
       debug_rtx (addr);
@@ -11803,7 +11446,7 @@ static const struct builtin_description bdesc_2arg[] =
 
 #define IWMMXT_BUILTIN2(code, builtin) \
   { FL_IWMMXT, CODE_FOR_##code, NULL, ARM_BUILTIN_##builtin, 0, 0 },
-  
+
   IWMMXT_BUILTIN2 (iwmmxt_wpackhss, WPACKHSS)
   IWMMXT_BUILTIN2 (iwmmxt_wpackwss, WPACKWSS)
   IWMMXT_BUILTIN2 (iwmmxt_wpackdss, WPACKDSS)
@@ -12490,7 +12133,7 @@ arm_expand_builtin (tree exp,
        return 0;
       emit_insn (pat);
       return target;
-      
+
     case ARM_BUILTIN_WZERO:
       target = gen_reg_rtx (DImode);
       emit_insn (gen_iwmmxt_clrdi (target));
@@ -12522,7 +12165,7 @@ replace_symbols_in_block (tree block, rtx orig, rtx new)
   for (; block; block = BLOCK_CHAIN (block))
     {
       tree sym;
-      
+
       if (!TREE_USED (block))
        continue;
 
@@ -12538,7 +12181,7 @@ replace_symbols_in_block (tree block, rtx orig, rtx new)
 
          SET_DECL_RTL (sym, new);
        }
-      
+
       replace_symbols_in_block (BLOCK_SUBBLOCKS (block), orig, new);
     }
 }
@@ -12670,7 +12313,7 @@ thumb_exit (FILE *f, int reg_containing_return_addr)
   /* If we have any popping registers left over, remove them.  */
   if (available > 0)
     regs_available_for_popping &= ~available;
-  
+
   /* Otherwise if we need another popping register we can use
      the fourth argument register.  */
   else if (pops_needed)
@@ -12688,15 +12331,15 @@ thumb_exit (FILE *f, int reg_containing_return_addr)
          /* Register a4 is being used to hold part of the return value,
             but we have dire need of a free, low register.  */
          restore_a4 = TRUE;
-         
+
          asm_fprintf (f, "\tmov\t%r, %r\n",IP_REGNUM, LAST_ARG_REGNUM);
        }
-      
+
       if (reg_containing_return_addr != LAST_ARG_REGNUM)
        {
          /* The fourth argument register is available.  */
          regs_available_for_popping |= 1 << LAST_ARG_REGNUM;
-         
+
          --pops_needed;
        }
     }
@@ -12710,7 +12353,7 @@ thumb_exit (FILE *f, int reg_containing_return_addr)
     {
       /* The return address was popped into the lowest numbered register.  */
       regs_to_pop &= ~(1 << LR_REGNUM);
-      
+
       reg_containing_return_addr =
        number_of_first_bit_set (regs_available_for_popping);
 
@@ -12723,7 +12366,7 @@ thumb_exit (FILE *f, int reg_containing_return_addr)
   if (regs_available_for_popping)
     {
       int frame_pointer;
-      
+
       /* Work out which register currently contains the frame pointer.  */
       frame_pointer = number_of_first_bit_set (regs_available_for_popping);
 
@@ -12734,18 +12377,18 @@ thumb_exit (FILE *f, int reg_containing_return_addr)
       /* (Temporarily) remove it from the mask of popped registers.  */
       regs_available_for_popping &= ~(1 << frame_pointer);
       regs_to_pop &= ~(1 << ARM_HARD_FRAME_POINTER_REGNUM);
-      
+
       if (regs_available_for_popping)
        {
          int stack_pointer;
-         
+
          /* We popped the stack pointer as well,
             find the register that contains it.  */
          stack_pointer = number_of_first_bit_set (regs_available_for_popping);
 
          /* Move it into the stack register.  */
          asm_fprintf (f, "\tmov\t%r, %r\n", SP_REGNUM, stack_pointer);
-         
+
          /* At this point we have popped all necessary registers, so
             do not worry about restoring regs_available_for_popping
             to its correct value:
@@ -12762,7 +12405,7 @@ thumb_exit (FILE *f, int reg_containing_return_addr)
          regs_available_for_popping |= (1 << frame_pointer);
        }
     }
-  
+
   /* If we still have registers left on the stack, but we no longer have
      any registers into which we can pop them, then we must move the return
      address into the link register and make available the register that
@@ -12770,10 +12413,10 @@ thumb_exit (FILE *f, int reg_containing_return_addr)
   if (regs_available_for_popping == 0 && pops_needed > 0)
     {
       regs_available_for_popping |= 1 << reg_containing_return_addr;
-      
+
       asm_fprintf (f, "\tmov\t%r, %r\n", LR_REGNUM,
                   reg_containing_return_addr);
-      
+
       reg_containing_return_addr = LR_REGNUM;
     }
 
@@ -12783,7 +12426,7 @@ thumb_exit (FILE *f, int reg_containing_return_addr)
     {
       int  popped_into;
       int  move_to;
-      
+
       thumb_pushpop (f, regs_available_for_popping, FALSE, NULL,
                     regs_available_for_popping);
 
@@ -12798,13 +12441,13 @@ thumb_exit (FILE *f, int reg_containing_return_addr)
 
       --pops_needed;
     }
-  
+
   /* If we still have not popped everything then we must have only
      had one register available to us and we are now popping the SP.  */
   if (pops_needed > 0)
     {
       int  popped_into;
-      
+
       thumb_pushpop (f, regs_available_for_popping, FALSE, NULL,
                     regs_available_for_popping);
 
@@ -12825,7 +12468,7 @@ thumb_exit (FILE *f, int reg_containing_return_addr)
          asm_fprintf (f, "\tmov\t%r, %r\n", LR_REGNUM, LAST_ARG_REGNUM);
          reg_containing_return_addr = LR_REGNUM;
        }
-    
+
       asm_fprintf (f, "\tmov\t%r, %r\n", LAST_ARG_REGNUM, IP_REGNUM);
     }
 
@@ -12858,7 +12501,7 @@ thumb_pushpop (FILE *f, int mask, int push, int *cfa_offset, int real_regs)
       thumb_exit (f, -1);
       return;
     }
-      
+
   fprintf (f, "\t%s\t{", push ? "push" : "pop");
 
   /* Look at the low registers first.  */
@@ -12867,20 +12510,20 @@ thumb_pushpop (FILE *f, int mask, int push, int *cfa_offset, int real_regs)
       if (lo_mask & 1)
        {
          asm_fprintf (f, "%r", regno);
-         
+
          if ((lo_mask & ~1) != 0)
            fprintf (f, ", ");
 
          pushed_words++;
        }
     }
-  
+
   if (push && (mask & (1 << LR_REGNUM)))
     {
       /* Catch pushing the LR.  */
       if (mask & 0xFF)
        fprintf (f, ", ");
-      
+
       asm_fprintf (f, "%r", LR_REGNUM);
 
       pushed_words++;
@@ -12903,11 +12546,11 @@ thumb_pushpop (FILE *f, int mask, int push, int *cfa_offset, int real_regs)
        {
          if (mask & 0xFF)
            fprintf (f, ", ");
-         
+
          asm_fprintf (f, "%r", PC_REGNUM);
        }
     }
-       
+
   fprintf (f, "}\n");
 
   if (push && pushed_words && dwarf2out_do_frame ())
@@ -12944,7 +12587,7 @@ thumb_shiftable_const (unsigned HOST_WIDE_INT val)
 
   if (val == 0) /* XXX */
     return 0;
-  
+
   for (i = 0; i < 25; i++)
     if ((val & (mask << i)) == val)
       return 1;
@@ -12961,7 +12604,7 @@ thumb_far_jump_used_p (void)
 
   /* This test is only important for leaf functions.  */
   /* assert (!leaf_function_p ()); */
-  
+
   /* If we have already decided that far jumps may be used,
      do not bother checking again, and always return true even if
      it turns out that they are not being used.  Once we have made
@@ -13019,7 +12662,7 @@ thumb_far_jump_used_p (void)
          return 1;
        }
     }
-  
+
   return 0;
 }
 
@@ -13034,7 +12677,7 @@ is_called_in_ARM_mode (tree func)
   if (TARGET_CALLEE_INTERWORKING && TREE_PUBLIC (func))
     return TRUE;
 
-#ifdef ARM_PE 
+#ifdef ARM_PE
   return lookup_attribute ("interfacearm", DECL_ATTRIBUTES (func)) != NULL_TREE;
 #else
   return FALSE;
@@ -13049,6 +12692,8 @@ thumb_unexpanded_epilogue (void)
   int live_regs_mask = 0;
   int high_regs_pushed = 0;
   int had_to_push_lr;
+  int size;
+  int mode;
 
   if (return_used_this_function)
     return "";
@@ -13056,16 +12701,23 @@ thumb_unexpanded_epilogue (void)
   if (IS_NAKED (arm_current_func_type ()))
     return "";
 
-  for (regno = 0; regno <= LAST_LO_REGNUM; regno++)
-    if (THUMB_REG_PUSHED_P (regno))
-      live_regs_mask |= 1 << regno;
+  live_regs_mask = thumb_compute_save_reg_mask ();
+  high_regs_pushed = bit_count (live_regs_mask & 0x0f00);
+
+  /* If we can deduce the registers used from the function's return value.
+     This is more reliable that examining regs_ever_live[] because that
+     will be set if the register is ever used in the function, not just if
+     the register is used to hold a return value.  */
+
+  if (current_function_return_rtx != 0)
+    mode = GET_MODE (current_function_return_rtx);
+  else
+    mode = DECL_MODE (DECL_RESULT (current_function_decl));
 
-  for (regno = 8; regno < 13; regno++)
-    if (THUMB_REG_PUSHED_P (regno))
-      high_regs_pushed++;
+  size = GET_MODE_SIZE (mode);
 
   /* The prolog may have pushed some high registers to use as
-     work registers.  eg the testsuite file:
+     work registers.  e.g. the testsuite file:
      gcc/testsuite/gcc/gcc.c-torture/execute/complex-2.c
      compiles to produce:
        push    {r4, r5, r6, r7, lr}
@@ -13073,39 +12725,27 @@ thumb_unexpanded_epilogue (void)
        mov     r6, r8
        push    {r6, r7}
      as part of the prolog.  We have to undo that pushing here.  */
-  
+
   if (high_regs_pushed)
     {
-      int mask = live_regs_mask;
+      int mask = live_regs_mask & 0xff;
       int next_hi_reg;
-      int size;
-      int mode;
-       
-      /* If we can deduce the registers used from the function's return value.
-        This is more reliable that examining regs_ever_live[] because that
-        will be set if the register is ever used in the function, not just if
-        the register is used to hold a return value.  */
-
-      if (current_function_return_rtx != 0)
-       mode = GET_MODE (current_function_return_rtx);
-      else
-       mode = DECL_MODE (DECL_RESULT (current_function_decl));
-
-      size = GET_MODE_SIZE (mode);
 
-      /* Unless we are returning a type of size > 12 register r3 is
-         available.  */
-      if (size < 13)
+      /* The available low registers depend on the size of the value we are
+         returning.  */
+      if (size <= 12)
        mask |=  1 << 3;
+      if (size <= 8)
+       mask |= 1 << 2;
 
       if (mask == 0)
        /* Oh dear!  We have no low registers into which we can pop
            high registers!  */
        internal_error
          ("no low registers available for popping high registers");
-      
+
       for (next_hi_reg = 8; next_hi_reg < 13; next_hi_reg++)
-       if (THUMB_REG_PUSHED_P (next_hi_reg))
+       if (live_regs_mask & (1 << next_hi_reg))
          break;
 
       while (high_regs_pushed)
@@ -13132,31 +12772,23 @@ thumb_unexpanded_epilogue (void)
                {
                  asm_fprintf (asm_out_file, "\tmov\t%r, %r\n", next_hi_reg,
                               regno);
-                 
+
                  for (next_hi_reg++; next_hi_reg < 13; next_hi_reg++)
-                   if (THUMB_REG_PUSHED_P (next_hi_reg))
+                   if (live_regs_mask & (1 << next_hi_reg))
                      break;
                }
            }
        }
+      live_regs_mask &= ~0x0f00;
     }
 
-  had_to_push_lr = (live_regs_mask || thumb_force_lr_save ());
-  
-  if (TARGET_BACKTRACE
-      && ((live_regs_mask & 0xFF) == 0)
-      && regs_ever_live [LAST_ARG_REGNUM] != 0)
-    {
-      /* The stack backtrace structure creation code had to
-        push R7 in order to get a work register, so we pop
-        it now.  */
-      live_regs_mask |= (1 << LAST_LO_REGNUM);
-    }
-  
+  had_to_push_lr = (live_regs_mask & (1 << LR_REGNUM)) != 0;
+  live_regs_mask &= 0xff;
+
   if (current_function_pretend_args_size == 0 || TARGET_BACKTRACE)
     {
-      if (had_to_push_lr
-         && !is_called_in_ARM_mode (current_function_decl))
+      /* Pop the return address into the PC.  */
+      if (had_to_push_lr)
        live_regs_mask |= 1 << PC_REGNUM;
 
       /* Either no argument registers were pushed or a backtrace
@@ -13165,38 +12797,54 @@ thumb_unexpanded_epilogue (void)
       if (live_regs_mask)
        thumb_pushpop (asm_out_file, live_regs_mask, FALSE, NULL,
                       live_regs_mask);
-      
+
       /* We have either just popped the return address into the
-        PC or it is was kept in LR for the entire function or
-        it is still on the stack because we do not want to
-        return by doing a pop {pc}.  */
-      if ((live_regs_mask & (1 << PC_REGNUM)) == 0)
-       thumb_exit (asm_out_file,
-                   (had_to_push_lr
-                    && is_called_in_ARM_mode (current_function_decl)) ?
-                   -1 : LR_REGNUM);
+        PC or it is was kept in LR for the entire function.  */
+      if (!had_to_push_lr)
+       thumb_exit (asm_out_file, LR_REGNUM);
     }
   else
     {
       /* Pop everything but the return address.  */
-      live_regs_mask &= ~(1 << PC_REGNUM);
-      
       if (live_regs_mask)
        thumb_pushpop (asm_out_file, live_regs_mask, FALSE, NULL,
                       live_regs_mask);
 
       if (had_to_push_lr)
-       /* Get the return address into a temporary register.  */
-       thumb_pushpop (asm_out_file, 1 << LAST_ARG_REGNUM, 0, NULL,
-                      1 << LAST_ARG_REGNUM);
-      
+       {
+         if (size > 12)
+           {
+             /* We have no free low regs, so save one.  */
+             asm_fprintf (asm_out_file, "\tmov\t%r, %r\n", IP_REGNUM,
+                          LAST_ARG_REGNUM);
+           }
+
+         /* Get the return address into a temporary register.  */
+         thumb_pushpop (asm_out_file, 1 << LAST_ARG_REGNUM, 0, NULL,
+                        1 << LAST_ARG_REGNUM);
+
+         if (size > 12)
+           {
+             /* Move the return address to lr.  */
+             asm_fprintf (asm_out_file, "\tmov\t%r, %r\n", LR_REGNUM,
+                          LAST_ARG_REGNUM);
+             /* Restore the low register.  */
+             asm_fprintf (asm_out_file, "\tmov\t%r, %r\n", LAST_ARG_REGNUM,
+                          IP_REGNUM);
+             regno = LR_REGNUM;
+           }
+         else
+           regno = LAST_ARG_REGNUM;
+       }
+      else
+       regno = LR_REGNUM;
+
       /* Remove the argument registers that were pushed onto the stack.  */
       asm_fprintf (asm_out_file, "\tadd\t%r, %r, #%d\n",
                   SP_REGNUM, SP_REGNUM,
                   current_function_pretend_args_size);
-      
-      thumb_exit (asm_out_file,
-                 had_to_push_lr ? LAST_ARG_REGNUM : LR_REGNUM);
+
+      thumb_exit (asm_out_file, regno);
     }
 
   return "";
@@ -13209,7 +12857,7 @@ arm_init_machine_status (void)
   struct machine_function *machine;
   machine = (machine_function *) ggc_alloc_cleared (sizeof (machine_function));
 
-#if ARM_FT_UNKNOWN != 0  
+#if ARM_FT_UNKNOWN != 0
   machine->func_type = ARM_FT_UNKNOWN;
 #endif
   return machine;
@@ -13302,9 +12950,10 @@ thumb_expand_prologue (void)
   arm_stack_offsets *offsets;
   unsigned long func_type;
   int regno;
+  unsigned long live_regs_mask;
 
   func_type = arm_current_func_type ();
-  
+
   /* Naked functions don't have prologues.  */
   if (IS_NAKED (func_type))
     return;
@@ -13315,6 +12964,11 @@ thumb_expand_prologue (void)
       return;
     }
 
+  /* Load the pic register before setting the frame pointer, so we can use r7
+     as a temporary work register.  */
+  if (flag_pic)
+    arm_load_pic_register ();
+
   offsets = arm_get_frame_offsets ();
 
   if (frame_pointer_needed)
@@ -13324,6 +12978,7 @@ thumb_expand_prologue (void)
       RTX_FRAME_RELATED_P (insn) = 1;
     }
 
+  live_regs_mask = thumb_compute_save_reg_mask ();
   amount = offsets->outgoing_args - offsets->saved_regs;
   if (amount)
     {
@@ -13352,7 +13007,7 @@ thumb_expand_prologue (void)
             been pushed at the start of the prologue and so we can corrupt
             it now.  */
          for (regno = LAST_ARG_REGNUM + 1; regno <= LAST_LO_REGNUM; regno++)
-           if (THUMB_REG_PUSHED_P (regno)
+           if (live_regs_mask & (1 << regno)
                && !(frame_pointer_needed
                     && (regno == THUMB_HARD_FRAME_POINTER_REGNUM)))
              break;
@@ -13384,7 +13039,7 @@ thumb_expand_prologue (void)
 
              /* Restore the low register's original value.  */
              emit_insn (gen_movsi (reg, spare));
-             
+
              /* Emit a USE of the restored scratch register, so that flow
                 analysis will not consider the restore redundant.  The
                 register won't be used again in this function and isn't
@@ -13416,19 +13071,13 @@ thumb_expand_prologue (void)
        emit_insn (gen_stack_tie (stack_pointer_rtx,
                                  hard_frame_pointer_rtx));
     }
-  
+
   if (current_function_profile || TARGET_NO_SCHED_PRO)
     emit_insn (gen_blockage ());
 
   cfun->machine->lr_save_eliminated = !thumb_force_lr_save ();
-  for (regno = 0; regno <= LAST_LO_REGNUM; regno++)
-    {
-      if (THUMB_REG_PUSHED_P (regno))
-        {
-          cfun->machine->lr_save_eliminated = 0;
-          break;
-        }
-    }
+  if (live_regs_mask & 0xff)
+    cfun->machine->lr_save_eliminated = 0;
 
   /* If the link register is being kept alive, with the return address in it,
      then make sure that it does not get reused by the ce2 pass.  */
@@ -13436,6 +13085,7 @@ thumb_expand_prologue (void)
     emit_insn (gen_prologue_use (gen_rtx_REG (SImode, LR_REGNUM)));
 }
 
+
 void
 thumb_expand_epilogue (void)
 {
@@ -13466,7 +13116,7 @@ thumb_expand_epilogue (void)
          emit_insn (gen_addsi3 (stack_pointer_rtx, stack_pointer_rtx, reg));
        }
     }
-      
+
   /* Emit a USE (stack_pointer_rtx), so that
      the stack adjustment will not be deleted.  */
   emit_insn (gen_prologue_use (stack_pointer_rtx));
@@ -13488,6 +13138,7 @@ static void
 thumb_output_function_prologue (FILE *f, HOST_WIDE_INT size ATTRIBUTE_UNUSED)
 {
   int live_regs_mask = 0;
+  int l_mask;
   int high_regs_pushed = 0;
   int cfa_offset = 0;
   int regno;
@@ -13504,7 +13155,7 @@ thumb_output_function_prologue (FILE *f, HOST_WIDE_INT size ATTRIBUTE_UNUSED)
       if (GET_CODE (XEXP (DECL_RTL (current_function_decl), 0)) != SYMBOL_REF)
        abort ();
       name = XSTR  (XEXP (DECL_RTL (current_function_decl), 0), 0);
-      
+
       /* Generate code sequence to switch us into Thumb mode.  */
       /* The .code 32 directive has already been emitted by
         ASM_DECLARE_FUNCTION_NAME.  */
@@ -13517,29 +13168,29 @@ thumb_output_function_prologue (FILE *f, HOST_WIDE_INT size ATTRIBUTE_UNUSED)
         is called from a Thumb encoded function elsewhere in the
         same file.  Hence the definition of STUB_NAME here must
         agree with the definition in gas/config/tc-arm.c.  */
-      
+
 #define STUB_NAME ".real_start_of"
-      
+
       fprintf (f, "\t.code\t16\n");
 #ifdef ARM_PE
       if (arm_dllexport_name_p (name))
         name = arm_strip_name_encoding (name);
-#endif        
+#endif
       asm_fprintf (f, "\t.globl %s%U%s\n", STUB_NAME, name);
       fprintf (f, "\t.thumb_func\n");
       asm_fprintf (f, "%s%U%s:\n", STUB_NAME, name);
     }
-    
+
   if (current_function_pretend_args_size)
     {
       if (cfun->machine->uses_anonymous_args)
        {
          int num_pushes;
-         
+
          fprintf (f, "\tpush\t{");
 
          num_pushes = ARM_NUM_INTS (current_function_pretend_args_size);
-         
+
          for (regno = LAST_ARG_REGNUM + 1 - num_pushes;
               regno <= LAST_ARG_REGNUM;
               regno++)
@@ -13549,7 +13200,7 @@ thumb_output_function_prologue (FILE *f, HOST_WIDE_INT size ATTRIBUTE_UNUSED)
          fprintf (f, "}\n");
        }
       else
-       asm_fprintf (f, "\tsub\t%r, %r, #%d\n", 
+       asm_fprintf (f, "\tsub\t%r, %r, #%d\n",
                     SP_REGNUM, SP_REGNUM,
                     current_function_pretend_args_size);
 
@@ -13564,26 +13215,22 @@ thumb_output_function_prologue (FILE *f, HOST_WIDE_INT size ATTRIBUTE_UNUSED)
        }
     }
 
-  for (regno = 0; regno <= LAST_LO_REGNUM; regno++)
-    if (THUMB_REG_PUSHED_P (regno))
-      live_regs_mask |= 1 << regno;
-
-  if (live_regs_mask || thumb_force_lr_save ())
-    live_regs_mask |= 1 << LR_REGNUM;
+  live_regs_mask = thumb_compute_save_reg_mask ();
+  /* Just low regs and lr. */
+  l_mask = live_regs_mask & 0x40ff;
 
   if (TARGET_BACKTRACE)
     {
       int    offset;
-      int    work_register = 0;
-      int    wr;
-      
+      int    work_register;
+
       /* We have been asked to create a stack backtrace structure.
          The code looks like this:
-        
+
         0   .align 2
         0   func:
          0     sub   SP, #16         Reserve space for 4 registers.
-        2     push  {R7}            Get a work register.
+        2     push  {R7}            Push low registers.
          4     add   R7, SP, #20     Get the stack pointer before the push.
          6     str   R7, [SP, #8]    Store the stack pointer (before reserving the space).
          8     mov   R7, PC          Get hold of the start of this code plus 12.
@@ -13595,25 +13242,8 @@ thumb_output_function_prologue (FILE *f, HOST_WIDE_INT size ATTRIBUTE_UNUSED)
         20     add   R7, SP, #16     Point at the start of the backtrace structure.
         22     mov   FP, R7          Put this value into the frame pointer.  */
 
-      if ((live_regs_mask & 0xFF) == 0)
-       {
-         /* See if the a4 register is free.  */
-
-         if (regs_ever_live [LAST_ARG_REGNUM] == 0)
-           work_register = LAST_ARG_REGNUM;
-         else    /* We must push a register of our own.  */
-           live_regs_mask |= (1 << LAST_LO_REGNUM);
-       }
+      work_register = thumb_find_work_register (live_regs_mask);
 
-      if (work_register == 0)
-       {
-         /* Select a register from the list that will be pushed to
-             use as our work register.  */
-         for (work_register = (LAST_LO_REGNUM + 1); work_register--;)
-           if ((1 << work_register) & live_regs_mask)
-             break;
-       }
-      
       asm_fprintf
        (f, "\tsub\t%r, %r, #16\t%@ Create stack backtrace structure\n",
         SP_REGNUM, SP_REGNUM);
@@ -13625,22 +13255,23 @@ thumb_output_function_prologue (FILE *f, HOST_WIDE_INT size ATTRIBUTE_UNUSED)
          dwarf2out_def_cfa (l, SP_REGNUM, cfa_offset);
        }
 
-      if (live_regs_mask)
-       thumb_pushpop (f, live_regs_mask, 1, &cfa_offset, live_regs_mask);
-      
-      for (offset = 0, wr = 1 << 15; wr != 0; wr >>= 1)
-       if (wr & live_regs_mask)
-         offset += 4;
-      
+      if (l_mask)
+       {
+         thumb_pushpop (f, l_mask, 1, &cfa_offset, l_mask);
+         offset = bit_count (l_mask);
+       }
+      else
+       offset = 0;
+
       asm_fprintf (f, "\tadd\t%r, %r, #%d\n", work_register, SP_REGNUM,
                   offset + 16 + current_function_pretend_args_size);
-      
+
       asm_fprintf (f, "\tstr\t%r, [%r, #%d]\n", work_register, SP_REGNUM,
                   offset + 4);
 
       /* Make sure that the instruction fetching the PC is in the right place
         to calculate "start of backtrace creation code + 12".  */
-      if (live_regs_mask)
+      if (l_mask)
        {
          asm_fprintf (f, "\tmov\t%r, %r\n", work_register, PC_REGNUM);
          asm_fprintf (f, "\tstr\t%r, [%r, #%d]\n", work_register, SP_REGNUM,
@@ -13660,7 +13291,7 @@ thumb_output_function_prologue (FILE *f, HOST_WIDE_INT size ATTRIBUTE_UNUSED)
          asm_fprintf (f, "\tstr\t%r, [%r, #%d]\n", work_register, SP_REGNUM,
                       offset + 12);
        }
-      
+
       asm_fprintf (f, "\tmov\t%r, %r\n", work_register, LR_REGNUM);
       asm_fprintf (f, "\tstr\t%r, [%r, #%d]\n", work_register, SP_REGNUM,
                   offset + 8);
@@ -13669,32 +13300,24 @@ thumb_output_function_prologue (FILE *f, HOST_WIDE_INT size ATTRIBUTE_UNUSED)
       asm_fprintf (f, "\tmov\t%r, %r\t\t%@ Backtrace structure created\n",
                   ARM_HARD_FRAME_POINTER_REGNUM, work_register);
     }
-  else if (live_regs_mask)
-    thumb_pushpop (f, live_regs_mask, 1, &cfa_offset, live_regs_mask);
+  else if (l_mask)
+    thumb_pushpop (f, l_mask, 1, &cfa_offset, l_mask);
 
-  for (regno = 8; regno < 13; regno++)
-    if (THUMB_REG_PUSHED_P (regno))
-      high_regs_pushed++;
+  high_regs_pushed = bit_count (live_regs_mask & 0x0f00);
 
   if (high_regs_pushed)
     {
       int pushable_regs = 0;
-      int mask = live_regs_mask & 0xff;
       int next_hi_reg;
 
       for (next_hi_reg = 12; next_hi_reg > LAST_LO_REGNUM; next_hi_reg--)
-       if (THUMB_REG_PUSHED_P (next_hi_reg))
+       if (live_regs_mask & (1 << next_hi_reg))
          break;
 
-      pushable_regs = mask;
+      pushable_regs = l_mask & 0xff;
 
       if (pushable_regs == 0)
-       {
-         /* Desperation time -- this probably will never happen.  */
-         if (THUMB_REG_PUSHED_P (LAST_ARG_REGNUM))
-           asm_fprintf (f, "\tmov\t%r, %r\n", IP_REGNUM, LAST_ARG_REGNUM);
-         mask = 1 << LAST_ARG_REGNUM;
-       }
+       pushable_regs = 1 << thumb_find_work_register (live_regs_mask);
 
       while (high_regs_pushed > 0)
        {
@@ -13702,34 +13325,30 @@ thumb_output_function_prologue (FILE *f, HOST_WIDE_INT size ATTRIBUTE_UNUSED)
 
          for (regno = LAST_LO_REGNUM; regno >= 0; regno--)
            {
-             if (mask & (1 << regno))
+             if (pushable_regs & (1 << regno))
                {
                  asm_fprintf (f, "\tmov\t%r, %r\n", regno, next_hi_reg);
-                 
+
                  high_regs_pushed--;
                  real_regs_mask |= (1 << next_hi_reg);
-                 
+
                  if (high_regs_pushed)
                    {
                      for (next_hi_reg--; next_hi_reg > LAST_LO_REGNUM;
                           next_hi_reg--)
-                       if (THUMB_REG_PUSHED_P (next_hi_reg))
+                       if (live_regs_mask & (1 << next_hi_reg))
                          break;
                    }
                  else
                    {
-                     mask &= ~((1 << regno) - 1);
+                     pushable_regs &= ~((1 << regno) - 1);
                      break;
                    }
                }
            }
 
-         thumb_pushpop (f, mask, 1, &cfa_offset, real_regs_mask);
+         thumb_pushpop (f, pushable_regs, 1, &cfa_offset, real_regs_mask);
        }
-
-      if (pushable_regs == 0
-         && (THUMB_REG_PUSHED_P (LAST_ARG_REGNUM)))
-       asm_fprintf (f, "\tmov\t%r, %r\n", LAST_ARG_REGNUM, IP_REGNUM);
     }
 }
 
@@ -13744,16 +13363,16 @@ thumb_load_double_from_address (rtx *operands)
   rtx offset;
   rtx arg1;
   rtx arg2;
-  
+
   if (GET_CODE (operands[0]) != REG)
     abort ();
-  
+
   if (GET_CODE (operands[1]) != MEM)
     abort ();
 
   /* Get the memory address.  */
   addr = XEXP (operands[1], 0);
-      
+
   /* Work out how the memory address is computed.  */
   switch (GET_CODE (addr))
     {
@@ -13772,25 +13391,25 @@ thumb_load_double_from_address (rtx *operands)
          output_asm_insn ("ldr\t%H0, %2", operands);
        }
       break;
-      
+
     case CONST:
       /* Compute <address> + 4 for the high order load.  */
       operands[2] = gen_rtx_MEM (SImode,
                                 plus_constant (XEXP (operands[1], 0), 4));
-      
+
       output_asm_insn ("ldr\t%0, %1", operands);
       output_asm_insn ("ldr\t%H0, %2", operands);
       break;
-         
+
     case PLUS:
       arg1   = XEXP (addr, 0);
       arg2   = XEXP (addr, 1);
-           
+
       if (CONSTANT_P (arg1))
        base = arg2, offset = arg1;
       else
        base = arg1, offset = arg2;
-  
+
       if (GET_CODE (base) != REG)
        abort ();
 
@@ -13800,17 +13419,17 @@ thumb_load_double_from_address (rtx *operands)
          int reg_offset = REGNO (offset);
          int reg_base   = REGNO (base);
          int reg_dest   = REGNO (operands[0]);
-         
+
          /* Add the base and offset registers together into the
              higher destination register.  */
          asm_fprintf (asm_out_file, "\tadd\t%r, %r, %r",
                       reg_dest + 1, reg_base, reg_offset);
-         
+
          /* Load the lower destination register from the address in
              the higher destination register.  */
          asm_fprintf (asm_out_file, "\tldr\t%r, [%r, #0]",
                       reg_dest, reg_dest + 1);
-         
+
          /* Load the higher destination register from its own address
              plus 4.  */
          asm_fprintf (asm_out_file, "\tldr\t%r, [%r, #4]",
@@ -13821,7 +13440,7 @@ thumb_load_double_from_address (rtx *operands)
          /* Compute <address> + 4 for the high order load.  */
          operands[2] = gen_rtx_MEM (SImode,
                                     plus_constant (XEXP (operands[1], 0), 4));
-         
+
          /* If the computed address is held in the low order register
             then load the high order register first, otherwise always
             load the low order register first.  */
@@ -13843,16 +13462,16 @@ thumb_load_double_from_address (rtx *operands)
          directly.  */
       operands[2] = gen_rtx_MEM (SImode,
                                 plus_constant (XEXP (operands[1], 0), 4));
-         
+
       output_asm_insn ("ldr\t%H0, %2", operands);
       output_asm_insn ("ldr\t%0, %1", operands);
       break;
-      
+
     default:
       abort ();
       break;
     }
-  
+
   return "";
 }
 
@@ -13893,7 +13512,7 @@ thumb_output_move_mem_multiple (int n, rtx *operands)
          operands[4] = operands[5];
          operands[5] = tmp;
        }
-      
+
       output_asm_insn ("ldmia\t%1!, {%4, %5, %6}", operands);
       output_asm_insn ("stmia\t%0!, {%4, %5, %6}", operands);
       break;
@@ -13919,13 +13538,13 @@ thumb_expand_movmemqi (rtx *operands)
       emit_insn (gen_movmem12b (out, in, out, in));
       len -= 12;
     }
-  
+
   if (len >= 8)
     {
       emit_insn (gen_movmem8b (out, in, out, in));
       len -= 8;
     }
-  
+
   if (len >= 4)
     {
       rtx reg = gen_reg_rtx (SImode);
@@ -13934,18 +13553,18 @@ thumb_expand_movmemqi (rtx *operands)
       len -= 4;
       offset += 4;
     }
-  
+
   if (len >= 2)
     {
       rtx reg = gen_reg_rtx (HImode);
-      emit_insn (gen_movhi (reg, gen_rtx_MEM (HImode, 
+      emit_insn (gen_movhi (reg, gen_rtx_MEM (HImode,
                                              plus_constant (in, offset))));
       emit_insn (gen_movhi (gen_rtx_MEM (HImode, plus_constant (out, offset)),
                            reg));
       len -= 2;
       offset += 2;
     }
-  
+
   if (len)
     {
       rtx reg = gen_reg_rtx (QImode);
@@ -13956,49 +13575,13 @@ thumb_expand_movmemqi (rtx *operands)
     }
 }
 
-int
-thumb_cmp_operand (rtx op, enum machine_mode mode)
-{
-  return ((GET_CODE (op) == CONST_INT
-          && INTVAL (op) < 256
-          && INTVAL (op) >= 0)
-         || s_register_operand (op, mode));
-}
-
-int
-thumb_cmpneg_operand (rtx op, enum machine_mode mode ATTRIBUTE_UNUSED)
-{
-  return (GET_CODE (op) == CONST_INT
-         && INTVAL (op) < 0
-         && INTVAL (op) > -256);
-}
-
-/* Return TRUE if a result can be stored in OP without clobbering the
-   condition code register.  Prior to reload we only accept a
-   register.  After reload we have to be able to handle memory as
-   well, since a pseudo may not get a hard reg and reload cannot
-   handle output-reloads on jump insns.
-
-   We could possibly handle mem before reload as well, but that might
-   complicate things with the need to handle increment
-   side-effects.  */
-
-int
-thumb_cbrch_target_operand (rtx op, enum machine_mode mode)
-{
-  return (s_register_operand (op, mode)
-         || ((reload_in_progress || reload_completed)
-             && memory_operand (op, mode)));
-}
-
-/* Handle storing a half-word to memory during reload.  */ 
 void
 thumb_reload_out_hi (rtx *operands)
 {
   emit_insn (gen_thumb_movhi_clobber (operands[0], operands[1], operands[2]));
 }
 
-/* Handle reading a half-word from memory during reload.  */ 
+/* Handle reading a half-word from memory during reload.  */
 void
 thumb_reload_in_hi (rtx *operands ATTRIBUTE_UNUSED)
 {
@@ -14013,7 +13596,7 @@ arm_get_strip_length (int c)
   switch (c)
     {
     ARM_NAME_ENCODING_LENGTHS
-      default: return 0; 
+      default: return 0;
     }
 }
 
@@ -14023,7 +13606,7 @@ const char *
 arm_strip_name_encoding (const char *name)
 {
   int skip;
-  
+
   while ((skip = arm_get_strip_length (* name)))
     name += skip;
 
@@ -14098,7 +13681,7 @@ aof_dump_pic_table (FILE *f)
               PIC_OFFSET_TABLE_REGNUM,
               PIC_OFFSET_TABLE_REGNUM);
   fputs ("|x$adcons|\n", f);
-  
+
   for (chain = aof_pic_chain; chain; chain = chain->next)
     {
       fputs ("\tDCD\t", f);
@@ -14252,62 +13835,6 @@ aof_file_end (void)
 }
 #endif /* AOF_ASSEMBLER */
 
-#ifdef OBJECT_FORMAT_ELF
-/* Switch to an arbitrary section NAME with attributes as specified
-   by FLAGS.  ALIGN specifies any known alignment requirements for
-   the section; 0 if the default should be used.
-
-   Differs from the default elf version only in the prefix character
-   used before the section type.  */
-
-static void
-arm_elf_asm_named_section (const char *name, unsigned int flags)
-{
-  char flagchars[10], *f = flagchars;
-
-  if (! named_section_first_declaration (name))
-    {
-      fprintf (asm_out_file, "\t.section\t%s\n", name);
-      return;
-    }
-
-  if (!(flags & SECTION_DEBUG))
-    *f++ = 'a';
-  if (flags & SECTION_WRITE)
-    *f++ = 'w';
-  if (flags & SECTION_CODE)
-    *f++ = 'x';
-  if (flags & SECTION_SMALL)
-    *f++ = 's';
-  if (flags & SECTION_MERGE)
-    *f++ = 'M';
-  if (flags & SECTION_STRINGS)
-    *f++ = 'S';
-  if (flags & SECTION_TLS)
-    *f++ = 'T';
-  *f = '\0';
-
-  fprintf (asm_out_file, "\t.section\t%s,\"%s\"", name, flagchars);
-
-  if (!(flags & SECTION_NOTYPE))
-    {
-      const char *type;
-
-      if (flags & SECTION_BSS)
-       type = "nobits";
-      else
-       type = "progbits";
-
-      fprintf (asm_out_file, ",%%%s", type);
-
-      if (flags & SECTION_ENTSIZE)
-       fprintf (asm_out_file, ",%d", flags & SECTION_ENTSIZE);
-    }
-
-  putc ('\n', asm_out_file);
-}
-#endif
-
 #ifndef ARM_PE
 /* Symbols in the text segment can be accessed without indirecting via the
    constant pool; it may take an extra binary operation, but this is still
@@ -14328,7 +13855,7 @@ arm_encode_section_info (tree decl, rtx rtl, int first)
   /* If we are referencing a function that is weak then encode a long call
      flag in the function name, otherwise if the function is static or
      or known to be defined in this file then encode a short call flag.  */
-  if (first && TREE_CODE_CLASS (TREE_CODE (decl)) == 'd')
+  if (first && DECL_P (decl))
     {
       if (TREE_CODE (decl) == FUNCTION_DECL && DECL_WEAK (decl))
         arm_encode_call_attribute (decl, LONG_CALL_FLAG_CHAR);
@@ -14442,15 +13969,15 @@ arm_output_load_gr (rtx *operands)
   rtx offset;
   rtx wcgr;
   rtx sum;
-  
+
   if (GET_CODE (operands [1]) != MEM
       || GET_CODE (sum = XEXP (operands [1], 0)) != PLUS
       || GET_CODE (reg = XEXP (sum, 0)) != REG
       || GET_CODE (offset = XEXP (sum, 1)) != CONST_INT
       || ((INTVAL (offset) < 1024) && (INTVAL (offset) > -1024)))
     return "wldrw%?\t%0, %1";
-  
-  /* Fix up an out-of-range load of a GR register.  */  
+
+  /* Fix up an out-of-range load of a GR register.  */
   output_asm_insn ("str%?\t%0, [sp, #-4]!\t@ Start of GR load expansion", & reg);
   wcgr = operands[0];
   operands[0] = reg;
@@ -14521,7 +14048,7 @@ arm_no_early_store_addr_dep (rtx producer, rtx consumer)
   if (GET_CODE (addr) == PARALLEL)
     addr = XVECEXP (addr, 0, 0);
   addr = XEXP (addr, 0);
-  
+
   return !reg_overlap_mentioned_p (value, addr);
 }
 
@@ -14546,7 +14073,7 @@ arm_no_early_alu_shift_dep (rtx producer, rtx consumer)
   if (GET_CODE (op) == PARALLEL)
     op = XVECEXP (op, 0, 0);
   op = XEXP (op, 1);
-  
+
   early_op = XEXP (op, 0);
   /* This is either an actual independent shift, or a shift applied to
      the first operand of another operation.  We want the whole shift
@@ -14578,7 +14105,7 @@ arm_no_early_alu_shift_value_dep (rtx producer, rtx consumer)
   if (GET_CODE (op) == PARALLEL)
     op = XVECEXP (op, 0, 0);
   op = XEXP (op, 1);
-  
+
   early_op = XEXP (op, 0);
 
   /* This is either an actual independent shift, or a shift applied to
@@ -14586,7 +14113,7 @@ arm_no_early_alu_shift_value_dep (rtx producer, rtx consumer)
      shifted, in either case.  */
   if (GET_CODE (early_op) != REG)
     early_op = XEXP (early_op, 0);
-  
+
   return !reg_overlap_mentioned_p (value, early_op);
 }
 
@@ -14610,7 +14137,7 @@ arm_no_early_mul_dep (rtx producer, rtx consumer)
   if (GET_CODE (op) == PARALLEL)
     op = XVECEXP (op, 0, 0);
   op = XEXP (op, 1);
-  
+
   return (GET_CODE (op) == PLUS
          && !reg_overlap_mentioned_p (value, XEXP (op, 0)));
 }
@@ -14672,8 +14199,7 @@ arm_get_cookie_size (tree type)
   if (!TARGET_AAPCS_BASED)
     return default_cxx_get_cookie_size (type);
 
-  size = build_int_2 (8, 0);
-  TREE_TYPE (size) = sizetype;
+  size = build_int_cst (sizetype, 8);
   return size;
 }
 
@@ -14696,6 +14222,25 @@ arm_cxx_cdtor_returns_this (void)
   return TARGET_AAPCS_BASED;
 }
 
+/* The EABI says that an inline function may never be the key
+   method.  */
+
+static bool
+arm_cxx_key_method_may_be_inline (void)
+{
+  return !TARGET_AAPCS_BASED;
+}
+
+/* The EABI says that the virtual table, etc., for a class must be
+   exported if it has a key method.  The EABI does not specific the
+   behavior if there is no key method, but there is no harm in
+   exporting the class data in that case too.  */
+
+static bool
+arm_cxx_export_class_data (void)
+{
+  return TARGET_AAPCS_BASED;
+}
 
 void
 arm_set_return_address (rtx source, rtx scratch)
@@ -14719,7 +14264,7 @@ arm_set_return_address (rtx source, rtx scratch)
          offsets = arm_get_frame_offsets ();
          delta = offsets->outgoing_args - (offsets->frame + 4);
 
-         
+
          if (delta >= 4096)
            {
              emit_insn (gen_addsi3 (scratch, stack_pointer_rtx,
@@ -14741,24 +14286,15 @@ void
 thumb_set_return_address (rtx source, rtx scratch)
 {
   arm_stack_offsets *offsets;
-  bool lr_saved;
   HOST_WIDE_INT delta;
   int reg;
   rtx addr;
+  unsigned long mask;
 
   emit_insn (gen_rtx_USE (VOIDmode, source));
-  lr_saved = FALSE;
-  for (reg = 0; reg <= LAST_LO_REGNUM; reg++)
-    {
-      if (THUMB_REG_PUSHED_P (reg))
-       {
-         lr_saved = TRUE;
-         break;
-       }
-    }
-  lr_saved |= thumb_force_lr_save ();
 
-  if (lr_saved)
+  mask = thumb_compute_save_reg_mask ();
+  if (mask & (1 << LR_REGNUM))
     {
       offsets = arm_get_frame_offsets ();
 
@@ -14778,7 +14314,7 @@ thumb_set_return_address (rtx source, rtx scratch)
        delta -= 16;
       /* The link register is always the first saved register.  */
       delta -= 4;
-      
+
       /* Construct the address.  */
       addr = gen_rtx_REG (SImode, reg);
       if ((reg != SP_REGNUM && delta >= 128)
@@ -14797,3 +14333,25 @@ thumb_set_return_address (rtx source, rtx scratch)
     emit_move_insn (gen_rtx_REG (Pmode, LR_REGNUM), source);
 }
 
+/* Implements target hook vector_mode_supported_p.  */
+bool
+arm_vector_mode_supported_p (enum machine_mode mode)
+{
+  if ((mode == V2SImode)
+      || (mode == V4HImode)
+      || (mode == V8QImode))
+    return true;
+
+  return false;
+}
+
+/* Implement TARGET_SHIFT_TRUNCATION_MASK.  SImode shifts use normal
+   ARM insns and therefore guarantee that the shift count is modulo 256.
+   DImode shifts (those implemented by lib1funcs.asm or by optabs.c)
+   guarantee no particular behavior for out-of-range counts.  */
+
+static unsigned HOST_WIDE_INT
+arm_shift_truncation_mask (enum machine_mode mode)
+{
+  return mode == SImode ? 255 : 0;
+}