OSDN Git Service

* config.gcc: Add an extra_header for ARM targets.
[pf3gnuchains/gcc-fork.git] / gcc / local-alloc.c
index 62142f0..8e06b8d 100644 (file)
@@ -1,23 +1,23 @@
 /* Allocate registers within a basic block, for GNU compiler.
    Copyright (C) 1987, 1988, 1991, 1993, 1994, 1995, 1996, 1997, 1998,
-   1999, 2000 Free Software Foundation, Inc.
+   1999, 2000, 2001, 2002, 2003 Free Software Foundation, Inc.
 
-This file is part of GNU CC.
+This file is part of GCC.
 
-GNU CC is free software; you can redistribute it and/or modify
-it under the terms of the GNU General Public License as published by
-the Free Software Foundation; either version 2, or (at your option)
-any later version.
+GCC is free software; you can redistribute it and/or modify it under
+the terms of the GNU General Public License as published by the Free
+Software Foundation; either version 2, or (at your option) any later
+version.
 
-GNU CC is distributed in the hope that it will be useful,
-but WITHOUT ANY WARRANTY; without even the implied warranty of
-MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
-GNU General Public License for more details.
+GCC is distributed in the hope that it will be useful, but WITHOUT ANY
+WARRANTY; without even the implied warranty of MERCHANTABILITY or
+FITNESS FOR A PARTICULAR PURPOSE.  See the GNU General Public License
+for more details.
 
 You should have received a copy of the GNU General Public License
-along with GNU CC; see the file COPYING.  If not, write to
-the Free Software Foundation, 59 Temple Place - Suite 330,
-Boston, MA 02111-1307, USA.  */
+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.  */
 
 /* Allocation of hard register numbers to pseudo registers is done in
    two passes.  In this pass we consider only regs that are born and
@@ -61,10 +61,12 @@ Boston, MA 02111-1307, USA.  */
 
 #include "config.h"
 #include "system.h"
+#include "coretypes.h"
+#include "tm.h"
+#include "hard-reg-set.h"
 #include "rtl.h"
 #include "tm_p.h"
 #include "flags.h"
-#include "hard-reg-set.h"
 #include "basic-block.h"
 #include "regs.h"
 #include "function.h"
@@ -73,18 +75,24 @@ Boston, MA 02111-1307, USA.  */
 #include "recog.h"
 #include "output.h"
 #include "toplev.h"
+#include "except.h"
+#include "integrate.h"
 \f
 /* Next quantity number available for allocation.  */
 
 static int next_qty;
 
-/* Information we maitain about each quantity.  */
+/* Information we maintain about each quantity.  */
 struct qty
 {
   /* The number of refs to quantity Q.  */
 
   int n_refs;
 
+  /* The frequency of uses of quantity Q.  */
+
+  int freq;
+
   /* Insn number (counting from head of basic block)
      where quantity Q was born.  -1 if birth has not been recorded.  */
 
@@ -138,12 +146,6 @@ struct qty
      or -1 if none was found.  */
 
   short phys_reg;
-
-  /* Nonzero if this quantity has been used in a SUBREG in some
-     way that is illegal.  */
-
-  char changes_mode;
-
 };
 
 static struct qty *qty;
@@ -239,7 +241,7 @@ static rtx this_insn;
 struct equivalence
 {
   /* Set when an attempt should be made to replace a register
-     with the associated src entry.  */
+     with the associated src_p entry.  */
 
   char replace;
 
@@ -249,7 +251,7 @@ struct equivalence
 
   rtx replacement;
 
-  rtx src;
+  rtx *src_p;
 
   /* Loop depth is used to recognize equivalences which appear
      to be present within the same loop (or in an inner loop).  */
@@ -281,9 +283,9 @@ static void update_equiv_regs       PARAMS ((void));
 static void no_equiv           PARAMS ((rtx, rtx, void *));
 static void block_alloc                PARAMS ((int));
 static int qty_sugg_compare            PARAMS ((int, int));
-static int qty_sugg_compare_1  PARAMS ((const PTR, const PTR));
+static int qty_sugg_compare_1  PARAMS ((const void *, const void *));
 static int qty_compare         PARAMS ((int, int));
-static int qty_compare_1       PARAMS ((const PTR, const PTR));
+static int qty_compare_1       PARAMS ((const void *, const void *));
 static int combine_regs                PARAMS ((rtx, rtx, int, int, rtx, int));
 static int reg_meets_class_p   PARAMS ((int, enum reg_class));
 static void update_qty_class   PARAMS ((int, int));
@@ -307,7 +309,7 @@ alloc_qty (regno, mode, size, birth)
      enum machine_mode mode;
      int size, birth;
 {
-  register int qtyno = next_qty++;
+  int qtyno = next_qty++;
 
   reg_qty[regno] = qtyno;
   reg_offset[regno] = 0;
@@ -321,7 +323,7 @@ alloc_qty (regno, mode, size, birth)
   qty[qtyno].min_class = reg_preferred_class (regno);
   qty[qtyno].alternate_class = reg_alternate_class (regno);
   qty[qtyno].n_refs = REG_N_REFS (regno);
-  qty[qtyno].changes_mode = REG_CHANGES_MODE (regno);
+  qty[qtyno].freq = REG_FREQ (regno);
 }
 \f
 /* Main entry point of this file.  */
@@ -329,8 +331,9 @@ alloc_qty (regno, mode, size, birth)
 int
 local_alloc ()
 {
-  register int b, i;
+  int i;
   int max_qty;
+  basic_block b;
 
   /* We need to keep track of whether or not we recorded a LABEL_REF so
      that we know if the jump optimizer needs to be rerun.  */
@@ -345,7 +348,8 @@ local_alloc ()
 
   /* Promote REG_EQUAL notes to REG_EQUIV notes and adjust status of affected
      registers.  */
-  update_equiv_regs ();
+  if (optimize)
+    update_equiv_regs ();
 
   /* This sets the maximum number of quantities we can have.  Quantity
      numbers start at zero and we can have one for each pseudo.  */
@@ -366,9 +370,6 @@ local_alloc ()
   reg_offset = (char *) xmalloc (max_regno * sizeof (char));
   reg_next_in_qty = (int *) xmalloc (max_regno * sizeof (int));
 
-  /* Allocate the reg_renumber array.  */
-  allocate_reg_info (max_regno, FALSE, TRUE);
-
   /* Determine which pseudo-registers can be allocated by local-alloc.
      In general, these are the registers used only in a single block and
      which only die once.
@@ -389,7 +390,7 @@ local_alloc ()
 
   /* Allocate each block's local registers, block by block.  */
 
-  for (b = 0; b < n_basic_blocks; b++)
+  FOR_EACH_BB (b)
     {
       /* NEXT_QTY indicates which elements of the `qty_...'
         vectors might need to be initialized because they were used
@@ -421,7 +422,7 @@ local_alloc ()
 
       next_qty = 0;
 
-      block_alloc (b);
+      block_alloc (b->index);
     }
 
   free (qty);
@@ -493,7 +494,7 @@ validate_equiv_mem (start, reg, memref)
        return 1;
 
       if (GET_CODE (insn) == CALL_INSN && ! RTX_UNCHANGING_P (memref)
-         && ! CONST_CALL_P (insn))
+         && ! CONST_OR_PURE_CALL_P (insn))
        return 0;
 
       note_stores (PATTERN (insn), validate_equiv_mem_from_store, NULL);
@@ -520,9 +521,9 @@ static int
 equiv_init_varies_p (x)
      rtx x;
 {
-  register RTX_CODE code = GET_CODE (x);
-  register int i;
-  register const char *fmt;
+  RTX_CODE code = GET_CODE (x);
+  int i;
+  const char *fmt;
 
   switch (code)
     {
@@ -535,12 +536,13 @@ equiv_init_varies_p (x)
     case CONST:
     case CONST_INT:
     case CONST_DOUBLE:
+    case CONST_VECTOR:
     case SYMBOL_REF:
     case LABEL_REF:
       return 0;
 
     case REG:
-      return reg_equiv[REGNO (x)].replace == 0 && rtx_varies_p (x);
+      return reg_equiv[REGNO (x)].replace == 0 && rtx_varies_p (x, 0);
 
     case ASM_OPERANDS:
       if (MEM_VOLATILE_P (x))
@@ -570,7 +572,7 @@ equiv_init_varies_p (x)
   return 0;
 }
 
-/* Returns non-zero if X (used to initialize register REGNO) is movable.
+/* Returns nonzero if X (used to initialize register REGNO) is movable.
    X is only movable if the registers it uses have equivalent initializations
    which appear to be within the same loop (or in an inner loop) and movable
    or if they are not candidates for local_alloc and don't vary.  */
@@ -604,7 +606,7 @@ equiv_init_movable_p (x, regno)
     case REG:
       return (reg_equiv[REGNO (x)].loop_depth >= reg_equiv[regno].loop_depth
              && reg_equiv[REGNO (x)].replace)
-            || (REG_BASIC_BLOCK (REGNO (x)) < 0 && ! rtx_varies_p (x));
+            || (REG_BASIC_BLOCK (REGNO (x)) < 0 && ! rtx_varies_p (x, 0));
 
     case UNSPEC_VOLATILE:
       return 0;
@@ -654,10 +656,10 @@ contains_replace_regs (x)
     case LABEL_REF:
     case SYMBOL_REF:
     case CONST_DOUBLE:
+    case CONST_VECTOR:
     case PC:
     case CC0:
     case HIGH:
-    case LO_SUM:
       return 0;
 
     case REG:
@@ -704,6 +706,7 @@ memref_referenced_p (memref, x)
     case LABEL_REF:
     case SYMBOL_REF:
     case CONST_DOUBLE:
+    case CONST_VECTOR:
     case PC:
     case CC0:
     case HIGH:
@@ -775,6 +778,12 @@ memref_used_between_p (memref, start, end)
 }
 \f
 /* Return nonzero if the rtx X is invariant over the current function.  */
+/* ??? Actually, the places this is used in reload expect exactly what
+   is tested here, and not everything that is function invariant.  In
+   particular, the frame pointer and arg pointer are special cased;
+   pic_offset_table_rtx is not, and this will cause aborts when we
+   go to spill these things to memory.  */
+
 int
 function_invariant_p (x)
      rtx x;
@@ -802,7 +811,7 @@ static void
 update_equiv_regs ()
 {
   rtx insn;
-  int block;
+  basic_block bb;
   int loop_depth;
   regset_head cleared_regs;
   int clear_regnos = 0;
@@ -815,224 +824,209 @@ update_equiv_regs ()
   /* Scan the insns and find which registers have equivalences.  Do this
      in a separate scan of the insns because (due to -fcse-follow-jumps)
      a register can be set below its use.  */
-  loop_depth = 0;
-  for (insn = get_insns (); insn; insn = NEXT_INSN (insn))
+  FOR_EACH_BB (bb)
     {
-      rtx note;
-      rtx set;
-      rtx dest, src;
-      int regno;
+      loop_depth = bb->loop_depth;
 
-      if (GET_CODE (insn) == NOTE)
+      for (insn = bb->head; insn != NEXT_INSN (bb->end); insn = NEXT_INSN (insn))
        {
-         if (NOTE_LINE_NUMBER (insn) == NOTE_INSN_LOOP_BEG)
-           ++loop_depth;
-         else if (NOTE_LINE_NUMBER (insn) == NOTE_INSN_LOOP_END)
-           {
-             if (! loop_depth)
-               abort ();
-             --loop_depth;
-           }
-       }
+         rtx note;
+         rtx set;
+         rtx dest, src;
+         int regno;
 
-      if (! INSN_P (insn))
-       continue;
-
-      for (note = REG_NOTES (insn); note; note = XEXP (note, 1))
-       if (REG_NOTE_KIND (note) == REG_INC)
-         no_equiv (XEXP (note, 0), note, NULL);
+         if (! INSN_P (insn))
+           continue;
 
-      set = single_set (insn);
+         for (note = REG_NOTES (insn); note; note = XEXP (note, 1))
+           if (REG_NOTE_KIND (note) == REG_INC)
+             no_equiv (XEXP (note, 0), note, NULL);
 
-      /* If this insn contains more (or less) than a single SET,
-        only mark all destinations as having no known equivalence.  */
-      if (set == 0)
-       {
-         note_stores (PATTERN (insn), no_equiv, NULL);
-         continue;
-       }
-      else if (GET_CODE (PATTERN (insn)) == PARALLEL)
-       {
-         int i;
+         set = single_set (insn);
 
-         for (i = XVECLEN (PATTERN (insn), 0) - 1; i >= 0; i--)
+         /* If this insn contains more (or less) than a single SET,
+            only mark all destinations as having no known equivalence.  */
+         if (set == 0)
            {
-             rtx part = XVECEXP (PATTERN (insn), 0, i);
-             if (part != set)
-               note_stores (part, no_equiv, NULL);
+             note_stores (PATTERN (insn), no_equiv, NULL);
+             continue;
            }
-       }
+         else if (GET_CODE (PATTERN (insn)) == PARALLEL)
+           {
+             int i;
 
-      dest = SET_DEST (set);
-      src = SET_SRC (set);
-
-      /* If this sets a MEM to the contents of a REG that is only used
-        in a single basic block, see if the register is always equivalent
-        to that memory location and if moving the store from INSN to the
-        insn that set REG is safe.  If so, put a REG_EQUIV note on the
-        initializing insn.
-
-        Don't add a REG_EQUIV note if the insn already has one.  The existing
-        REG_EQUIV is likely more useful than the one we are adding.
-
-        If one of the regs in the address has reg_equiv[REGNO].replace set,
-        then we can't add this REG_EQUIV note.  The reg_equiv[REGNO].replace
-        optimization may move the set of this register immediately before
-        insn, which puts it after reg_equiv[REGNO].init_insns, and hence
-        the mention in the REG_EQUIV note would be to an uninitialized
-        pseudo.  */
-      /* ????? This test isn't good enough; we might see a MEM with a use of
-        a pseudo register before we see its setting insn that will cause
-        reg_equiv[].replace for that pseudo to be set.
-        Equivalences to MEMs should be made in another pass, after the
-        reg_equiv[].replace information has been gathered.  */
-
-      if (GET_CODE (dest) == MEM && GET_CODE (src) == REG
-         && (regno = REGNO (src)) >= FIRST_PSEUDO_REGISTER
-         && REG_BASIC_BLOCK (regno) >= 0
-         && REG_N_SETS (regno) == 1
-         && reg_equiv[regno].init_insns != 0
-         && reg_equiv[regno].init_insns != const0_rtx
-         && ! find_reg_note (XEXP (reg_equiv[regno].init_insns, 0),
-                             REG_EQUIV, NULL_RTX)
-         && ! contains_replace_regs (XEXP (dest, 0)))
-       {
-         rtx init_insn = XEXP (reg_equiv[regno].init_insns, 0);
-         if (validate_equiv_mem (init_insn, src, dest)
-             && ! memref_used_between_p (dest, init_insn, insn))
-           REG_NOTES (init_insn)
-             = gen_rtx_EXPR_LIST (REG_EQUIV, dest, REG_NOTES (init_insn));
-       }
+             for (i = XVECLEN (PATTERN (insn), 0) - 1; i >= 0; i--)
+               {
+                 rtx part = XVECEXP (PATTERN (insn), 0, i);
+                 if (part != set)
+                   note_stores (part, no_equiv, NULL);
+               }
+           }
 
-      /* We only handle the case of a pseudo register being set
-        once, or always to the same value.  */
-      /* ??? The mn10200 port breaks if we add equivalences for
-        values that need an ADDRESS_REGS register and set them equivalent
-        to a MEM of a pseudo.  The actual problem is in the over-conservative
-        handling of INPADDR_ADDRESS / INPUT_ADDRESS / INPUT triples in
-        calculate_needs, but we traditionally work around this problem
-        here by rejecting equivalences when the destination is in a register
-        that's likely spilled.  This is fragile, of course, since the
-        preferred class of a pseudo depends on all instructions that set
-        or use it.  */
-
-      if (GET_CODE (dest) != REG
-         || (regno = REGNO (dest)) < FIRST_PSEUDO_REGISTER
-         || reg_equiv[regno].init_insns == const0_rtx
-         || (CLASS_LIKELY_SPILLED_P (reg_preferred_class (regno))
-             && GET_CODE (src) == MEM))
-       {
-         /* This might be seting a SUBREG of a pseudo, a pseudo that is
-            also set somewhere else to a constant.  */
-         note_stores (set, no_equiv, NULL);
-         continue;
-       }
+         dest = SET_DEST (set);
+         src = SET_SRC (set);
+
+         /* If this sets a MEM to the contents of a REG that is only used
+            in a single basic block, see if the register is always equivalent
+            to that memory location and if moving the store from INSN to the
+            insn that set REG is safe.  If so, put a REG_EQUIV note on the
+            initializing insn.
+
+            Don't add a REG_EQUIV note if the insn already has one.  The existing
+            REG_EQUIV is likely more useful than the one we are adding.
+
+            If one of the regs in the address has reg_equiv[REGNO].replace set,
+            then we can't add this REG_EQUIV note.  The reg_equiv[REGNO].replace
+            optimization may move the set of this register immediately before
+            insn, which puts it after reg_equiv[REGNO].init_insns, and hence
+            the mention in the REG_EQUIV note would be to an uninitialized
+            pseudo.  */
+         /* ????? This test isn't good enough; we might see a MEM with a use of
+            a pseudo register before we see its setting insn that will cause
+            reg_equiv[].replace for that pseudo to be set.
+            Equivalences to MEMs should be made in another pass, after the
+            reg_equiv[].replace information has been gathered.  */
+
+         if (GET_CODE (dest) == MEM && GET_CODE (src) == REG
+             && (regno = REGNO (src)) >= FIRST_PSEUDO_REGISTER
+             && REG_BASIC_BLOCK (regno) >= 0
+             && REG_N_SETS (regno) == 1
+             && reg_equiv[regno].init_insns != 0
+             && reg_equiv[regno].init_insns != const0_rtx
+             && ! find_reg_note (XEXP (reg_equiv[regno].init_insns, 0),
+                                 REG_EQUIV, NULL_RTX)
+             && ! contains_replace_regs (XEXP (dest, 0)))
+           {
+             rtx init_insn = XEXP (reg_equiv[regno].init_insns, 0);
+             if (validate_equiv_mem (init_insn, src, dest)
+                 && ! memref_used_between_p (dest, init_insn, insn))
+               REG_NOTES (init_insn)
+                 = gen_rtx_EXPR_LIST (REG_EQUIV, dest, REG_NOTES (init_insn));
+           }
 
-      note = find_reg_note (insn, REG_EQUAL, NULL_RTX);
-
-      /* cse sometimes generates function invariants, but doesn't put a
-        REG_EQUAL note on the insn.  Since this note would be redundant,
-         there's no point creating it earlier than here.  */
-      if (! note && ! rtx_varies_p (src))
-       REG_NOTES (insn)
-         = note = gen_rtx_EXPR_LIST (REG_EQUAL, src, REG_NOTES (insn));
-
-      /* Don't bother considering a REG_EQUAL note containing an EXPR_LIST
-        since it represents a function call */
-      if (note && GET_CODE (XEXP (note, 0)) == EXPR_LIST)
-       note = NULL_RTX;
-
-      if (REG_N_SETS (regno) != 1
-         && (! note
-             || rtx_varies_p (XEXP (note, 0))
-             || (reg_equiv[regno].replacement
-                 && ! rtx_equal_p (XEXP (note, 0),
-                                   reg_equiv[regno].replacement))))
-       {
-         no_equiv (dest, set, NULL);
-         continue;
-       }
-      /* Record this insn as initializing this register.  */
-      reg_equiv[regno].init_insns
-       = gen_rtx_INSN_LIST (VOIDmode, insn, reg_equiv[regno].init_insns);
-
-      /* If this register is known to be equal to a constant, record that
-        it is always equivalent to the constant.  */
-      if (note && ! rtx_varies_p (XEXP (note, 0)))
-       PUT_MODE (note, (enum machine_mode) REG_EQUIV);
-
-      /* If this insn introduces a "constant" register, decrease the priority
-        of that register.  Record this insn if the register is only used once
-        more and the equivalence value is the same as our source.
-
-        The latter condition is checked for two reasons:  First, it is an
-        indication that it may be more efficient to actually emit the insn
-        as written (if no registers are available, reload will substitute
-        the equivalence).  Secondly, it avoids problems with any registers
-        dying in this insn whose death notes would be missed.
-
-        If we don't have a REG_EQUIV note, see if this insn is loading
-        a register used only in one basic block from a MEM.  If so, and the
-        MEM remains unchanged for the life of the register, add a REG_EQUIV
-        note.  */
-
-      note = find_reg_note (insn, REG_EQUIV, NULL_RTX);
-
-      if (note == 0 && REG_BASIC_BLOCK (regno) >= 0
-         && GET_CODE (SET_SRC (set)) == MEM
-         && validate_equiv_mem (insn, dest, SET_SRC (set)))
-       REG_NOTES (insn) = note = gen_rtx_EXPR_LIST (REG_EQUIV, SET_SRC (set),
-                                                    REG_NOTES (insn));
-
-      if (note)
-       {
-         int regno = REGNO (dest);
-
-         /* Record whether or not we created a REG_EQUIV note for a LABEL_REF.
-            We might end up substituting the LABEL_REF for uses of the
-            pseudo here or later.  That kind of transformation may turn an
-            indirect jump into a direct jump, in which case we must rerun the
-            jump optimizer to ensure that the JUMP_LABEL fields are valid.  */
-         if (GET_CODE (XEXP (note, 0)) == LABEL_REF
-             || (GET_CODE (XEXP (note, 0)) == CONST
-                 && GET_CODE (XEXP (XEXP (note, 0), 0)) == PLUS
-                 && (GET_CODE (XEXP (XEXP (XEXP (note, 0), 0), 0))
-                     == LABEL_REF)))
-           recorded_label_ref = 1;
-
-         reg_equiv[regno].replacement = XEXP (note, 0);
-         reg_equiv[regno].src = src;
-         reg_equiv[regno].loop_depth = loop_depth;
-
-         /* Don't mess with things live during setjmp.  */
-         if (REG_LIVE_LENGTH (regno) >= 0)
+         /* We only handle the case of a pseudo register being set
+            once, or always to the same value.  */
+         /* ??? The mn10200 port breaks if we add equivalences for
+            values that need an ADDRESS_REGS register and set them equivalent
+            to a MEM of a pseudo.  The actual problem is in the over-conservative
+            handling of INPADDR_ADDRESS / INPUT_ADDRESS / INPUT triples in
+            calculate_needs, but we traditionally work around this problem
+            here by rejecting equivalences when the destination is in a register
+            that's likely spilled.  This is fragile, of course, since the
+            preferred class of a pseudo depends on all instructions that set
+            or use it.  */
+
+         if (GET_CODE (dest) != REG
+             || (regno = REGNO (dest)) < FIRST_PSEUDO_REGISTER
+             || reg_equiv[regno].init_insns == const0_rtx
+             || (CLASS_LIKELY_SPILLED_P (reg_preferred_class (regno))
+                 && GET_CODE (src) == MEM))
            {
-             /* Note that the statement below does not affect the priority
-                in local-alloc!  */
-             REG_LIVE_LENGTH (regno) *= 2;
-
-
-             /* If the register is referenced exactly twice, meaning it is
-                set once and used once, indicate that the reference may be
-                replaced by the equivalence we computed above.  Do this
-                even if the register is only used in one block so that
-                dependencies can be handled where the last register is
-                used in a different block (i.e. HIGH / LO_SUM sequences)
-                and to reduce the number of registers alive across calls.
-
-                It would be nice to use "loop_depth * 2" in the compare
-                below.  Unfortunately, LOOP_DEPTH need not be constant within
-                a basic block so this would be too complicated.
-
-                This case normally occurs when a parameter is read from
-                memory and then used exactly once, not in a loop.  */
-
-               if (REG_N_REFS (regno) == 2
-                   && (rtx_equal_p (XEXP (note, 0), src)
-                       || ! equiv_init_varies_p (src))
-                   && GET_CODE (insn) == INSN
-                   && equiv_init_movable_p (PATTERN (insn), regno))
-                 reg_equiv[regno].replace = 1;
+             /* This might be seting a SUBREG of a pseudo, a pseudo that is
+                also set somewhere else to a constant.  */
+             note_stores (set, no_equiv, NULL);
+             continue;
+           }
+
+         note = find_reg_note (insn, REG_EQUAL, NULL_RTX);
+
+         /* cse sometimes generates function invariants, but doesn't put a
+            REG_EQUAL note on the insn.  Since this note would be redundant,
+            there's no point creating it earlier than here.  */
+         if (! note && ! rtx_varies_p (src, 0))
+           note = set_unique_reg_note (insn, REG_EQUAL, src);
+
+         /* Don't bother considering a REG_EQUAL note containing an EXPR_LIST
+            since it represents a function call */
+         if (note && GET_CODE (XEXP (note, 0)) == EXPR_LIST)
+           note = NULL_RTX;
+
+         if (REG_N_SETS (regno) != 1
+             && (! note
+                 || rtx_varies_p (XEXP (note, 0), 0)
+                 || (reg_equiv[regno].replacement
+                     && ! rtx_equal_p (XEXP (note, 0),
+                                       reg_equiv[regno].replacement))))
+           {
+             no_equiv (dest, set, NULL);
+             continue;
+           }
+         /* Record this insn as initializing this register.  */
+         reg_equiv[regno].init_insns
+           = gen_rtx_INSN_LIST (VOIDmode, insn, reg_equiv[regno].init_insns);
+
+         /* If this register is known to be equal to a constant, record that
+            it is always equivalent to the constant.  */
+         if (note && ! rtx_varies_p (XEXP (note, 0), 0))
+           PUT_MODE (note, (enum machine_mode) REG_EQUIV);
+
+         /* If this insn introduces a "constant" register, decrease the priority
+            of that register.  Record this insn if the register is only used once
+            more and the equivalence value is the same as our source.
+
+            The latter condition is checked for two reasons:  First, it is an
+            indication that it may be more efficient to actually emit the insn
+            as written (if no registers are available, reload will substitute
+            the equivalence).  Secondly, it avoids problems with any registers
+            dying in this insn whose death notes would be missed.
+
+            If we don't have a REG_EQUIV note, see if this insn is loading
+            a register used only in one basic block from a MEM.  If so, and the
+            MEM remains unchanged for the life of the register, add a REG_EQUIV
+            note.  */
+
+         note = find_reg_note (insn, REG_EQUIV, NULL_RTX);
+
+         if (note == 0 && REG_BASIC_BLOCK (regno) >= 0
+             && GET_CODE (SET_SRC (set)) == MEM
+             && validate_equiv_mem (insn, dest, SET_SRC (set)))
+           REG_NOTES (insn) = note = gen_rtx_EXPR_LIST (REG_EQUIV, SET_SRC (set),
+                                                        REG_NOTES (insn));
+
+         if (note)
+           {
+             int regno = REGNO (dest);
+
+             /* Record whether or not we created a REG_EQUIV note for a LABEL_REF.
+                We might end up substituting the LABEL_REF for uses of the
+                pseudo here or later.  That kind of transformation may turn an
+                indirect jump into a direct jump, in which case we must rerun the
+                jump optimizer to ensure that the JUMP_LABEL fields are valid.  */
+             if (GET_CODE (XEXP (note, 0)) == LABEL_REF
+                 || (GET_CODE (XEXP (note, 0)) == CONST
+                     && GET_CODE (XEXP (XEXP (note, 0), 0)) == PLUS
+                     && (GET_CODE (XEXP (XEXP (XEXP (note, 0), 0), 0))
+                         == LABEL_REF)))
+               recorded_label_ref = 1;
+
+             reg_equiv[regno].replacement = XEXP (note, 0);
+             reg_equiv[regno].src_p = &SET_SRC (set);
+             reg_equiv[regno].loop_depth = loop_depth;
+
+             /* Don't mess with things live during setjmp.  */
+             if (REG_LIVE_LENGTH (regno) >= 0 && optimize)
+               {
+                 /* Note that the statement below does not affect the priority
+                    in local-alloc!  */
+                 REG_LIVE_LENGTH (regno) *= 2;
+
+
+                 /* If the register is referenced exactly twice, meaning it is
+                    set once and used once, indicate that the reference may be
+                    replaced by the equivalence we computed above.  Do this
+                    even if the register is only used in one block so that
+                    dependencies can be handled where the last register is
+                    used in a different block (i.e. HIGH / LO_SUM sequences)
+                    and to reduce the number of registers alive across
+                    calls.  */
+
+                   if (REG_N_REFS (regno) == 2
+                       && (rtx_equal_p (XEXP (note, 0), src)
+                           || ! equiv_init_varies_p (src))
+                       && GET_CODE (insn) == INSN
+                       && equiv_init_movable_p (PATTERN (insn), regno))
+                     reg_equiv[regno].replace = 1;
+               }
            }
        }
     }
@@ -1044,124 +1038,112 @@ update_equiv_regs ()
      can't replace the reference, and the initialzing reference is
      within the same loop (or in an inner loop), then move the register
      initialization just before the use, so that they are in the same
-     basic block.
-
-     Skip this optimization if loop_depth isn't initially zero since
-     that indicates a mismatch between loop begin and loop end notes
-     (i.e. gcc.dg/noncompile/920721-2.c).  */
-  block = n_basic_blocks - 1;
-  for (insn = (loop_depth == 0) ? get_last_insn () : NULL_RTX;
-       insn; insn = PREV_INSN (insn))
+     basic block.  */
+  FOR_EACH_BB_REVERSE (bb)
     {
-      rtx link;
-
-      if (! INSN_P (insn))
+      loop_depth = bb->loop_depth;
+      for (insn = bb->end; insn != PREV_INSN (bb->head); insn = PREV_INSN (insn))
        {
-         if (GET_CODE (insn) == NOTE)
+         rtx link;
+
+         if (! INSN_P (insn))
+           continue;
+
+         for (link = REG_NOTES (insn); link; link = XEXP (link, 1))
            {
-             if (NOTE_INSN_BASIC_BLOCK_P (insn))
-               block = NOTE_BASIC_BLOCK (insn)->index - 1;
-             else if (NOTE_LINE_NUMBER (insn) == NOTE_INSN_LOOP_BEG)
+             if (REG_NOTE_KIND (link) == REG_DEAD
+                 /* Make sure this insn still refers to the register.  */
+                 && reg_mentioned_p (XEXP (link, 0), PATTERN (insn)))
                {
-                 if (! loop_depth)
-                   abort ();
-                 --loop_depth;
-               }
-             else if (NOTE_LINE_NUMBER (insn) == NOTE_INSN_LOOP_END)
-               ++loop_depth;
-           }
+                 int regno = REGNO (XEXP (link, 0));
+                 rtx equiv_insn;
 
-         continue;
-       }
+                 if (! reg_equiv[regno].replace
+                     || reg_equiv[regno].loop_depth < loop_depth)
+                   continue;
 
-      for (link = REG_NOTES (insn); link; link = XEXP (link, 1))
-       {
-         if (REG_NOTE_KIND (link) == REG_DEAD
-             /* Make sure this insn still refers to the register.  */
-             && reg_mentioned_p (XEXP (link, 0), PATTERN (insn)))
-           {
-             int regno = REGNO (XEXP (link, 0));
-             rtx equiv_insn;
+                 /* reg_equiv[REGNO].replace gets set only when
+                    REG_N_REFS[REGNO] is 2, i.e. the register is set
+                    once and used once.  (If it were only set, but not used,
+                    flow would have deleted the setting insns.)  Hence
+                    there can only be one insn in reg_equiv[REGNO].init_insns.  */
+                 if (reg_equiv[regno].init_insns == NULL_RTX
+                     || XEXP (reg_equiv[regno].init_insns, 1) != NULL_RTX)
+                   abort ();
+                 equiv_insn = XEXP (reg_equiv[regno].init_insns, 0);
 
-             if (! reg_equiv[regno].replace
-                 || reg_equiv[regno].loop_depth < loop_depth)
-               continue;
+                 /* We may not move instructions that can throw, since
+                    that changes basic block boundaries and we are not
+                    prepared to adjust the CFG to match.  */
+                 if (can_throw_internal (equiv_insn))
+                   continue;
 
-             /* reg_equiv[REGNO].replace gets set only when
-                REG_N_REFS[REGNO] is 2, i.e. the register is set
-                once and used once.  (If it were only set, but not used,
-                flow would have deleted the setting insns.)  Hence
-                there can only be one insn in reg_equiv[REGNO].init_insns.  */
-             if (reg_equiv[regno].init_insns == NULL_RTX
-                 || XEXP (reg_equiv[regno].init_insns, 1) != NULL_RTX)
-               abort ();
-             equiv_insn = XEXP (reg_equiv[regno].init_insns, 0);
-
-             if (asm_noperands (PATTERN (equiv_insn)) < 0
-                 && validate_replace_rtx (regno_reg_rtx[regno],
-                                          reg_equiv[regno].src, insn))
-               {
-                 rtx equiv_link;
-                 rtx last_link;
-                 rtx note;
-
-                 /* Find the last note.  */
-                 for (last_link = link; XEXP (last_link, 1);
-                      last_link = XEXP (last_link, 1))
-                   ;
-
-                 /* Append the REG_DEAD notes from equiv_insn.  */
-                 equiv_link = REG_NOTES (equiv_insn);
-                 while (equiv_link)
+                 if (asm_noperands (PATTERN (equiv_insn)) < 0
+                     && validate_replace_rtx (regno_reg_rtx[regno],
+                                              *(reg_equiv[regno].src_p), insn))
                    {
-                     note = equiv_link;
-                     equiv_link = XEXP (equiv_link, 1);
-                     if (REG_NOTE_KIND (note) == REG_DEAD)
+                     rtx equiv_link;
+                     rtx last_link;
+                     rtx note;
+
+                     /* Find the last note.  */
+                     for (last_link = link; XEXP (last_link, 1);
+                          last_link = XEXP (last_link, 1))
+                       ;
+
+                     /* Append the REG_DEAD notes from equiv_insn.  */
+                     equiv_link = REG_NOTES (equiv_insn);
+                     while (equiv_link)
                        {
-                         remove_note (equiv_insn, note);
-                         XEXP (last_link, 1) = note;
-                         XEXP (note, 1) = NULL_RTX;
-                         last_link = note;
+                         note = equiv_link;
+                         equiv_link = XEXP (equiv_link, 1);
+                         if (REG_NOTE_KIND (note) == REG_DEAD)
+                           {
+                             remove_note (equiv_insn, note);
+                             XEXP (last_link, 1) = note;
+                             XEXP (note, 1) = NULL_RTX;
+                             last_link = note;
+                           }
                        }
+
+                     remove_death (regno, insn);
+                     REG_N_REFS (regno) = 0;
+                     REG_FREQ (regno) = 0;
+                     delete_insn (equiv_insn);
+
+                     reg_equiv[regno].init_insns
+                       = XEXP (reg_equiv[regno].init_insns, 1);
                    }
+                 /* Move the initialization of the register to just before
+                    INSN.  Update the flow information.  */
+                 else if (PREV_INSN (insn) != equiv_insn)
+                   {
+                     rtx new_insn;
 
-                 remove_death (regno, insn);
-                 REG_N_REFS (regno) = 0;
-                 PUT_CODE (equiv_insn, NOTE);
-                 NOTE_LINE_NUMBER (equiv_insn) = NOTE_INSN_DELETED;
-                 NOTE_SOURCE_FILE (equiv_insn) = 0;
-                 
-                 reg_equiv[regno].init_insns = 
-                   XEXP (reg_equiv[regno].init_insns, 1);
-               }
-             /* Move the initialization of the register to just before
-                INSN.  Update the flow information.  */
-             else if (PREV_INSN (insn) != equiv_insn)
-               {
-                 rtx new_insn;
+                     new_insn = emit_insn_before (PATTERN (equiv_insn), insn);
+                     REG_NOTES (new_insn) = REG_NOTES (equiv_insn);
+                     REG_NOTES (equiv_insn) = 0;
 
-                 new_insn = emit_insn_before (copy_rtx (PATTERN (equiv_insn)),
-                                              insn);
-                 REG_NOTES (PREV_INSN (insn)) = REG_NOTES (equiv_insn);
-                 REG_NOTES (equiv_insn) = 0;
+                     /* Make sure this insn is recognized before reload begins,
+                        otherwise eliminate_regs_in_insn will abort.  */
+                     INSN_CODE (new_insn) = INSN_CODE (equiv_insn);
 
-                 PUT_CODE (equiv_insn, NOTE);
-                 NOTE_LINE_NUMBER (equiv_insn) = NOTE_INSN_DELETED;
-                 NOTE_SOURCE_FILE (equiv_insn) = 0;
+                     delete_insn (equiv_insn);
 
-                 XEXP (reg_equiv[regno].init_insns, 0) = new_insn;
+                     XEXP (reg_equiv[regno].init_insns, 0) = new_insn;
 
-                 REG_BASIC_BLOCK (regno) = block >= 0 ? block : 0;
-                 REG_N_CALLS_CROSSED (regno) = 0;
-                 REG_LIVE_LENGTH (regno) = 2;
+                     REG_BASIC_BLOCK (regno) = bb->index;
+                     REG_N_CALLS_CROSSED (regno) = 0;
+                     REG_LIVE_LENGTH (regno) = 2;
 
-                 if (block >= 0 && insn == BLOCK_HEAD (block))
-                   BLOCK_HEAD (block) = PREV_INSN (insn);
+                     if (insn == bb->head)
+                       bb->head = PREV_INSN (insn);
 
-                 /* Remember to clear REGNO from all basic block's live
-                    info.  */
-                 SET_REGNO_REG_SET (&cleared_regs, regno);
-                 clear_regnos++;
+                     /* Remember to clear REGNO from all basic block's live
+                        info.  */
+                     SET_REGNO_REG_SET (&cleared_regs, regno);
+                     clear_regnos++;
+                   }
                }
            }
        }
@@ -1170,24 +1152,22 @@ update_equiv_regs ()
   /* Clear all dead REGNOs from all basic block's live info.  */
   if (clear_regnos)
     {
-      int j, l;
+      int j;
       if (clear_regnos > 8)
-        {
-         for (l = 0; l < n_basic_blocks; l++)
+       {
+         FOR_EACH_BB (bb)
            {
-             AND_COMPL_REG_SET (BASIC_BLOCK (l)->global_live_at_start,
-                                &cleared_regs);
-             AND_COMPL_REG_SET (BASIC_BLOCK (l)->global_live_at_end,
-                                &cleared_regs);
+             AND_COMPL_REG_SET (bb->global_live_at_start, &cleared_regs);
+             AND_COMPL_REG_SET (bb->global_live_at_end, &cleared_regs);
            }
        }
       else
-        EXECUTE_IF_SET_IN_REG_SET (&cleared_regs, 0, j,
-          {
-           for (l = 0; l < n_basic_blocks; l++)
+       EXECUTE_IF_SET_IN_REG_SET (&cleared_regs, 0, j,
+         {
+           FOR_EACH_BB (bb)
              {
-               CLEAR_REGNO_REG_SET (BASIC_BLOCK (l)->global_live_at_start, j);
-               CLEAR_REGNO_REG_SET (BASIC_BLOCK (l)->global_live_at_end, j);
+               CLEAR_REGNO_REG_SET (bb->global_live_at_start, j);
+               CLEAR_REGNO_REG_SET (bb->global_live_at_end, j);
              }
          });
     }
@@ -1199,7 +1179,7 @@ update_equiv_regs ()
 }
 
 /* Mark REG as having no known equivalence.
-   Some instructions might have been proceessed before and furnished
+   Some instructions might have been processed before and furnished
    with REG_EQUIV notes for this register; these notes will have to be
    removed.
    STORE is the piece of RTL that does the non-constant / conflicting
@@ -1235,9 +1215,9 @@ static void
 block_alloc (b)
      int b;
 {
-  register int i, q;
-  register rtx insn;
-  rtx note;
+  int i, q;
+  rtx insn;
+  rtx note, hard_reg;
   int insn_number = 0;
   int insn_count = 0;
   int max_uid = get_max_uid ();
@@ -1278,9 +1258,9 @@ block_alloc (b)
 
       if (INSN_P (insn))
        {
-         register rtx link, set;
-         register int win = 0;
-         register rtx r0, r1 = NULL_RTX;
+         rtx link, set;
+         int win = 0;
+         rtx r0, r1 = NULL_RTX;
          int combined_regno = -1;
          int i;
 
@@ -1322,7 +1302,7 @@ block_alloc (b)
              for (i = 1; i < recog_data.n_operands; i++)
                {
                  const char *p = recog_data.constraints[i];
-                 int this_match = (requires_inout (p));
+                 int this_match = requires_inout (p);
 
                  n_matching_alts += this_match;
                  if (this_match == recog_data.n_alternatives)
@@ -1347,7 +1327,7 @@ block_alloc (b)
                     must match operand zero.  In that case, skip any
                     operand that doesn't list operand 0 since we know that
                     the operand always conflicts with operand 0.  We
-                    ignore commutatity in this case to keep things simple.  */
+                    ignore commutativity in this case to keep things simple.  */
                  if (n_matching_alts == recog_data.n_alternatives
                      && 0 == requires_inout (recog_data.constraints[i]))
                    continue;
@@ -1357,10 +1337,24 @@ block_alloc (b)
                  /* If the operand is an address, find a register in it.
                     There may be more than one register, but we only try one
                     of them.  */
-                 if (recog_data.constraints[i][0] == 'p')
+                 if (recog_data.constraints[i][0] == 'p'
+                     || EXTRA_ADDRESS_CONSTRAINT (recog_data.constraints[i][0],
+                                                  recog_data.constraints[i]))
                    while (GET_CODE (r1) == PLUS || GET_CODE (r1) == MULT)
                      r1 = XEXP (r1, 0);
 
+                 /* Avoid making a call-saved register unnecessarily
+                     clobbered.  */
+                 hard_reg = get_hard_reg_initial_reg (cfun, r1);
+                 if (hard_reg != NULL_RTX)
+                   {
+                     if (GET_CODE (hard_reg) == REG
+                         && IN_RANGE (REGNO (hard_reg),
+                                      0, FIRST_PSEUDO_REGISTER - 1)
+                         && ! call_used_regs[REGNO (hard_reg)])
+                       continue;
+                   }
+
                  if (GET_CODE (r0) == REG || GET_CODE (r0) == SUBREG)
                    {
                      /* We have two priorities for hard register preferences.
@@ -1603,7 +1597,7 @@ block_alloc (b)
             discourage the register allocator from creating false
             dependencies.
 
-            The adjustment value is choosen to indicate that this qty
+            The adjustment value is chosen to indicate that this qty
             conflicts with all the qtys in the instructions immediately
             before and after the lifetime of this qty.
 
@@ -1689,13 +1683,14 @@ block_alloc (b)
 
 /* Note that the quotient will never be bigger than
    the value of floor_log2 times the maximum number of
-   times a register can occur in one insn (surely less than 100).
-   Multiplying this by 10000 can't overflow.
+   times a register can occur in one insn (surely less than 100)
+   weighted by frequency (max REG_FREQ_MAX).
+   Multiplying this by 10000/REG_FREQ_MAX can't overflow.
    QTY_CMP_PRI is also used by qty_sugg_compare.  */
 
 #define QTY_CMP_PRI(q)         \
-  ((int) (((double) (floor_log2 (qty[q].n_refs) * qty[q].n_refs * qty[q].size) \
-         / (qty[q].death - qty[q].birth)) * 10000))
+  ((int) (((double) (floor_log2 (qty[q].n_refs) * qty[q].freq * qty[q].size) \
+         / (qty[q].death - qty[q].birth)) * (10000 / REG_FREQ_MAX)))
 
 static int
 qty_compare (q1, q2)
@@ -1706,11 +1701,11 @@ qty_compare (q1, q2)
 
 static int
 qty_compare_1 (q1p, q2p)
-     const PTR q1p;
-     const PTR q2p;
+     const void *q1p;
+     const void *q2p;
 {
-  register int q1 = *(const int *) q1p, q2 = *(const int *) q2p;
-  register int tem = QTY_CMP_PRI (q2) - QTY_CMP_PRI (q1);
+  int q1 = *(const int *) q1p, q2 = *(const int *) q2p;
+  int tem = QTY_CMP_PRI (q2) - QTY_CMP_PRI (q1);
 
   if (tem != 0)
     return tem;
@@ -1736,7 +1731,7 @@ static int
 qty_sugg_compare (q1, q2)
      int q1, q2;
 {
-  register int tem = QTY_CMP_SUGG (q1) - QTY_CMP_SUGG (q2);
+  int tem = QTY_CMP_SUGG (q1) - QTY_CMP_SUGG (q2);
 
   if (tem != 0)
     return tem;
@@ -1746,11 +1741,11 @@ qty_sugg_compare (q1, q2)
 
 static int
 qty_sugg_compare_1 (q1p, q2p)
-     const PTR q1p;
-     const PTR q2p;
+     const void *q1p;
+     const void *q2p;
 {
-  register int q1 = *(const int *) q1p, q2 = *(const int *) q2p;
-  register int tem = QTY_CMP_SUGG (q1) - QTY_CMP_SUGG (q2);
+  int q1 = *(const int *) q1p, q2 = *(const int *) q2p;
+  int tem = QTY_CMP_SUGG (q1) - QTY_CMP_SUGG (q2);
 
   if (tem != 0)
     return tem;
@@ -1779,11 +1774,11 @@ qty_sugg_compare_1 (q1p, q2p)
    If we really combined them, we could lose if the pseudo lives
    across an insn that clobbers the hard reg (eg, movstr).
 
-   ALREADY_DEAD is non-zero if USEDREG is known to be dead even though
+   ALREADY_DEAD is nonzero if USEDREG is known to be dead even though
    there is no REG_DEAD note on INSN.  This occurs during the processing
    of REG_NO_CONFLICT blocks.
 
-   MAY_SAVE_COPYCOPY is non-zero if this insn is simply copying USEDREG to
+   MAY_SAVE_COPYCOPY is nonzero if this insn is simply copying USEDREG to
    SETREG or if the input and output must share a register.
    In that case, we record a hard reg suggestion in QTY_PHYS_COPY_SUGG.
 
@@ -1797,10 +1792,10 @@ combine_regs (usedreg, setreg, may_save_copy, insn_number, insn, already_dead)
      rtx insn;
      int already_dead;
 {
-  register int ureg, sreg;
-  register int offset = 0;
+  int ureg, sreg;
+  int offset = 0;
   int usize, ssize;
-  register int sqty;
+  int sqty;
 
   /* Determine the numbers and sizes of registers being used.  If a subreg
      is present that does not change the entire register, don't consider
@@ -1808,27 +1803,69 @@ combine_regs (usedreg, setreg, may_save_copy, insn_number, insn, already_dead)
 
   while (GET_CODE (usedreg) == SUBREG)
     {
-      if (GET_MODE_SIZE (GET_MODE (SUBREG_REG (usedreg))) > UNITS_PER_WORD)
-       may_save_copy = 0;
-      offset += SUBREG_WORD (usedreg);
-      usedreg = SUBREG_REG (usedreg);
+      rtx subreg = SUBREG_REG (usedreg);
+
+      if (GET_CODE (subreg) == REG)
+       {
+         if (GET_MODE_SIZE (GET_MODE (subreg)) > UNITS_PER_WORD)
+           may_save_copy = 0;
+
+         if (REGNO (subreg) < FIRST_PSEUDO_REGISTER)
+           offset += subreg_regno_offset (REGNO (subreg),
+                                          GET_MODE (subreg),
+                                          SUBREG_BYTE (usedreg),
+                                          GET_MODE (usedreg));
+         else
+           offset += (SUBREG_BYTE (usedreg)
+                     / REGMODE_NATURAL_SIZE (GET_MODE (usedreg)));
+       }
+
+      usedreg = subreg;
     }
+
   if (GET_CODE (usedreg) != REG)
     return 0;
+
   ureg = REGNO (usedreg);
-  usize = REG_SIZE (usedreg);
+  if (ureg < FIRST_PSEUDO_REGISTER)
+    usize = HARD_REGNO_NREGS (ureg, GET_MODE (usedreg));
+  else
+    usize = ((GET_MODE_SIZE (GET_MODE (usedreg))
+             + (REGMODE_NATURAL_SIZE (GET_MODE (usedreg)) - 1))
+            / REGMODE_NATURAL_SIZE (GET_MODE (usedreg)));
 
   while (GET_CODE (setreg) == SUBREG)
     {
-      if (GET_MODE_SIZE (GET_MODE (SUBREG_REG (setreg))) > UNITS_PER_WORD)
-       may_save_copy = 0;
-      offset -= SUBREG_WORD (setreg);
-      setreg = SUBREG_REG (setreg);
+      rtx subreg = SUBREG_REG (setreg);
+
+      if (GET_CODE (subreg) == REG)
+       {
+         if (GET_MODE_SIZE (GET_MODE (subreg)) > UNITS_PER_WORD)
+           may_save_copy = 0;
+
+         if (REGNO (subreg) < FIRST_PSEUDO_REGISTER)
+           offset -= subreg_regno_offset (REGNO (subreg),
+                                          GET_MODE (subreg),
+                                          SUBREG_BYTE (setreg),
+                                          GET_MODE (setreg));
+         else
+           offset -= (SUBREG_BYTE (setreg)
+                     / REGMODE_NATURAL_SIZE (GET_MODE (setreg)));
+       }
+
+      setreg = subreg;
     }
+
   if (GET_CODE (setreg) != REG)
     return 0;
+
   sreg = REGNO (setreg);
-  ssize = REG_SIZE (setreg);
+  if (sreg < FIRST_PSEUDO_REGISTER)
+    ssize = HARD_REGNO_NREGS (sreg, GET_MODE (setreg));
+  else
+    ssize = ((GET_MODE_SIZE (GET_MODE (setreg))
+             + (REGMODE_NATURAL_SIZE (GET_MODE (setreg)) - 1))
+            / REGMODE_NATURAL_SIZE (GET_MODE (setreg)));
 
   /* If UREG is a pseudo-register that hasn't already been assigned a
      quantity number, it means that it is not local to this block or dies
@@ -1939,9 +1976,10 @@ combine_regs (usedreg, setreg, may_save_copy, insn_number, insn, already_dead)
       /* Update info about quantity SQTY.  */
       qty[sqty].n_calls_crossed += REG_N_CALLS_CROSSED (sreg);
       qty[sqty].n_refs += REG_N_REFS (sreg);
+      qty[sqty].freq += REG_FREQ (sreg);
       if (usize < ssize)
        {
-         register int i;
+         int i;
 
          for (i = qty[sqty].first_reg; i >= 0; i = reg_next_in_qty[i])
            reg_offset[i] -= offset;
@@ -1965,7 +2003,7 @@ reg_meets_class_p (reg, class)
      int reg;
      enum reg_class class;
 {
-  register enum reg_class rclass = reg_preferred_class (reg);
+  enum reg_class rclass = reg_preferred_class (reg);
   return (reg_class_subset_p (rclass, class)
          || reg_class_subset_p (class, rclass));
 }
@@ -1984,9 +2022,6 @@ update_qty_class (qtyno, reg)
   rclass = reg_alternate_class (reg);
   if (reg_class_subset_p (rclass, qty[qtyno].alternate_class))
     qty[qtyno].alternate_class = rclass;
-
-  if (REG_CHANGES_MODE (reg))
-    qty[qtyno].changes_mode = 1;
 }
 \f
 /* Handle something which alters the value of an rtx REG.
@@ -2026,10 +2061,14 @@ reg_is_born (reg, birth)
      rtx reg;
      int birth;
 {
-  register int regno;
+  int regno;
 
   if (GET_CODE (reg) == SUBREG)
-    regno = REGNO (SUBREG_REG (reg)) + SUBREG_WORD (reg);
+    {
+      regno = REGNO (SUBREG_REG (reg));
+      if (regno < FIRST_PSEUDO_REGISTER)
+       regno = subreg_hard_regno (reg, 1);
+    }
   else
     regno = REGNO (reg);
 
@@ -2053,17 +2092,17 @@ reg_is_born (reg, birth)
     }
 }
 
-/* Record the death of REG in the current insn.  If OUTPUT_P is non-zero,
+/* Record the death of REG in the current insn.  If OUTPUT_P is nonzero,
    REG is an output that is dying (i.e., it is never used), otherwise it
    is an input (the normal case).
    If OUTPUT_P is 1, then we extend the life past the end of this insn.  */
 
 static void
 wipe_dead_reg (reg, output_p)
-     register rtx reg;
+     rtx reg;
      int output_p;
 {
-  register int regno = REGNO (reg);
+  int regno = REGNO (reg);
 
   /* If this insn has multiple results,
      and the dead reg is used in one of the results,
@@ -2122,7 +2161,7 @@ wipe_dead_reg (reg, output_p)
    If QTYNO crosses calls, insist on a register preserved by calls,
    unless ACCEPT_CALL_CLOBBERED is nonzero.
 
-   If JUST_TRY_SUGGESTED is non-zero, only try to see if the suggested
+   If JUST_TRY_SUGGESTED is nonzero, only try to see if the suggested
    register is available.  If not, return -1.  */
 
 static int
@@ -2135,14 +2174,10 @@ find_free_reg (class, mode, qtyno, accept_call_clobbered, just_try_suggested,
      int just_try_suggested;
      int born_index, dead_index;
 {
-  register int i, ins;
-#ifdef HARD_REG_SET
-  /* Declare it register if it's a scalar.  */
-  register
-#endif
-    HARD_REG_SET used, first_used;
+  int i, ins;
+  HARD_REG_SET first_used, used;
 #ifdef ELIMINABLE_REGS
-  static struct {int from, to; } eliminables[] = ELIMINABLE_REGS;
+  static const struct {const int from, to; } eliminables[] = ELIMINABLE_REGS;
 #endif
 
   /* Validate our parameters.  */
@@ -2188,10 +2223,8 @@ find_free_reg (class, mode, qtyno, accept_call_clobbered, just_try_suggested,
   SET_HARD_REG_BIT (used, FRAME_POINTER_REGNUM);
 #endif
 
-#ifdef CLASS_CANNOT_CHANGE_MODE
-  if (qty[qtyno].changes_mode)
-    IOR_HARD_REG_SET (used,
-                     reg_class_contents[(int) CLASS_CANNOT_CHANGE_MODE]);
+#ifdef CANNOT_CHANGE_MODE_CLASS
+  cannot_change_mode_set_regs (&used, mode, qty[qtyno].first_reg);
 #endif
 
   /* Normally, the registers that can be used for the first register in
@@ -2228,8 +2261,8 @@ find_free_reg (class, mode, qtyno, accept_call_clobbered, just_try_suggested,
              || accept_call_clobbered
              || ! HARD_REGNO_CALL_PART_CLOBBERED (regno, mode)))
        {
-         register int j;
-         register int size1 = HARD_REGNO_NREGS (regno, mode);
+         int j;
+         int size1 = HARD_REGNO_NREGS (regno, mode);
          for (j = 1; j < size1 && ! TEST_HARD_REG_BIT (used, regno + j); j++);
          if (j == size1)
            {
@@ -2281,16 +2314,16 @@ find_free_reg (class, mode, qtyno, accept_call_clobbered, just_try_suggested,
 }
 \f
 /* Mark that REGNO with machine-mode MODE is live starting from the current
-   insn (if LIFE is non-zero) or dead starting at the current insn (if LIFE
+   insn (if LIFE is nonzero) or dead starting at the current insn (if LIFE
    is zero).  */
 
 static void
 mark_life (regno, mode, life)
-     register int regno;
+     int regno;
      enum machine_mode mode;
      int life;
 {
-  register int j = HARD_REGNO_NREGS (regno, mode);
+  int j = HARD_REGNO_NREGS (regno, mode);
   if (life)
     while (--j >= 0)
       SET_HARD_REG_BIT (regs_live, regno + j);
@@ -2300,7 +2333,7 @@ mark_life (regno, mode, life)
 }
 
 /* Mark register number REGNO (with machine-mode MODE) as live (if LIFE
-   is non-zero) or dead (if LIFE is zero) from insn number BIRTH (inclusive)
+   is nonzero) or dead (if LIFE is zero) from insn number BIRTH (inclusive)
    to insn number DEATH (exclusive).  */
 
 static void
@@ -2309,7 +2342,7 @@ post_mark_life (regno, mode, life, birth, death)
      enum machine_mode mode;
      int life, birth, death;
 {
-  register int j = HARD_REGNO_NREGS (regno, mode);
+  int j = HARD_REGNO_NREGS (regno, mode);
 #ifdef HARD_REG_SET
   /* Declare it register if it's a scalar.  */
   register
@@ -2392,44 +2425,56 @@ requires_inout (p)
   int found_zero = 0;
   int reg_allowed = 0;
   int num_matching_alts = 0;
+  int len;
 
-  while ((c = *p++))
-    switch (c)
-      {
-      case '=':  case '+':  case '?':
-      case '#':  case '&':  case '!':
-      case '*':  case '%':
-      case '1':  case '2':  case '3':  case '4': case '5':
-      case '6':  case '7':  case '8':  case '9':
-      case 'm':  case '<':  case '>':  case 'V':  case 'o':
-      case 'E':  case 'F':  case 'G':  case 'H':
-      case 's':  case 'i':  case 'n':
-      case 'I':  case 'J':  case 'K':  case 'L':
-      case 'M':  case 'N':  case 'O':  case 'P':
-      case 'X':
-       /* These don't say anything we care about.  */
-       break;
+  for ( ; (c = *p); p += len)
+    {
+      len = CONSTRAINT_LEN (c, p);
+      switch (c)
+       {
+       case '=':  case '+':  case '?':
+       case '#':  case '&':  case '!':
+       case '*':  case '%':
+       case 'm':  case '<':  case '>':  case 'V':  case 'o':
+       case 'E':  case 'F':  case 'G':  case 'H':
+       case 's':  case 'i':  case 'n':
+       case 'I':  case 'J':  case 'K':  case 'L':
+       case 'M':  case 'N':  case 'O':  case 'P':
+       case 'X':
+         /* These don't say anything we care about.  */
+         break;
 
-      case ',':
-       if (found_zero && ! reg_allowed)
-         num_matching_alts++;
+       case ',':
+         if (found_zero && ! reg_allowed)
+           num_matching_alts++;
 
-       found_zero = reg_allowed = 0;
-       break;
+         found_zero = reg_allowed = 0;
+         break;
 
-      case '0':
-       found_zero = 1;
-       break;
+       case '0':
+         found_zero = 1;
+         break;
 
-      default:
-       if (REG_CLASS_FROM_LETTER (c) == NO_REGS)
+       case '1':  case '2':  case '3':  case '4': case '5':
+       case '6':  case '7':  case '8':  case '9':
+         /* Skip the balance of the matching constraint.  */
+         do
+           p++;
+         while (ISDIGIT (*p));
+         len = 0;
          break;
-       /* FALLTHRU */
-      case 'p':
-      case 'g': case 'r':
-       reg_allowed = 1;
-       break;
-      }
+
+       default:
+         if (REG_CLASS_FROM_CONSTRAINT (c, p) == NO_REGS
+             && !EXTRA_ADDRESS_CONSTRAINT (c, p))
+           break;
+         /* FALLTHRU */
+       case 'p':
+       case 'g': case 'r':
+         reg_allowed = 1;
+         break;
+       }
+    }
 
   if (found_zero && ! reg_allowed)
     num_matching_alts++;
@@ -2441,7 +2486,7 @@ void
 dump_local_alloc (file)
      FILE *file;
 {
-  register int i;
+  int i;
   for (i = FIRST_PSEUDO_REGISTER; i < max_regno; i++)
     if (reg_renumber[i] != -1)
       fprintf (file, ";; Register %d in %d.\n", i, reg_renumber[i]);