OSDN Git Service

2000-08-10 Alexandre Petit-Bianco <apbianco@cygnus.com>
[pf3gnuchains/gcc-fork.git] / gcc / integrate.c
index af7f9d2..2cf8813 100644 (file)
@@ -1,5 +1,6 @@
 /* Procedure integration for GNU CC.
-   Copyright (C) 1988, 91, 93-98, 1999 Free Software Foundation, Inc.
+   Copyright (C) 1988, 1991, 1993, 1994, 1995, 1996, 1997, 1998,
+   1999, 2000 Free Software Foundation, Inc.
    Contributed by Michael Tiemann (tiemann@cygnus.com)
 
 This file is part of GNU CC.
@@ -25,6 +26,7 @@ Boston, MA 02111-1307, USA.  */
 
 #include "rtl.h"
 #include "tree.h"
+#include "tm_p.h"
 #include "regs.h"
 #include "flags.h"
 #include "insn-config.h"
@@ -38,6 +40,7 @@ Boston, MA 02111-1307, USA.  */
 #include "function.h"
 #include "toplev.h"
 #include "intl.h"
+#include "loop.h"
 
 #include "obstack.h"
 #define        obstack_chunk_alloc     xmalloc
@@ -59,32 +62,33 @@ extern struct obstack *function_maybepermanent_obstack;
    ? (1 + (3 * list_length (DECL_ARGUMENTS (DECL))) / 2) \
    : (8 * (8 + list_length (DECL_ARGUMENTS (DECL)))))
 #endif
+
+/* Decide whether a function with a target specific attribute 
+   attached can be inlined.  By default we disallow this.  */
+#ifndef FUNCTION_ATTRIBUTE_INLINABLE_P
+#define FUNCTION_ATTRIBUTE_INLINABLE_P(FNDECL) 0
+#endif
 \f
-static rtvec initialize_for_inline     PROTO((tree, int));
-static void adjust_copied_decl_tree    PROTO((tree));
-static tree copy_decl_list             PROTO((tree));
-static tree copy_decl_tree             PROTO((tree));
-static void copy_decl_rtls             PROTO((tree));
-static void save_constants             PROTO((rtx *));
-static void note_modified_parmregs     PROTO((rtx, rtx));
-static rtx copy_for_inline             PROTO((rtx));
-static void integrate_parm_decls       PROTO((tree, struct inline_remap *,
-                                              rtvec));
-static void integrate_decl_tree                PROTO((tree, int,
-                                              struct inline_remap *));
-static void save_constants_in_decl_trees PROTO ((tree));
-static void subst_constants            PROTO((rtx *, rtx,
-                                              struct inline_remap *));
-static void restore_constants          PROTO((rtx *));
-static void set_block_origin_self      PROTO((tree));
-static void set_decl_origin_self       PROTO((tree));
-static void set_block_abstract_flags   PROTO((tree, int));
-static void process_reg_param          PROTO((struct inline_remap *, rtx,
-                                              rtx));
-
-
-void set_decl_abstract_flags           PROTO((tree, int));
-static tree copy_and_set_decl_abstract_origin PROTO((tree));
+static rtvec initialize_for_inline     PARAMS ((tree));
+static void note_modified_parmregs     PARAMS ((rtx, rtx, void *));
+static void integrate_parm_decls       PARAMS ((tree, struct inline_remap *,
+                                                rtvec));
+static tree integrate_decl_tree                PARAMS ((tree,
+                                                struct inline_remap *));
+static void subst_constants            PARAMS ((rtx *, rtx,
+                                                struct inline_remap *, int));
+static void set_block_origin_self      PARAMS ((tree));
+static void set_block_abstract_flags   PARAMS ((tree, int));
+static void process_reg_param          PARAMS ((struct inline_remap *, rtx,
+                                                rtx));
+void set_decl_abstract_flags           PARAMS ((tree, int));
+static rtx expand_inline_function_eh_labelmap PARAMS ((rtx));
+static void mark_stores                 PARAMS ((rtx, rtx, void *));
+static void save_parm_insns            PARAMS ((rtx, rtx));
+static void copy_insn_list              PARAMS ((rtx, struct inline_remap *,
+                                                rtx));
+static int compare_blocks               PARAMS ((const PTR, const PTR));
+static int find_block                   PARAMS ((const PTR, const PTR));
 
 /* The maximum number of instructions accepted for inlining a
    function.  Increasing values mean more agressive inlining.
@@ -95,6 +99,11 @@ static tree copy_and_set_decl_abstract_origin PROTO((tree));
 
 int inline_max_insns = 10000;
 
+/* Used by copy_rtx_and_substitute; this indicates whether the function is
+   called for the purpose of inlining or some other purpose (i.e. loop
+   unrolling).  This affects how constant pool references are handled.
+   This variable contains the FUNCTION_DECL for the inlined function.  */
+static struct function *inlining = 0;
 \f
 /* Returns the Ith entry in the label_map contained in MAP.  If the
    Ith entry has not yet been set, return a fresh label.  This function
@@ -147,9 +156,16 @@ function_cannot_inline_p (fndecl)
   if (current_function_calls_alloca)
     return N_("function using alloca cannot be inline");
 
+  if (current_function_calls_setjmp)
+    return N_("function using setjmp cannot be inline");
+
   if (current_function_contains_functions)
     return N_("function with nested functions cannot be inline");
 
+  if (forced_labels)
+    return
+      N_("function with label addresses used in initializers cannot inline");
+
   if (current_function_cannot_inline)
     return current_function_cannot_inline;
 
@@ -175,7 +191,8 @@ function_cannot_inline_p (fndecl)
     return N_("inline functions not supported for this return value type");
 
   /* We can't inline functions that return structures of varying size.  */
-  if (int_size_in_bytes (TREE_TYPE (TREE_TYPE (fndecl))) < 0)
+  if (TREE_CODE (TREE_TYPE (TREE_TYPE (fndecl))) != VOID_TYPE
+      && int_size_in_bytes (TREE_TYPE (TREE_TYPE (fndecl))) < 0)
     return N_("function with varying-size return value cannot be inline");
 
   /* Cannot inline a function with a varying size argument or one that
@@ -184,7 +201,8 @@ function_cannot_inline_p (fndecl)
     {
       if (int_size_in_bytes (TREE_TYPE (parms)) < 0)
        return N_("function with varying-size parameter cannot be inline");
-      else if (TYPE_TRANSPARENT_UNION (TREE_TYPE (parms)))
+      else if (TREE_CODE (TREE_TYPE (parms)) == UNION_TYPE
+              && TYPE_TRANSPARENT_UNION (TREE_TYPE (parms)))
        return N_("function with transparent unit parameter cannot be inline");
     }
 
@@ -193,7 +211,7 @@ function_cannot_inline_p (fndecl)
       for (ninsns = 0, insn = get_first_nonparm_insn ();
           insn && ninsns < max_insns;
           insn = NEXT_INSN (insn))
-       if (GET_RTX_CLASS (GET_CODE (insn)) == 'i')
+       if (INSN_P (insn))
          ninsns++;
 
       if (ninsns >= max_insns)
@@ -228,66 +246,30 @@ function_cannot_inline_p (fndecl)
   if (result && GET_CODE (result) == PARALLEL)
     return N_("inline functions not supported for this return value type");
 
-  return 0;
+  /* If the function has a target specific attribute attached to it,
+     then we assume that we should not inline it.  This can be overriden
+     by the target if it defines FUNCTION_ATTRIBUTE_INLINABLE_P.  */
+  if (DECL_MACHINE_ATTRIBUTES (fndecl)
+      && ! FUNCTION_ATTRIBUTE_INLINABLE_P (fndecl))
+    return N_("function with target specific attribute(s) cannot be inlined");
+
+  return NULL;
 }
 \f
-/* Variables used within save_for_inline.  */
-
-/* Mapping from old pseudo-register to new pseudo-registers.
-   The first element of this map is reg_map[FIRST_PSEUDO_REGISTER].
-   It is allocated in `save_for_inline' and `expand_inline_function',
-   and deallocated on exit from each of those routines.  */
-static rtx *reg_map;
-
-/* Mapping from old code-labels to new code-labels.
-   The first element of this map is label_map[min_labelno].
-   It is allocated in `save_for_inline' and `expand_inline_function',
-   and deallocated on exit from each of those routines.  */
-static rtx *label_map;
-
-/* Mapping from old insn uid's to copied insns.
-   It is allocated in `save_for_inline' and `expand_inline_function',
-   and deallocated on exit from each of those routines.  */
-static rtx *insn_map;
-
 /* Map pseudo reg number into the PARM_DECL for the parm living in the reg.
    Zero for a reg that isn't a parm's home.
    Only reg numbers less than max_parm_reg are mapped here.  */
 static tree *parmdecl_map;
 
-/* When an insn is being copied by copy_for_inline,
-   this is nonzero if we have copied an ASM_OPERANDS.
-   In that case, it is the original input-operand vector.  */
-static rtvec orig_asm_operands_vector;
-
-/* When an insn is being copied by copy_for_inline,
-   this is nonzero if we have copied an ASM_OPERANDS.
-   In that case, it is the copied input-operand vector.  */
-static rtvec copy_asm_operands_vector;
-
-/* Likewise, this is the copied constraints vector.  */
-static rtvec copy_asm_constraints_vector;
-
 /* In save_for_inline, nonzero if past the parm-initialization insns.  */
 static int in_nonparm_insns;
 \f
-/* subroutines passed to duplicate_eh_handlers to map exception labels */
-
-static rtx 
-save_for_inline_eh_labelmap (label)
-     rtx label;
-{
-  int index = CODE_LABEL_NUMBER (label);
-  return label_map[index];
-}
-
-/* Subroutine for `save_for_inline{copying,nocopy}'.  Performs initialization
+/* Subroutine for `save_for_inline_nocopy'.  Performs initialization
    needed to save FNDECL's insns and info for future inline expansion.  */
-   
+
 static rtvec
-initialize_for_inline (fndecl, copy)
+initialize_for_inline (fndecl)
      tree fndecl;
-     int copy;
 {
   int i;
   rtvec arg_vector;
@@ -302,7 +284,6 @@ initialize_for_inline (fndecl, copy)
        parms = TREE_CHAIN (parms), i++)
     {
       rtx p = DECL_RTL (parms);
-      int copied_incoming = 0;
 
       /* If we have (mem (addressof (mem ...))), use the inner MEM since
         otherwise the copy_rtx call below will not unshare the MEM since
@@ -311,25 +292,6 @@ initialize_for_inline (fndecl, copy)
          && GET_CODE (XEXP (XEXP (p, 0), 0)) == MEM)
        p = XEXP (XEXP (p, 0), 0);
 
-      if (GET_CODE (p) == MEM && copy)
-       {
-         /* Copy the rtl so that modifications of the addresses
-            later in compilation won't affect this arg_vector.
-            Virtual register instantiation can screw the address
-            of the rtl.  */
-         rtx new = copy_rtx (p);
-
-         /* Don't leave the old copy anywhere in this decl.  */
-         if (DECL_RTL (parms) == DECL_INCOMING_RTL (parms)
-             || (GET_CODE (DECL_RTL (parms)) == MEM
-                 && GET_CODE (DECL_INCOMING_RTL (parms)) == MEM
-                 && (XEXP (DECL_RTL (parms), 0)
-                     == XEXP (DECL_INCOMING_RTL (parms), 0))))
-           DECL_INCOMING_RTL (parms) = new, copied_incoming = 1;
-
-         DECL_RTL (parms) = new;
-       }
-
       RTVEC_ELT (arg_vector, i) = p;
 
       if (GET_CODE (p) == REG)
@@ -348,456 +310,76 @@ initialize_for_inline (fndecl, copy)
       /* This flag is cleared later
         if the function ever modifies the value of the parm.  */
       TREE_READONLY (parms) = 1;
-
-      /* Copy DECL_INCOMING_RTL if not done already.  This can
-        happen if DECL_RTL is a reg.  */
-      if (copy && ! copied_incoming)
-       {
-         p = DECL_INCOMING_RTL (parms);
-
-         /* If we have (mem (addressof (mem ...))), use the inner MEM since
-            otherwise the copy_rtx call below will not unshare the MEM since
-            it shares ADDRESSOF.  */
-         if (GET_CODE (p) == MEM && GET_CODE (XEXP (p, 0)) == ADDRESSOF
-             && GET_CODE (XEXP (XEXP (p, 0), 0)) == MEM)
-           p = XEXP (XEXP (p, 0), 0);
-
-         if (GET_CODE (p) == MEM)
-           DECL_INCOMING_RTL (parms) = copy_rtx (p);
-       }
     }
 
   return arg_vector;
 }
 
-/* Adjust the BLOCK_END_NOTE pointers in a given copied DECL tree so that
-   they all point to the new (copied) rtxs.  */
+/* Copy NODE (which must be a DECL, but not a PARM_DECL).  The DECL
+   originally was in the FROM_FN, but now it will be in the 
+   TO_FN.  */
 
-static void
-adjust_copied_decl_tree (block)
-     register tree block;
+tree
+copy_decl_for_inlining (decl, from_fn, to_fn)
+     tree decl;
+     tree from_fn;
+     tree to_fn;
 {
-  register tree subblock;
-  register rtx original_end;
+  tree copy;
 
-  original_end = BLOCK_END_NOTE (block);
-  if (original_end)
+  /* Copy the declaration.  */
+  if (TREE_CODE (decl) == PARM_DECL || TREE_CODE (decl) == RESULT_DECL)
     {
-      BLOCK_END_NOTE (block) = (rtx) NOTE_SOURCE_FILE (original_end);
-      NOTE_SOURCE_FILE (original_end) = 0;
+      /* For a parameter, we must make an equivalent VAR_DECL, not a
+        new PARM_DECL.  */
+      copy = build_decl (VAR_DECL, DECL_NAME (decl), TREE_TYPE (decl));
+      TREE_ADDRESSABLE (copy) = TREE_ADDRESSABLE (decl);
+      TREE_READONLY (copy) = TREE_READONLY (decl);
+      TREE_THIS_VOLATILE (copy) = TREE_THIS_VOLATILE (decl);
     }
-
-  /* Process all subblocks.  */
-  for (subblock = BLOCK_SUBBLOCKS (block);
-       subblock;
-       subblock = TREE_CHAIN (subblock))
-    adjust_copied_decl_tree (subblock);
-}
-
-/* Make the insns and PARM_DECLs of the current function permanent
-   and record other information in DECL_SAVED_INSNS to allow inlining
-   of this function in subsequent calls.
-
-   This function is called when we are going to immediately compile
-   the insns for FNDECL.  The insns in maybepermanent_obstack cannot be
-   modified by the compilation process, so we copy all of them to
-   new storage and consider the new insns to be the insn chain to be
-   compiled.  Our caller (rest_of_compilation) saves the original
-   DECL_INITIAL and DECL_ARGUMENTS; here we copy them.  */
-
-/* ??? The nonlocal_label list should be adjusted also.  However, since
-   a function that contains a nested function never gets inlined currently,
-   the nonlocal_label list will always be empty, so we don't worry about
-   it for now.  */
-
-void
-save_for_inline_copying (fndecl)
-     tree fndecl;
-{
-  rtvec argvec;
-  rtx new_first_insn, new_last_insn, insn;
-  int max_labelno, min_labelno, i, len;
-  int max_reg;
-  int max_uid;
-  rtx first_nonparm_insn;
-  char *new, *new1;
-  rtx *new_parm_reg_stack_loc;
-  rtx *new2;
-  struct emit_status *es
-    = (struct emit_status *) xmalloc (sizeof (struct emit_status));
-
-  /* Make and emit a return-label if we have not already done so. 
-     Do this before recording the bounds on label numbers.  */
-
-  if (return_label == 0)
-    {
-      return_label = gen_label_rtx ();
-      emit_label (return_label);
-    }
-
-  *es = *current_function->emit;
-
-  /* Get some bounds on the labels and registers used.  */
-
-  max_labelno = max_label_num ();
-  min_labelno = get_first_label_num ();
-  max_reg = max_reg_num ();
-
-  /* Set up PARMDECL_MAP which maps pseudo-reg number to its PARM_DECL.
-     Later we set TREE_READONLY to 0 if the parm is modified inside the fn.
-     Also set up ARG_VECTOR, which holds the unmodified DECL_RTX values
-     for the parms, prior to elimination of virtual registers.
-     These values are needed for substituting parms properly.  */
-
-  parmdecl_map = (tree *) alloca (max_parm_reg * sizeof (tree));
-
-  argvec = initialize_for_inline (fndecl, 1);
-
-  if (current_function_uses_const_pool)
+  else
     {
-      /* Replace any constant pool references with the actual constant.  We
-        will put the constants back in the copy made below.  */
-      for (insn = get_insns (); insn; insn = NEXT_INSN (insn))
-       if (GET_RTX_CLASS (GET_CODE (insn)) == 'i')
-         {
-           save_constants (&PATTERN (insn));
-           if (REG_NOTES (insn))
-             save_constants (&REG_NOTES (insn));
-         }
-
-      /* Also scan all decls, and replace any constant pool references with the
-        actual constant.  */
-      save_constants_in_decl_trees (DECL_INITIAL (fndecl));
-
-      /* Clear out the constant pool so that we can recreate it with the
-        copied constants below.  */
-      init_const_rtx_hash_table ();
-      clear_const_double_mem ();
+      copy = copy_node (decl);
+      if (DECL_LANG_SPECIFIC (copy))
+       copy_lang_decl (copy);
+
+      /* TREE_ADDRESSABLE isn't used to indicate that a label's
+        address has been taken; it's for internal bookkeeping in
+        expand_goto_internal.  */
+      if (TREE_CODE (copy) == LABEL_DECL)
+       TREE_ADDRESSABLE (copy) = 0;
     }
 
-  max_uid = get_max_uid ();
-
-  /* We have now allocated all that needs to be allocated permanently
-     on the rtx obstack.  Set our high-water mark, so that we
-     can free the rest of this when the time comes.  */
-
-  preserve_data ();
-
-  /* Copy the chain insns of this function.
-     Install the copied chain as the insns of this function,
-     for continued compilation;
-     the original chain is recorded as the DECL_SAVED_INSNS
-     for inlining future calls.  */
-
-  /* If there are insns that copy parms from the stack into pseudo registers,
-     those insns are not copied.  `expand_inline_function' must
-     emit the correct code to handle such things.  */
-
-  insn = get_insns ();
-  if (GET_CODE (insn) != NOTE)
-    abort ();
-  new_first_insn = rtx_alloc (NOTE);
-  NOTE_SOURCE_FILE (new_first_insn) = NOTE_SOURCE_FILE (insn);
-  NOTE_LINE_NUMBER (new_first_insn) = NOTE_LINE_NUMBER (insn);
-  INSN_UID (new_first_insn) = INSN_UID (insn);
-  PREV_INSN (new_first_insn) = NULL;
-  NEXT_INSN (new_first_insn) = NULL;
-  new_last_insn = new_first_insn;
-
-  /* Each pseudo-reg in the old insn chain must have a unique rtx in the copy.
-     Make these new rtx's now, and install them in regno_reg_rtx, so they
-     will be the official pseudo-reg rtx's for the rest of compilation.  */
-
-  reg_map = (rtx *) savealloc (es->regno_pointer_flag_length * sizeof (rtx));
+  /* Set the DECL_ABSTRACT_ORIGIN so the debugging routines know what
+     declaration inspired this copy.  */
+  DECL_ABSTRACT_ORIGIN (copy) = DECL_ORIGIN (decl);
 
-  len = sizeof (struct rtx_def) + (GET_RTX_LENGTH (REG) - 1) * sizeof (rtunion);
-  for (i = max_reg - 1; i > LAST_VIRTUAL_REGISTER; i--)
-    reg_map[i] = (rtx)obstack_copy (function_maybepermanent_obstack,
-                                   regno_reg_rtx[i], len);
+  /* The new variable/label has no RTL, yet.  */
+  DECL_RTL (copy) = NULL_RTX;
 
-  es->x_regno_reg_rtx = reg_map;
+  /* These args would always appear unused, if not for this.  */
+  TREE_USED (copy) = 1;
 
-  /* Put copies of all the virtual register rtx into the new regno_reg_rtx.  */
-  init_virtual_regs (es);
-
-  /* Likewise each label rtx must have a unique rtx as its copy.  */
-
-  /* We used to use alloca here, but the size of what it would try to
-     allocate would occasionally cause it to exceed the stack limit and
-     cause unpredictable core dumps.  Some examples were > 2Mb in size.  */
-  label_map = (rtx *) xmalloc ((max_labelno) * sizeof (rtx));
-
-  for (i = min_labelno; i < max_labelno; i++)
-    label_map[i] = gen_label_rtx ();
-
-  /* Likewise for parm_reg_stack_slot.  */
-  new_parm_reg_stack_loc = (rtx *) savealloc (max_parm_reg * sizeof (rtx));
-  for (i = 0; i < max_parm_reg; i++)
-    new_parm_reg_stack_loc[i] = copy_for_inline (parm_reg_stack_loc[i]);
-
-  parm_reg_stack_loc = new_parm_reg_stack_loc;
-
-  /* Record the mapping of old insns to copied insns.  */
-
-  insn_map = (rtx *) alloca (max_uid * sizeof (rtx));
-  bzero ((char *) insn_map, max_uid * sizeof (rtx));
-
-  /* Get the insn which signals the end of parameter setup code.  */
-  first_nonparm_insn = get_first_nonparm_insn ();
-
-  /* Copy any entries in regno_reg_rtx or DECL_RTLs that reference MEM
-     (the former occurs when a variable has its address taken)
-     since these may be shared and can be changed by virtual
-     register instantiation.  DECL_RTL values for our arguments
-     have already been copied by initialize_for_inline.  */
-  for (i = LAST_VIRTUAL_REGISTER + 1; i < max_reg; i++)
-    if (GET_CODE (regno_reg_rtx[i]) == MEM)
-      XEXP (regno_reg_rtx[i], 0)
-       = copy_for_inline (XEXP (regno_reg_rtx[i], 0));
-
-  /* Copy the parm_reg_stack_loc array, and substitute for all of the rtx
-     contained in it.  */
-  new2 = (rtx *) savealloc (max_parm_reg * sizeof (rtx));
-  bcopy ((char *) parm_reg_stack_loc, (char *) new2,
-        max_parm_reg * sizeof (rtx));
-  parm_reg_stack_loc = new2;
-  for (i = LAST_VIRTUAL_REGISTER + 1; i < max_parm_reg; ++i)
-    if (parm_reg_stack_loc[i])
-      parm_reg_stack_loc[i] = copy_for_inline (parm_reg_stack_loc[i]);
-
-  /* Copy the tree of subblocks of the function, and the decls in them.
-     We will use the copy for compiling this function, then restore the original
-     subblocks and decls for use when inlining this function.
-
-     Several parts of the compiler modify BLOCK trees.  In particular,
-     instantiate_virtual_regs will instantiate any virtual regs
-     mentioned in the DECL_RTLs of the decls, and loop
-     unrolling will replicate any BLOCK trees inside an unrolled loop.
-
-     The modified subblocks or DECL_RTLs would be incorrect for the original rtl
-     which we will use for inlining.  The rtl might even contain pseudoregs
-     whose space has been freed.  */
-
-  DECL_INITIAL (fndecl) = copy_decl_tree (DECL_INITIAL (fndecl));
-  DECL_ARGUMENTS (fndecl) = copy_decl_list (DECL_ARGUMENTS (fndecl));
-
-  /* Now copy each DECL_RTL which is a MEM,
-     so it is safe to modify their addresses.  */
-  copy_decl_rtls (DECL_INITIAL (fndecl));
-
-  /* The fndecl node acts as its own progenitor, so mark it as such.  */
-  DECL_ABSTRACT_ORIGIN (fndecl) = fndecl;
-
-  /* Now copy the chain of insns.  Do this twice.  The first copy the insn
-     itself and its body.  The second time copy of REG_NOTES.  This is because
-     a REG_NOTE may have a forward pointer to another insn.  */
-
-  for (insn = NEXT_INSN (insn); insn; insn = NEXT_INSN (insn))
-    {
-      rtx copy;
-      orig_asm_operands_vector = 0;
-
-      if (insn == first_nonparm_insn)
-       in_nonparm_insns = 1;
-
-      switch (GET_CODE (insn))
-       {
-       case NOTE:
-         /* No need to keep these.  */
-         if (NOTE_LINE_NUMBER (insn) == NOTE_INSN_DELETED)
-           continue;
-
-         copy = rtx_alloc (NOTE);
-         NOTE_LINE_NUMBER (copy) = NOTE_LINE_NUMBER (insn);
-         if (NOTE_LINE_NUMBER (insn) != NOTE_INSN_BLOCK_END)
-           NOTE_SOURCE_FILE (copy) = NOTE_SOURCE_FILE (insn);
-         else
-           {
-             NOTE_SOURCE_FILE (insn) = (char *) copy;
-             NOTE_SOURCE_FILE (copy) = 0;
-           }
-         if (NOTE_LINE_NUMBER (copy) == NOTE_INSN_EH_REGION_BEG
-             || NOTE_LINE_NUMBER (copy) == NOTE_INSN_EH_REGION_END)
-           {
-              int new_region = CODE_LABEL_NUMBER 
-                                        (label_map[NOTE_BLOCK_NUMBER (copy)]);
-
-              /* we have to duplicate the handlers for the original */
-              if (NOTE_LINE_NUMBER (copy) == NOTE_INSN_EH_REGION_BEG) 
-                duplicate_eh_handlers (NOTE_BLOCK_NUMBER (copy), new_region,
-                                       save_for_inline_eh_labelmap);
-                
-             /* We have to forward these both to match the new exception
-                region.  */
-             NOTE_BLOCK_NUMBER (copy) = new_region;
-             
-           }
-         RTX_INTEGRATED_P (copy) = RTX_INTEGRATED_P (insn);
-         break;
-
-       case INSN:
-       case JUMP_INSN:
-       case CALL_INSN:
-         copy = rtx_alloc (GET_CODE (insn));
-
-         if (GET_CODE (insn) == CALL_INSN)
-           CALL_INSN_FUNCTION_USAGE (copy)
-             = copy_for_inline (CALL_INSN_FUNCTION_USAGE (insn));
-
-         PATTERN (copy) = copy_for_inline (PATTERN (insn));
-         INSN_CODE (copy) = -1;
-         LOG_LINKS (copy) = NULL_RTX;
-         RTX_INTEGRATED_P (copy) = RTX_INTEGRATED_P (insn);
-         break;
-
-       case CODE_LABEL:
-         copy = label_map[CODE_LABEL_NUMBER (insn)];
-         LABEL_NAME (copy) = LABEL_NAME (insn);
-         break;
-
-       case BARRIER:
-         copy = rtx_alloc (BARRIER);
-         break;
-
-       default:
-         abort ();
-       }
-      INSN_UID (copy) = INSN_UID (insn);
-      insn_map[INSN_UID (insn)] = copy;
-      NEXT_INSN (new_last_insn) = copy;
-      PREV_INSN (copy) = new_last_insn;
-      new_last_insn = copy;
-    }
-
-  adjust_copied_decl_tree (DECL_INITIAL (fndecl));
-
-  /* Now copy the REG_NOTES.  */
-  for (insn = NEXT_INSN (get_insns ()); insn; insn = NEXT_INSN (insn))
-    if (GET_RTX_CLASS (GET_CODE (insn)) == 'i'
-       && insn_map[INSN_UID(insn)])
-      REG_NOTES (insn_map[INSN_UID (insn)])
-       = copy_for_inline (REG_NOTES (insn));
-
-  NEXT_INSN (new_last_insn) = NULL;
-
-  /* Make new versions of the register tables.  */
-  new = (char *) savealloc (es->regno_pointer_flag_length);
-  memcpy (new, es->regno_pointer_flag, es->regno_pointer_flag_length);
-  new1 = (char *) savealloc (es->regno_pointer_flag_length);
-  memcpy (new1, es->regno_pointer_align, es->regno_pointer_flag_length);
-  es->regno_pointer_flag = new;
-  es->regno_pointer_align = new1;
-
-  free (label_map);
-
-  current_function->inl_max_label_num = max_label_num ();
-  current_function->inl_last_parm_insn = current_function->x_last_parm_insn;
-  current_function->original_arg_vector = argvec;
-  current_function->original_decl_initial = DECL_INITIAL (fndecl);
-  /* Use the copy we made for compiling the function now, and
-     use the original values for inlining.  */
-  current_function->inl_emit = current_function->emit;
-  current_function->emit = es;
-  set_new_first_and_last_insn (new_first_insn, new_last_insn);
-  DECL_SAVED_INSNS (fndecl) = current_function;
-}
-
-/* Copy NODE (as with copy_node).  NODE must be a DECL.  Set the
-   DECL_ABSTRACT_ORIGIN for the new accordinly.  */
-
-static tree
-copy_and_set_decl_abstract_origin (node)
-     tree node;
-{
-  tree copy = copy_node (node);
-  if (DECL_ABSTRACT_ORIGIN (copy) != NULL_TREE)
-    /* That means that NODE already had a DECL_ABSTRACT_ORIGIN.  (This
-       situation occurs if we inline a function which itself made
-       calls to inline functions.)  Since DECL_ABSTRACT_ORIGIN is the
-       most distant ancestor, we don't have to do anything here.  */
+  /* Set the context for the new declaration.  */
+  if (!DECL_CONTEXT (decl))
+    /* Globals stay global.  */
+       ;
+  else if (DECL_CONTEXT (decl) != from_fn)
+    /* Things that weren't in the scope of the function we're inlining
+       from aren't in the scope we're inlining too, either.  */
+    ;
+  else if (TREE_STATIC (decl))
+    /* Function-scoped static variables should say in the original
+       function.  */
     ;
   else
-    /* The most distant ancestor must be NODE.  */
-    DECL_ABSTRACT_ORIGIN (copy) = node;
+    /* Ordinary automatic local variables are now in the scope of the
+       new function.  */
+    DECL_CONTEXT (copy) = to_fn;
 
   return copy;
 }
 
-/* Return a copy of a chain of nodes, chained through the TREE_CHAIN field.
-   For example, this can copy a list made of TREE_LIST nodes.  While copying,
-   set DECL_ABSTRACT_ORIGIN appropriately.  */
-
-static tree
-copy_decl_list (list)
-     tree list;
-{
-  tree head;
-  register tree prev, next;
-
-  if (list == 0)
-    return 0;
-
-  head = prev = copy_and_set_decl_abstract_origin (list);
-  next = TREE_CHAIN (list);
-  while (next)
-    {
-      register tree copy;
-
-      copy = copy_and_set_decl_abstract_origin (next);
-      TREE_CHAIN (prev) = copy;
-      prev = copy;
-      next = TREE_CHAIN (next);
-    }
-  return head;
-}
-
-/* Make a copy of the entire tree of blocks BLOCK, and return it.  */
-
-static tree
-copy_decl_tree (block)
-     tree block;
-{
-  tree t, vars, subblocks;
-
-  vars = copy_decl_list (BLOCK_VARS (block));
-  subblocks = 0;
-
-  /* Process all subblocks.  */
-  for (t = BLOCK_SUBBLOCKS (block); t; t = TREE_CHAIN (t))
-    {
-      tree copy = copy_decl_tree (t);
-      TREE_CHAIN (copy) = subblocks;
-      subblocks = copy;
-    }
-
-  t = copy_node (block);
-  BLOCK_VARS (t) = vars;
-  BLOCK_SUBBLOCKS (t) = nreverse (subblocks);
-  /* If the BLOCK being cloned is already marked as having been instantiated
-     from something else, then leave that `origin' marking alone.  Otherwise,
-     mark the clone as having originated from the BLOCK we are cloning.  */
-  if (BLOCK_ABSTRACT_ORIGIN (t) == NULL_TREE)
-    BLOCK_ABSTRACT_ORIGIN (t) = block;
-  return t;
-}
-
-/* Copy DECL_RTLs in all decls in the given BLOCK node.  */
-
-static void
-copy_decl_rtls (block)
-     tree block;
-{
-  tree t;
-
-  for (t = BLOCK_VARS (block); t; t = TREE_CHAIN (t))
-    if (DECL_RTL (t) && GET_CODE (DECL_RTL (t)) == MEM)
-      DECL_RTL (t) = copy_for_inline (DECL_RTL (t));
-
-  /* Process all subblocks.  */
-  for (t = BLOCK_SUBBLOCKS (block); t; t = TREE_CHAIN (t))
-    copy_decl_rtls (t);
-}
-
 /* Make the insns and PARM_DECLs of the current function permanent
    and record other information in DECL_SAVED_INSNS to allow inlining
    of this function in subsequent calls.
@@ -831,7 +413,7 @@ save_for_inline_nocopy (fndecl)
      for the parms, prior to elimination of virtual registers.
      These values are needed for substituting parms properly.  */
 
-  parmdecl_map = (tree *) alloca (max_parm_reg * sizeof (tree));
+  parmdecl_map = (tree *) xmalloc (max_parm_reg * sizeof (tree));
 
   /* Make and emit a return-label if we have not already done so.  */
 
@@ -841,7 +423,7 @@ save_for_inline_nocopy (fndecl)
       emit_label (return_label);
     }
 
-  argvec = initialize_for_inline (fndecl, 0);
+  argvec = initialize_for_inline (fndecl);
 
   /* If there are insns that copy parms from the stack into pseudo registers,
      those insns are not copied.  `expand_inline_function' must
@@ -860,32 +442,8 @@ save_for_inline_nocopy (fndecl)
      perform constant folding when its incoming value is constant).
      Otherwise, we have to copy its value into a new register and track
      the new register's life.  */
-
-  for (insn = NEXT_INSN (insn); insn; insn = NEXT_INSN (insn))
-    {
-      if (insn == first_nonparm_insn)
-       in_nonparm_insns = 1;
-
-      if (GET_RTX_CLASS (GET_CODE (insn)) == 'i')
-       {
-         if (current_function_uses_const_pool)
-           {
-             /* Replace any constant pool references with the actual constant.
-                We will put the constant back if we need to write the
-                function out after all.  */
-             save_constants (&PATTERN (insn));
-             if (REG_NOTES (insn))
-               save_constants (&REG_NOTES (insn));
-           }
-
-         /* Record what interesting things happen to our parameters.  */
-         note_stores (PATTERN (insn), note_modified_parmregs);
-       }
-    }
-
-  /* Also scan all decls, and replace any constant pool references with the
-     actual constant.  */
-  save_constants_in_decl_trees (DECL_INITIAL (fndecl));
+  in_nonparm_insns = 0;
+  save_parm_insns (insn, first_nonparm_insn);
 
   /* We have now allocated all that needs to be allocated permanently
      on the rtx obstack.  Set our high-water mark, so that we
@@ -893,90 +451,51 @@ save_for_inline_nocopy (fndecl)
 
   preserve_data ();
 
-  current_function->inl_emit = current_function->emit;
-  current_function->inl_max_label_num = max_label_num ();
-  current_function->inl_last_parm_insn = current_function->x_last_parm_insn;
-  current_function->original_arg_vector = argvec;
-  current_function->original_decl_initial = DECL_INITIAL (fndecl);
-  DECL_SAVED_INSNS (fndecl) = current_function;
+  cfun->inl_max_label_num = max_label_num ();
+  cfun->inl_last_parm_insn = cfun->x_last_parm_insn;
+  cfun->original_arg_vector = argvec;
+  cfun->original_decl_initial = DECL_INITIAL (fndecl);
+  DECL_SAVED_INSNS (fndecl) = cfun;
+
+  /* Clean up.  */
+  free (parmdecl_map);
 }
-\f
-/* Given PX, a pointer into an insn, search for references to the constant
-   pool.  Replace each with a CONST that has the mode of the original
-   constant, contains the constant, and has RTX_INTEGRATED_P set.
-   Similarly, constant pool addresses not enclosed in a MEM are replaced
-   with an ADDRESS and CONST rtx which also gives the constant, its
-   mode, the mode of the address, and has RTX_INTEGRATED_P set.  */
+
+/* Scan the chain of insns to see what happens to our PARM_DECLs.  If a
+   PARM_DECL is used but never modified, we can substitute its rtl directly
+   when expanding inline (and perform constant folding when its incoming
+   value is constant). Otherwise, we have to copy its value into a new
+   register and track the new register's life.  */
 
 static void
-save_constants (px)
-     rtx *px;
+save_parm_insns (insn, first_nonparm_insn)
+    rtx insn;
+    rtx first_nonparm_insn;
 {
-  rtx x;
-  int i, j;
-
- again:
-  x = *px;
-
-  /* If this is a CONST_DOUBLE, don't try to fix things up in 
-     CONST_DOUBLE_MEM, because this is an infinite recursion.  */
-  if (GET_CODE (x) == CONST_DOUBLE)
+  if (insn == NULL_RTX)
     return;
-  else if (GET_CODE (x) == MEM && GET_CODE (XEXP (x, 0)) == SYMBOL_REF
-          && CONSTANT_POOL_ADDRESS_P (XEXP (x,0)))
-    {
-      enum machine_mode const_mode = get_pool_mode (XEXP (x, 0));
-      rtx new = gen_rtx_CONST (const_mode, get_pool_constant (XEXP (x, 0)));
-      RTX_INTEGRATED_P (new) = 1;
-
-      /* If the MEM was in a different mode than the constant (perhaps we
-        were only looking at the low-order part), surround it with a 
-        SUBREG so we can save both modes.  */
-
-      if (GET_MODE (x) != const_mode)
-       {
-         new = gen_rtx_SUBREG (GET_MODE (x), new, 0);
-         RTX_INTEGRATED_P (new) = 1;
-       }
 
-      *px = new;
-      save_constants (&XEXP (*px, 0));
-    }
-  else if (GET_CODE (x) == SYMBOL_REF
-          && CONSTANT_POOL_ADDRESS_P (x))
-    {
-      *px = gen_rtx_ADDRESS (GET_MODE (x),
-                            gen_rtx_CONST (get_pool_mode (x),
-                                           get_pool_constant (x)));
-      save_constants (&XEXP (*px, 0));
-      RTX_INTEGRATED_P (*px) = 1;
-    }
-
-  else
+  for (insn = NEXT_INSN (insn); insn; insn = NEXT_INSN (insn))
     {
-      char *fmt = GET_RTX_FORMAT (GET_CODE (x));
-      int len = GET_RTX_LENGTH (GET_CODE (x));
+      if (insn == first_nonparm_insn)
+       in_nonparm_insns = 1;
 
-      for (i = len-1; i >= 0; i--)
+      if (INSN_P (insn))
        {
-         switch (fmt[i])
+         /* Record what interesting things happen to our parameters.  */
+         note_stores (PATTERN (insn), note_modified_parmregs, NULL);
+
+         /* If this is a CALL_PLACEHOLDER insn then we need to look into the
+            three attached sequences: normal call, sibling call and tail
+            recursion. */
+         if (GET_CODE (insn) == CALL_INSN
+             && GET_CODE (PATTERN (insn)) == CALL_PLACEHOLDER)
            {
-           case 'E':
-             for (j = 0; j < XVECLEN (x, i); j++)
-               save_constants (&XVECEXP (x, i, j));
-             break;
+             int i;
 
-           case 'e':
-             if (XEXP (x, i) == 0)
-               continue;
-             if (i == 0)
-               {
-                 /* Hack tail-recursion here.  */
-                 px = &XEXP (x, 0);
-                 goto again;
-               }
-             save_constants (&XEXP (x, i));
-             break;
+             for (i = 0; i < 3; i++)
+               save_parm_insns (XEXP (PATTERN (insn), i),
+                                first_nonparm_insn);
            }
        }
     }
@@ -985,9 +504,10 @@ save_constants (px)
 /* Note whether a parameter is modified or not.  */
 
 static void
-note_modified_parmregs (reg, x)
+note_modified_parmregs (reg, x, data)
      rtx reg;
      rtx x ATTRIBUTE_UNUSED;
+     void *data ATTRIBUTE_UNUSED;
 {
   if (GET_CODE (reg) == REG && in_nonparm_insns
       && REGNO (reg) < max_parm_reg
@@ -996,262 +516,6 @@ note_modified_parmregs (reg, x)
     TREE_READONLY (parmdecl_map[REGNO (reg)]) = 0;
 }
 
-/* Copy the rtx ORIG recursively, replacing pseudo-regs and labels
-   according to `reg_map' and `label_map'.  The original rtl insns
-   will be saved for inlining; this is used to make a copy
-   which is used to finish compiling the inline function itself.
-
-   If we find a "saved" constant pool entry, one which was replaced with
-   the value of the constant, convert it back to a constant pool entry.
-   Since the pool wasn't touched, this should simply restore the old
-   address.
-
-   All other kinds of rtx are copied except those that can never be
-   changed during compilation.  */
-
-static rtx
-copy_for_inline (orig)
-     rtx orig;
-{
-  register rtx x = orig;
-  register rtx new;
-  register int i;
-  register enum rtx_code code;
-  register char *format_ptr;
-
-  if (x == 0)
-    return x;
-
-  code = GET_CODE (x);
-
-  /* These types may be freely shared.  */
-
-  switch (code)
-    {
-    case QUEUED:
-    case CONST_INT:
-    case PC:
-    case CC0:
-      return x;
-
-    case SYMBOL_REF:
-      if (! SYMBOL_REF_NEED_ADJUST (x))
-        return x;
-      return rethrow_symbol_map (x, save_for_inline_eh_labelmap);
-
-    case CONST_DOUBLE:
-      /* We have to make a new CONST_DOUBLE to ensure that we account for
-        it correctly.  Using the old CONST_DOUBLE_MEM data is wrong.  */
-      if (GET_MODE_CLASS (GET_MODE (x)) == MODE_FLOAT)
-       {
-         REAL_VALUE_TYPE d;
-
-         REAL_VALUE_FROM_CONST_DOUBLE (d, x);
-         return CONST_DOUBLE_FROM_REAL_VALUE (d, GET_MODE (x));
-       }
-      else
-       return immed_double_const (CONST_DOUBLE_LOW (x), CONST_DOUBLE_HIGH (x),
-                                  VOIDmode);
-
-    case CONST:
-      /* Get constant pool entry for constant in the pool.  */
-      if (RTX_INTEGRATED_P (x))
-       return validize_mem (force_const_mem (GET_MODE (x),
-                                             copy_for_inline (XEXP (x, 0))));
-      break;
-
-    case SUBREG:
-      /* Get constant pool entry, but access in different mode.  */
-      if (RTX_INTEGRATED_P (x))
-       {
-         new = force_const_mem (GET_MODE (SUBREG_REG (x)),
-                                copy_for_inline (XEXP (SUBREG_REG (x), 0)));
-
-         PUT_MODE (new, GET_MODE (x));
-         return validize_mem (new);
-       }
-      break;
-
-    case ADDRESS:
-      /* If not special for constant pool error.  Else get constant pool
-        address.  */
-      if (! RTX_INTEGRATED_P (x))
-       abort ();
-
-      new = force_const_mem (GET_MODE (XEXP (x, 0)),
-                            copy_for_inline (XEXP (XEXP (x, 0), 0)));
-      new = XEXP (new, 0);
-
-#ifdef POINTERS_EXTEND_UNSIGNED
-      if (GET_MODE (new) != GET_MODE (x))
-       new = convert_memory_address (GET_MODE (x), new);
-#endif
-
-      return new;
-
-    case ASM_OPERANDS:
-      /* If a single asm insn contains multiple output operands
-        then it contains multiple ASM_OPERANDS rtx's that share operand 3.
-        We must make sure that the copied insn continues to share it.  */
-      if (orig_asm_operands_vector == XVEC (orig, 3))
-       {
-         x = rtx_alloc (ASM_OPERANDS);
-         x->volatil = orig->volatil;
-         XSTR (x, 0) = XSTR (orig, 0);
-         XSTR (x, 1) = XSTR (orig, 1);
-         XINT (x, 2) = XINT (orig, 2);
-         XVEC (x, 3) = copy_asm_operands_vector;
-         XVEC (x, 4) = copy_asm_constraints_vector;
-         XSTR (x, 5) = XSTR (orig, 5);
-         XINT (x, 6) = XINT (orig, 6);
-         return x;
-       }
-      break;
-
-    case MEM:
-      /* A MEM is usually allowed to be shared if its address is constant
-        or is a constant plus one of the special registers.
-
-        We do not allow sharing of addresses that are either a special
-        register or the sum of a constant and a special register because
-        it is possible for unshare_all_rtl to copy the address, into memory
-        that won't be saved.  Although the MEM can safely be shared, and
-        won't be copied there, the address itself cannot be shared, and may
-        need to be copied. 
-
-        There are also two exceptions with constants: The first is if the
-        constant is a LABEL_REF or the sum of the LABEL_REF
-        and an integer.  This case can happen if we have an inline
-        function that supplies a constant operand to the call of another
-        inline function that uses it in a switch statement.  In this case,
-        we will be replacing the LABEL_REF, so we have to replace this MEM
-        as well.
-
-        The second case is if we have a (const (plus (address ..) ...)).
-        In that case we need to put back the address of the constant pool
-        entry.  */
-
-      if (CONSTANT_ADDRESS_P (XEXP (x, 0))
-         && GET_CODE (XEXP (x, 0)) != LABEL_REF
-         && ! (GET_CODE (XEXP (x, 0)) == CONST
-               && (GET_CODE (XEXP (XEXP (x, 0), 0)) == PLUS
-                   && ((GET_CODE (XEXP (XEXP (XEXP (x, 0), 0), 0))
-                       == LABEL_REF)
-                       || (GET_CODE (XEXP (XEXP (XEXP (x, 0), 0), 0))
-                           == ADDRESS)))))
-       return x;
-      break;
-
-    case LABEL_REF:
-      /* If this is a non-local label, just make a new LABEL_REF.
-        Otherwise, use the new label as well.  */
-      x = gen_rtx_LABEL_REF (GET_MODE (orig),
-                            LABEL_REF_NONLOCAL_P (orig) ? XEXP (orig, 0)
-                            : label_map[CODE_LABEL_NUMBER (XEXP (orig, 0))]);
-      LABEL_REF_NONLOCAL_P (x) = LABEL_REF_NONLOCAL_P (orig);
-      LABEL_OUTSIDE_LOOP_P (x) = LABEL_OUTSIDE_LOOP_P (orig);
-      return x;
-
-    case REG:
-      if (REGNO (x) > LAST_VIRTUAL_REGISTER)
-       return reg_map [REGNO (x)];
-      else
-       return x;
-
-    case SET:
-      /* If a parm that gets modified lives in a pseudo-reg,
-        clear its TREE_READONLY to prevent certain optimizations.  */
-      {
-       rtx dest = SET_DEST (x);
-
-       while (GET_CODE (dest) == STRICT_LOW_PART
-              || GET_CODE (dest) == ZERO_EXTRACT
-              || GET_CODE (dest) == SUBREG)
-         dest = XEXP (dest, 0);
-
-       if (GET_CODE (dest) == REG
-           && REGNO (dest) < max_parm_reg
-           && REGNO (dest) >= FIRST_PSEUDO_REGISTER
-           && parmdecl_map[REGNO (dest)] != 0
-           /* The insn to load an arg pseudo from a stack slot
-              does not count as modifying it.  */
-           && in_nonparm_insns)
-         TREE_READONLY (parmdecl_map[REGNO (dest)]) = 0;
-      }
-      break;
-
-#if 0 /* This is a good idea, but here is the wrong place for it.  */
-      /* Arrange that CONST_INTs always appear as the second operand
-        if they appear, and that `frame_pointer_rtx' or `arg_pointer_rtx'
-        always appear as the first.  */
-    case PLUS:
-      if (GET_CODE (XEXP (x, 0)) == CONST_INT
-         || (XEXP (x, 1) == frame_pointer_rtx
-             || (ARG_POINTER_REGNUM != FRAME_POINTER_REGNUM
-                 && XEXP (x, 1) == arg_pointer_rtx)))
-       {
-         rtx t = XEXP (x, 0);
-         XEXP (x, 0) = XEXP (x, 1);
-         XEXP (x, 1) = t;
-       }
-      break;
-#endif
-    default:
-      break;
-    }
-
-  /* Replace this rtx with a copy of itself.  */
-
-  x = rtx_alloc (code);
-  bcopy ((char *) orig, (char *) x,
-        (sizeof (*x) - sizeof (x->fld)
-         + sizeof (x->fld[0]) * GET_RTX_LENGTH (code)));
-
-  /* Now scan the subexpressions recursively.
-     We can store any replaced subexpressions directly into X
-     since we know X is not shared!  Any vectors in X
-     must be copied if X was copied.  */
-
-  format_ptr = GET_RTX_FORMAT (code);
-
-  for (i = 0; i < GET_RTX_LENGTH (code); i++)
-    {
-      switch (*format_ptr++)
-       {
-       case 'e':
-         XEXP (x, i) = copy_for_inline (XEXP (x, i));
-         break;
-
-       case 'u':
-         /* Change any references to old-insns to point to the
-            corresponding copied insns.  */
-         XEXP (x, i) = insn_map[INSN_UID (XEXP (x, i))];
-         break;
-
-       case 'E':
-         if (XVEC (x, i) != NULL && XVECLEN (x, i) != 0)
-           {
-             register int j;
-
-             XVEC (x, i) = gen_rtvec_vv (XVECLEN (x, i), XVEC (x, i)->elem);
-             for (j = 0; j < XVECLEN (x, i); j++)
-               XVECEXP (x, i, j)
-                 = copy_for_inline (XVECEXP (x, i, j));
-           }
-         break;
-       }
-    }
-
-  if (code == ASM_OPERANDS && orig_asm_operands_vector == 0)
-    {
-      orig_asm_operands_vector = XVEC (orig, 3);
-      copy_asm_operands_vector = XVEC (x, 3);
-      copy_asm_constraints_vector = XVEC (x, 4);
-    }
-
-  return x;
-}
-
 /* Unfortunately, we need a global copy of const_equiv map for communication
    with a function called from note_stores.  Be *very* careful that this
    is used properly in the presence of recursion.  */
@@ -1304,6 +568,35 @@ expand_inline_function_eh_labelmap (label)
   return get_label_from_map (eif_eh_map, index);
 }
 
+/* Compare two BLOCKs for qsort.  The key we sort on is the
+   BLOCK_ABSTRACT_ORIGIN of the blocks.  */
+
+static int
+compare_blocks (v1, v2)
+     const PTR v1;
+     const PTR v2;
+{
+  tree b1 = *((const tree *) v1);
+  tree b2 = *((const tree *) v2);
+
+  return ((char *) BLOCK_ABSTRACT_ORIGIN (b1) 
+         - (char *) BLOCK_ABSTRACT_ORIGIN (b2));
+}
+
+/* Compare two BLOCKs for bsearch.  The first pointer corresponds to
+   an original block; the second to a remapped equivalent.  */
+
+static int
+find_block (v1, v2)
+     const PTR v1;
+     const PTR v2;
+{
+  const union tree_node *b1 = (const union tree_node *) v1;
+  tree b2 = *((const tree *) v2);
+
+  return ((const char *) b1 - (char *) BLOCK_ABSTRACT_ORIGIN (b2));
+}
+
 /* Integrate the procedure defined by FNDECL.  Note that this function
    may wind up calling itself.  Since the static variables are not
    reentrant, we do not assign them until after the possibility
@@ -1326,21 +619,20 @@ expand_inline_function (fndecl, parms, target, ignore, type,
      tree type;
      rtx structure_value_addr;
 {
+  struct function *inlining_previous;
   struct function *inl_f = DECL_SAVED_INSNS (fndecl);
   tree formal, actual, block;
-  rtx parm_insns = inl_f->inl_emit->x_first_insn;
+  rtx parm_insns = inl_f->emit->x_first_insn;
   rtx insns = (inl_f->inl_last_parm_insn
               ? NEXT_INSN (inl_f->inl_last_parm_insn)
               : parm_insns);
   tree *arg_trees;
   rtx *arg_vals;
-  rtx insn;
   int max_regno;
   register int i;
-  int min_labelno = inl_f->inl_emit->x_first_label_num;
+  int min_labelno = inl_f->emit->x_first_label_num;
   int max_labelno = inl_f->inl_max_label_num;
   int nargs;
-  rtx local_return_label = 0;
   rtx loc;
   rtx stack_save = 0;
   rtx temp;
@@ -1357,12 +649,15 @@ expand_inline_function (fndecl, parms, target, ignore, type,
   rtx *real_label_map = 0;
 
   /* Allow for equivalences of the pseudos we make for virtual fp and ap.  */
-  max_regno = inl_f->inl_emit->x_reg_rtx_no + 3;
+  max_regno = inl_f->emit->x_reg_rtx_no + 3;
   if (max_regno < FIRST_PSEUDO_REGISTER)
     abort ();
 
   nargs = list_length (DECL_ARGUMENTS (fndecl));
 
+  if (cfun->preferred_stack_boundary < inl_f->preferred_stack_boundary)
+    cfun->preferred_stack_boundary = inl_f->preferred_stack_boundary;
+
   /* Check that the parms type match and that sufficient arguments were
      passed.  Since the appropriate conversions or default promotions have
      already been applied, the machine modes should match exactly.  */
@@ -1396,16 +691,11 @@ expand_inline_function (fndecl, parms, target, ignore, type,
     expand_expr (TREE_VALUE (actual), const0_rtx,
                 TYPE_MODE (TREE_TYPE (TREE_VALUE (actual))), 0);
 
-  /* Make a binding contour to keep inline cleanups called at
-     outer function-scope level from looking like they are shadowing
-     parameter declarations.  */
-  pushlevel (0);
-
   /* Expand the function arguments.  Do this first so that any
      new registers get created before we allocate the maps.  */
 
-  arg_vals = (rtx *) alloca (nargs * sizeof (rtx));
-  arg_trees = (tree *) alloca (nargs * sizeof (tree));
+  arg_vals = (rtx *) xmalloc (nargs * sizeof (rtx));
+  arg_trees = (tree *) xmalloc (nargs * sizeof (tree));
 
   for (formal = DECL_ARGUMENTS (fndecl), actual = parms, i = 0;
        formal;
@@ -1481,17 +771,16 @@ expand_inline_function (fndecl, parms, target, ignore, type,
       if (arg_vals[i] != 0 && GET_CODE (arg_vals[i]) == REG
          && POINTER_TYPE_P (TREE_TYPE (formal)))
        mark_reg_pointer (arg_vals[i],
-                         (TYPE_ALIGN (TREE_TYPE (TREE_TYPE (formal)))
-                          / BITS_PER_UNIT));
+                         TYPE_ALIGN (TREE_TYPE (TREE_TYPE (formal))));
     }
        
   /* Allocate the structures we use to remap things.  */
 
-  map = (struct inline_remap *) alloca (sizeof (struct inline_remap));
+  map = (struct inline_remap *) xmalloc (sizeof (struct inline_remap));
   map->fndecl = fndecl;
 
-  map->reg_map = (rtx *) alloca (max_regno * sizeof (rtx));
-  bzero ((char *) map->reg_map, max_regno * sizeof (rtx));
+  VARRAY_TREE_INIT (map->block_map, 10, "block_map");
+  map->reg_map = (rtx *) xcalloc (max_regno, sizeof (rtx));
 
   /* We used to use alloca here, but the size of what it would try to
      allocate would occasionally cause it to exceed the stack limit and
@@ -1500,9 +789,8 @@ expand_inline_function (fndecl, parms, target, ignore, type,
     = (rtx *) xmalloc ((max_labelno) * sizeof (rtx));
   map->label_map = real_label_map;
 
-  inl_max_uid = (inl_f->inl_emit->x_cur_insn_uid + 1);
-  map->insn_map = (rtx *) alloca (inl_max_uid * sizeof (rtx));
-  bzero ((char *) map->insn_map, inl_max_uid * sizeof (rtx));
+  inl_max_uid = (inl_f->emit->x_cur_insn_uid + 1);
+  map->insn_map = (rtx *) xcalloc (inl_max_uid, sizeof (rtx));
   map->min_insnno = 0;
   map->max_insnno = inl_max_uid;
 
@@ -1536,8 +824,8 @@ expand_inline_function (fndecl, parms, target, ignore, type,
   if (map->insns_at_start == 0)
     map->insns_at_start = emit_note (NULL_PTR, NOTE_INSN_DELETED);
 
-  map->regno_pointer_flag = inl_f->inl_emit->regno_pointer_flag;
-  map->regno_pointer_align = inl_f->inl_emit->regno_pointer_align;
+  map->regno_pointer_flag = inl_f->emit->regno_pointer_flag;
+  map->regno_pointer_align = inl_f->emit->regno_pointer_align;
 
   /* Update the outgoing argument size to allow for those in the inlined
      function.  */
@@ -1603,11 +891,19 @@ expand_inline_function (fndecl, parms, target, ignore, type,
        }
       else if (GET_CODE (loc) == MEM)
        {
-         /* This is the case of a parameter that lives in memory.
-            It will live in the block we allocate in the called routine's
+         /* This is the case of a parameter that lives in memory.  It
+            will live in the block we allocate in the called routine's
             frame that simulates the incoming argument area.  Do nothing
-            now; we will call store_expr later.  */
-         ;
+            with the parameter now; we will call store_expr later.  In
+            this case, however, we must ensure that the virtual stack and
+            incoming arg rtx values are expanded now so that we can be
+            sure we have enough slots in the const equiv map since the
+            store_expr call can easily blow the size estimate.  */
+         if (DECL_FRAME_SIZE (fndecl) != 0)
+           copy_rtx_and_substitute (virtual_stack_vars_rtx, map, 0);
+
+         if (DECL_SAVED_INSNS (fndecl)->args_size != 0)
+           copy_rtx_and_substitute (virtual_incoming_args_rtx, map, 0);
        }
       else if (GET_CODE (loc) == REG)
        process_reg_param (map, loc, copy);
@@ -1625,6 +921,12 @@ expand_inline_function (fndecl, parms, target, ignore, type,
        abort ();
     }
 
+  /* Tell copy_rtx_and_substitute to handle constant pool SYMBOL_REFs
+     specially.  This function can be called recursively, so we need to
+     save the previous value.  */
+  inlining_previous = inlining;
+  inlining = inl_f;
+
   /* Now do the parameters that will be placed in memory.  */
 
   for (formal = DECL_ARGUMENTS (fndecl), i = 0;
@@ -1644,8 +946,8 @@ expand_inline_function (fndecl, parms, target, ignore, type,
 
          /* Compute the address in the area we reserved and store the
             value there.  */
-         temp = copy_rtx_and_substitute (loc, map);
-         subst_constants (&temp, NULL_RTX, map);
+         temp = copy_rtx_and_substitute (loc, map, 1);
+         subst_constants (&temp, NULL_RTX, map, 1);
          apply_change_group ();
          if (! memory_address_p (GET_MODE (temp), XEXP (temp, 0)))
            temp = change_address (temp, VOIDmode, XEXP (temp, 0));
@@ -1669,8 +971,8 @@ expand_inline_function (fndecl, parms, target, ignore, type,
     {
       if (GET_CODE (XEXP (loc, 0)) == ADDRESSOF)
        {
-         temp = copy_rtx_and_substitute (loc, map);
-         subst_constants (&temp, NULL_RTX, map);
+         temp = copy_rtx_and_substitute (loc, map, 1);
+         subst_constants (&temp, NULL_RTX, map, 1);
          apply_change_group ();
          target = temp;
        }
@@ -1710,8 +1012,8 @@ expand_inline_function (fndecl, parms, target, ignore, type,
            }
          else
            {
-             temp = copy_rtx_and_substitute (loc, map);
-             subst_constants (&temp, NULL_RTX, map);
+             temp = copy_rtx_and_substitute (loc, map, 1);
+             subst_constants (&temp, NULL_RTX, map, 0);
              apply_change_group ();
              emit_move_insn (temp, structure_value_addr);
            }
@@ -1792,16 +1094,32 @@ expand_inline_function (fndecl, parms, target, ignore, type,
   else
     abort ();
 
-  /* Make a fresh binding contour that we can easily remove.  Do this after
-     expanding our arguments so cleanups are properly scoped.  */
-  pushlevel (0);
-  expand_start_bindings (0);
-
   /* Initialize label_map.  get_label_from_map will actually make
      the labels.  */
   bzero ((char *) &map->label_map [min_labelno],
         (max_labelno - min_labelno) * sizeof (rtx));
 
+  /* Make copies of the decls of the symbols in the inline function, so that
+     the copies of the variables get declared in the current function.  Set
+     up things so that lookup_static_chain knows that to interpret registers
+     in SAVE_EXPRs for TYPE_SIZEs as local.  */
+  inline_function_decl = fndecl;
+  integrate_parm_decls (DECL_ARGUMENTS (fndecl), map, arg_vector);
+  block = integrate_decl_tree (inl_f->original_decl_initial, map);
+  BLOCK_ABSTRACT_ORIGIN (block) = DECL_ORIGIN (fndecl);
+  inline_function_decl = 0;
+
+  /* Make a fresh binding contour that we can easily remove.  Do this after
+     expanding our arguments so cleanups are properly scoped.  */
+  expand_start_bindings_and_block (0, block);
+
+  /* Sort the block-map so that it will be easy to find remapped
+     blocks later.  */
+  qsort (&VARRAY_TREE (map->block_map, 0), 
+        map->block_map->elements_used,
+        sizeof (tree),
+        compare_blocks);
+
   /* Perform postincrements before actually calling the function.  */
   emit_queue ();
 
@@ -1819,7 +1137,100 @@ expand_inline_function (fndecl, parms, target, ignore, type,
   if (inl_f->calls_alloca)
     emit_stack_save (SAVE_BLOCK, &stack_save, NULL_RTX);
 
-  /* Now copy the insns one by one.  Do this in two passes, first the insns and
+  /* Now copy the insns one by one.  */
+  copy_insn_list (insns, map, static_chain_value);
+
+  /* Restore the stack pointer if we saved it above.  */
+  if (inl_f->calls_alloca)
+    emit_stack_restore (SAVE_BLOCK, stack_save, NULL_RTX);
+
+  if (! cfun->x_whole_function_mode_p)
+    /* In statement-at-a-time mode, we just tell the front-end to add
+       this block to the list of blocks at this binding level.  We
+       can't do it the way it's done for function-at-a-time mode the
+       superblocks have not been created yet.  */
+    insert_block (block);
+  else
+    {
+      BLOCK_CHAIN (block) 
+       = BLOCK_CHAIN (DECL_INITIAL (current_function_decl));
+      BLOCK_CHAIN (DECL_INITIAL (current_function_decl)) = block;
+    }
+
+  /* End the scope containing the copied formal parameter variables
+     and copied LABEL_DECLs.  We pass NULL_TREE for the variables list
+     here so that expand_end_bindings will not check for unused
+     variables.  That's already been checked for when the inlined
+     function was defined.  */
+  expand_end_bindings (NULL_TREE, 1, 1);
+
+  /* Must mark the line number note after inlined functions as a repeat, so
+     that the test coverage code can avoid counting the call twice.  This
+     just tells the code to ignore the immediately following line note, since
+     there already exists a copy of this note before the expanded inline call.
+     This line number note is still needed for debugging though, so we can't
+     delete it.  */
+  if (flag_test_coverage)
+    emit_note (0, NOTE_INSN_REPEATED_LINE_NUMBER);
+
+  emit_line_note (input_filename, lineno);
+
+  /* If the function returns a BLKmode object in a register, copy it
+     out of the temp register into a BLKmode memory object. */
+  if (target 
+      && TYPE_MODE (TREE_TYPE (TREE_TYPE (fndecl))) == BLKmode
+      && ! aggregate_value_p (TREE_TYPE (TREE_TYPE (fndecl))))
+    target = copy_blkmode_from_reg (0, target, TREE_TYPE (TREE_TYPE (fndecl)));
+  
+  if (structure_value_addr)
+    {
+      target = gen_rtx_MEM (TYPE_MODE (type),
+                           memory_address (TYPE_MODE (type),
+                                           structure_value_addr));
+      set_mem_attributes (target, type, 1);
+    }
+
+  /* Make sure we free the things we explicitly allocated with xmalloc.  */
+  if (real_label_map)
+    free (real_label_map);
+  VARRAY_FREE (map->const_equiv_varray);
+  free (map->reg_map);
+  VARRAY_FREE (map->block_map);
+  free (map->insn_map);
+  free (map);
+  free (arg_vals);
+  free (arg_trees);
+
+  inlining = inlining_previous;
+
+  return target;
+}
+
+/* Make copies of each insn in the given list using the mapping
+   computed in expand_inline_function. This function may call itself for
+   insns containing sequences.
+   
+   Copying is done in two passes, first the insns and then their REG_NOTES,
+   just like save_for_inline.
+
+   If static_chain_value is non-zero, it represents the context-pointer
+   register for the function. */
+
+static void
+copy_insn_list (insns, map, static_chain_value)
+    rtx insns;
+    struct inline_remap *map;
+    rtx static_chain_value;
+{
+  register int i;
+  rtx insn;
+  rtx temp;
+  rtx local_return_label = NULL_RTX;
+#ifdef HAVE_cc0
+  rtx cc0_insn = 0;
+#endif
+
+  /* Copy the insns one by one.  Do this in two passes, first the insns and
      then their REG_NOTES, just like save_for_inline.  */
 
   /* This loop is very similar to the loop in copy_loop_body in unroll.c.  */
@@ -1862,7 +1273,7 @@ expand_inline_function (fndecl, parms, target, ignore, type,
 
                  /* If we must not delete the source,
                     load it into a new temporary.  */
-                 copy = emit_insn (copy_rtx_and_substitute (pattern, map));
+                 copy = emit_insn (copy_rtx_and_substitute (pattern, map, 0));
 
                  new_set = single_set (copy);
                  if (new_set == 0)
@@ -1875,7 +1286,7 @@ expand_inline_function (fndecl, parms, target, ignore, type,
                 has a note on it, keep the insn.  */
              else if (rtx_equal_p (SET_DEST (set), SET_SRC (set))
                       && REG_NOTES (insn) != 0)
-               copy = emit_insn (copy_rtx_and_substitute (pattern, map));
+               copy = emit_insn (copy_rtx_and_substitute (pattern, map, 0));
              else
                break;
            }
@@ -1895,13 +1306,48 @@ expand_inline_function (fndecl, parms, target, ignore, type,
                   && rtx_equal_p (SET_SRC (set),
                                   static_chain_incoming_rtx))
            {
-             rtx newdest = copy_rtx_and_substitute (SET_DEST (set), map);
+             rtx newdest = copy_rtx_and_substitute (SET_DEST (set), map, 1);
 
              copy = emit_move_insn (newdest, static_chain_value);
              static_chain_value = 0;
            }
+
+         /* If this is setting the virtual stack vars register, this must
+            be the code at the handler for a builtin longjmp.  The value
+            saved in the setjmp buffer will be the address of the frame
+            we've made for this inlined instance within our frame.  But we
+            know the offset of that value so we can use it to reconstruct
+            our virtual stack vars register from that value.  If we are
+            copying it from the stack pointer, leave it unchanged.  */
+         else if (set != 0
+                  && rtx_equal_p (SET_DEST (set), virtual_stack_vars_rtx))
+           {
+             HOST_WIDE_INT offset;
+             temp = map->reg_map[REGNO (SET_DEST (set))];
+             temp = VARRAY_CONST_EQUIV (map->const_equiv_varray,
+                                        REGNO (temp)).rtx;
+
+             if (rtx_equal_p (temp, virtual_stack_vars_rtx))
+               offset = 0;
+             else if (GET_CODE (temp) == PLUS
+                      && rtx_equal_p (XEXP (temp, 0), virtual_stack_vars_rtx)
+                      && GET_CODE (XEXP (temp, 1)) == CONST_INT)
+               offset = INTVAL (XEXP (temp, 1));
+             else
+               abort ();
+
+             if (rtx_equal_p (SET_SRC (set), stack_pointer_rtx))
+               temp = SET_SRC (set);
+             else
+               temp = force_operand (plus_constant (SET_SRC (set),
+                                                    - offset),
+                                     NULL_RTX);
+
+             copy = emit_move_insn (virtual_stack_vars_rtx, temp);
+           }
+
          else
-           copy = emit_insn (copy_rtx_and_substitute (pattern, map));
+           copy = emit_insn (copy_rtx_and_substitute (pattern, map, 0));
          /* REG_NOTES will be copied later.  */
 
 #ifdef HAVE_cc0
@@ -1942,7 +1388,7 @@ expand_inline_function (fndecl, parms, target, ignore, type,
              pattern = gen_jump (local_return_label);
            }
          else
-           pattern = copy_rtx_and_substitute (PATTERN (insn), map);
+           pattern = copy_rtx_and_substitute (PATTERN (insn), map, 0);
 
          copy = emit_jump_insn (pattern);
 
@@ -1955,7 +1401,7 @@ expand_inline_function (fndecl, parms, target, ignore, type,
 
          /* If this used to be a conditional jump insn but whose branch
             direction is now know, we must do something special.  */
-         if (condjump_p (insn) && ! simplejump_p (insn) && map->last_pc_value)
+         if (any_condjump_p (insn) && onlyjump_p (insn) && map->last_pc_value)
            {
 #ifdef HAVE_cc0
              /* If the previous insn set cc0 for us, delete it.  */
@@ -1978,13 +1424,53 @@ expand_inline_function (fndecl, parms, target, ignore, type,
          break;
 
        case CALL_INSN:
-         pattern = copy_rtx_and_substitute (PATTERN (insn), map);
+         /* If this is a CALL_PLACEHOLDER insn then we need to copy the
+            three attached sequences: normal call, sibling call and tail
+            recursion. */
+         if (GET_CODE (PATTERN (insn)) == CALL_PLACEHOLDER)
+           {
+             rtx sequence[3];
+             rtx tail_label;
+
+             for (i = 0; i < 3; i++)
+               {
+                 rtx seq;
+                 
+                 sequence[i] = NULL_RTX;
+                 seq = XEXP (PATTERN (insn), i);
+                 if (seq)
+                   {
+                     start_sequence ();
+                     copy_insn_list (seq, map, static_chain_value);
+                     sequence[i] = get_insns ();
+                     end_sequence ();
+                   }
+               }
+
+             /* Find the new tail recursion label.  
+                It will already be substituted into sequence[2].  */
+             tail_label = copy_rtx_and_substitute (XEXP (PATTERN (insn), 3),
+                                                   map, 0);
+
+             copy = emit_call_insn (gen_rtx_CALL_PLACEHOLDER (VOIDmode, 
+                                                       sequence[0],
+                                                       sequence[1],
+                                                       sequence[2],
+                                                       tail_label));
+             break;
+           }
+
+         pattern = copy_rtx_and_substitute (PATTERN (insn), map, 0);
          copy = emit_call_insn (pattern);
 
+         SIBLING_CALL_P (copy) = SIBLING_CALL_P (insn);
+
          /* Because the USAGE information potentially contains objects other
             than hard registers, we need to copy it.  */
+
          CALL_INSN_FUNCTION_USAGE (copy)
-           = copy_rtx_and_substitute (CALL_INSN_FUNCTION_USAGE (insn), map);
+           = copy_rtx_and_substitute (CALL_INSN_FUNCTION_USAGE (insn),
+                                      map, 0);
 
 #ifdef HAVE_cc0
          if (cc0_insn)
@@ -1993,7 +1479,7 @@ expand_inline_function (fndecl, parms, target, ignore, type,
 #endif
          try_constants (copy, map);
 
-         /* Be lazy and assume CALL_INSNs clobber all hard registers.  */
+             /* Be lazy and assume CALL_INSNs clobber all hard registers.  */
          for (i = 0; i < FIRST_PSEUDO_REGISTER; i++)
            VARRAY_CONST_EQUIV (map->const_equiv_varray, i).rtx = 0;
          break;
@@ -2010,14 +1496,23 @@ expand_inline_function (fndecl, parms, target, ignore, type,
          break;
 
        case NOTE:
-         /* It is important to discard function-end and function-beg notes,
-            so we have only one of each in the current function.
-            Also, NOTE_INSN_DELETED notes aren't useful (save_for_inline
+         /* NOTE_INSN_FUNCTION_END and NOTE_INSN_FUNCTION_BEG are 
+            discarded because it is important to have only one of 
+            each in the current function.
+
+            NOTE_INSN_DELETED notes aren't useful (save_for_inline
             deleted these in the copy used for continuing compilation,
-            not the copy used for inlining).  */
+            not the copy used for inlining).
+
+            NOTE_INSN_BASIC_BLOCK is discarded because the saved bb
+            pointer (which will soon be dangling) confuses flow's
+            attempts to preserve bb structures during the compilation
+            of a function.  */
+
          if (NOTE_LINE_NUMBER (insn) != NOTE_INSN_FUNCTION_END
              && NOTE_LINE_NUMBER (insn) != NOTE_INSN_FUNCTION_BEG
-             && NOTE_LINE_NUMBER (insn) != NOTE_INSN_DELETED)
+             && NOTE_LINE_NUMBER (insn) != NOTE_INSN_DELETED
+             && NOTE_LINE_NUMBER (insn) != NOTE_INSN_BASIC_BLOCK)
            {
              copy = emit_note (NOTE_SOURCE_FILE (insn),
                                NOTE_LINE_NUMBER (insn));
@@ -2026,7 +1521,7 @@ expand_inline_function (fndecl, parms, target, ignore, type,
                      || NOTE_LINE_NUMBER (copy) == NOTE_INSN_EH_REGION_END))
                {
                  rtx label
-                   = get_label_from_map (map, NOTE_BLOCK_NUMBER (copy));
+                   = get_label_from_map (map, NOTE_EH_HANDLER (copy));
 
                   /* we have to duplicate the handlers for the original */
                   if (NOTE_LINE_NUMBER (copy) == NOTE_INSN_EH_REGION_BEG)
@@ -2034,14 +1529,33 @@ expand_inline_function (fndecl, parms, target, ignore, type,
                       /* We need to duplicate the handlers for the EH region
                          and we need to indicate where the label map is */
                       eif_eh_map = map;
-                      duplicate_eh_handlers (NOTE_BLOCK_NUMBER (copy), 
+                      duplicate_eh_handlers (NOTE_EH_HANDLER (copy), 
                                              CODE_LABEL_NUMBER (label),
                                              expand_inline_function_eh_labelmap);
                     }
 
                  /* We have to forward these both to match the new exception
                     region.  */
-                 NOTE_BLOCK_NUMBER (copy) = CODE_LABEL_NUMBER (label);
+                 NOTE_EH_HANDLER (copy) = CODE_LABEL_NUMBER (label);
+               }
+             else if (copy
+                      && (NOTE_LINE_NUMBER (copy) == NOTE_INSN_BLOCK_BEG
+                          || NOTE_LINE_NUMBER (copy) == NOTE_INSN_BLOCK_END)
+                      && NOTE_BLOCK (insn))
+               {
+                 tree *mapped_block_p;
+
+                 mapped_block_p
+                   = (tree *) bsearch (NOTE_BLOCK (insn), 
+                                       &VARRAY_TREE (map->block_map, 0),
+                                       map->block_map->elements_used,
+                                       sizeof (tree),
+                                       find_block);
+                 
+                 if (!mapped_block_p)
+                   abort ();
+                 else
+                   NOTE_BLOCK (copy) = *mapped_block_p;
                }
            }
          else
@@ -2050,7 +1564,6 @@ expand_inline_function (fndecl, parms, target, ignore, type,
 
        default:
          abort ();
-         break;
        }
 
       if (copy)
@@ -2064,76 +1577,29 @@ expand_inline_function (fndecl, parms, target, ignore, type,
      are valid across the entire function.  */
   map->const_age++;
   for (insn = insns; insn; insn = NEXT_INSN (insn))
-    if (GET_RTX_CLASS (GET_CODE (insn)) == 'i'
+    if (INSN_P (insn)
        && map->insn_map[INSN_UID (insn)]
        && REG_NOTES (insn))
       {
-       rtx tem = copy_rtx_and_substitute (REG_NOTES (insn), map);
+       rtx next, note = copy_rtx_and_substitute (REG_NOTES (insn), map, 0);
+
        /* We must also do subst_constants, in case one of our parameters
           has const type and constant value.  */
-       subst_constants (&tem, NULL_RTX, map);
+       subst_constants (&note, NULL_RTX, map, 0);
        apply_change_group ();
-       REG_NOTES (map->insn_map[INSN_UID (insn)]) = tem;
+       REG_NOTES (map->insn_map[INSN_UID (insn)]) = note;
+
+       /* Finally, delete any REG_LABEL notes from the chain.  */                
+       for (; note; note = next)
+         {
+           next = XEXP (note, 1);
+           if (REG_NOTE_KIND (note) == REG_LABEL)
+             remove_note (map->insn_map[INSN_UID (insn)], note);
+         }
       }
 
   if (local_return_label)
     emit_label (local_return_label);
-
-  /* Restore the stack pointer if we saved it above.  */
-  if (inl_f->calls_alloca)
-    emit_stack_restore (SAVE_BLOCK, stack_save, NULL_RTX);
-
-  /* Make copies of the decls of the symbols in the inline function, so that
-     the copies of the variables get declared in the current function.  Set
-     up things so that lookup_static_chain knows that to interpret registers
-     in SAVE_EXPRs for TYPE_SIZEs as local.  */
-
-  inline_function_decl = fndecl;
-  integrate_parm_decls (DECL_ARGUMENTS (fndecl), map, arg_vector);
-  integrate_decl_tree (inl_f->original_decl_initial, 0, map);
-  inline_function_decl = 0;
-
-  /* End the scope containing the copied formal parameter variables
-     and copied LABEL_DECLs.  */
-
-  expand_end_bindings (getdecls (), 1, 1);
-  block = poplevel (1, 1, 0);
-  BLOCK_ABSTRACT_ORIGIN (block) = (DECL_ABSTRACT_ORIGIN (fndecl) == NULL
-                                  ? fndecl : DECL_ABSTRACT_ORIGIN (fndecl));
-  poplevel (0, 0, 0);
-
-  /* Must mark the line number note after inlined functions as a repeat, so
-     that the test coverage code can avoid counting the call twice.  This
-     just tells the code to ignore the immediately following line note, since
-     there already exists a copy of this note before the expanded inline call.
-     This line number note is still needed for debugging though, so we can't
-     delete it.  */
-  if (flag_test_coverage)
-    emit_note (0, NOTE_REPEATED_LINE_NUMBER);
-
-  emit_line_note (input_filename, lineno);
-
-  /* If the function returns a BLKmode object in a register, copy it
-     out of the temp register into a BLKmode memory object. */
-  if (TYPE_MODE (TREE_TYPE (TREE_TYPE (fndecl))) == BLKmode
-      && ! aggregate_value_p (TREE_TYPE (TREE_TYPE (fndecl))))
-    target = copy_blkmode_from_reg (0, target, TREE_TYPE (TREE_TYPE (fndecl)));
-  
-  if (structure_value_addr)
-    {
-      target = gen_rtx_MEM (TYPE_MODE (type),
-                           memory_address (TYPE_MODE (type),
-                                           structure_value_addr));
-      MEM_SET_IN_STRUCT_P (target, 1);
-    }
-
-  /* Make sure we free the things we explicitly allocated with xmalloc.  */
-  if (real_label_map)
-    free (real_label_map);
-  if (map)
-    VARRAY_FREE (map->const_equiv_varray);
-
-  return target;
 }
 \f
 /* Given a chain of PARM_DECLs, ARGS, copy each decl into a VAR_DECL,
@@ -2150,25 +1616,19 @@ integrate_parm_decls (args, map, arg_vector)
 
   for (tail = args, i = 0; tail; tail = TREE_CHAIN (tail), i++)
     {
-      register tree decl = build_decl (VAR_DECL, DECL_NAME (tail),
-                                      TREE_TYPE (tail));
+      tree decl = copy_decl_for_inlining (tail, map->fndecl,
+                                         current_function_decl);
       rtx new_decl_rtl
-       = copy_rtx_and_substitute (RTVEC_ELT (arg_vector, i), map);
+       = copy_rtx_and_substitute (RTVEC_ELT (arg_vector, i), map, 1);
 
-      DECL_ARG_TYPE (decl) = DECL_ARG_TYPE (tail);
       /* We really should be setting DECL_INCOMING_RTL to something reasonable
         here, but that's going to require some more work.  */
       /* DECL_INCOMING_RTL (decl) = ?; */
-      /* These args would always appear unused, if not for this.  */
-      TREE_USED (decl) = 1;
-      /* Prevent warning for shadowing with these.  */
-      DECL_ABSTRACT_ORIGIN (decl) = DECL_ORIGIN (tail);
-      pushdecl (decl);
       /* Fully instantiate the address with the equivalent form so that the
         debugging information contains the actual register, instead of the
         virtual register.   Do this by not passing an insn to
         subst_constants.  */
-      subst_constants (&new_decl_rtl, NULL_RTX, map);
+      subst_constants (&new_decl_rtl, NULL_RTX, map, 1);
       apply_change_group ();
       DECL_RTL (decl) = new_decl_rtl;
     }
@@ -2177,104 +1637,90 @@ integrate_parm_decls (args, map, arg_vector)
 /* Given a BLOCK node LET, push decls and levels so as to construct in the
    current function a tree of contexts isomorphic to the one that is given.
 
-   LEVEL indicates how far down into the BLOCK tree is the node we are
-   currently traversing.  It is always zero except for recursive calls.
-
    MAP, if nonzero, is a pointer to an inline_remap map which indicates how
    registers used in the DECL_RTL field should be remapped.  If it is zero,
    no mapping is necessary.  */
 
-static void
-integrate_decl_tree (let, level, map)
+static tree
+integrate_decl_tree (let, map)
      tree let;
-     int level;
      struct inline_remap *map;
 {
-  tree t, node;
+  tree t;
+  tree new_block;
+  tree *next;
+
+  new_block = make_node (BLOCK);
+  VARRAY_PUSH_TREE (map->block_map, new_block);
+  next = &BLOCK_VARS (new_block);
 
-  if (level > 0)
-    pushlevel (0);
-  
   for (t = BLOCK_VARS (let); t; t = TREE_CHAIN (t))
     {
       tree d;
 
       push_obstacks_nochange ();
       saveable_allocation ();
-      d = copy_and_set_decl_abstract_origin (t);
+      d = copy_decl_for_inlining (t, map->fndecl, current_function_decl);
       pop_obstacks ();
 
       if (DECL_RTL (t) != 0)
        {
-         DECL_RTL (d) = copy_rtx_and_substitute (DECL_RTL (t), map);
+         DECL_RTL (d) = copy_rtx_and_substitute (DECL_RTL (t), map, 1);
+
          /* Fully instantiate the address with the equivalent form so that the
             debugging information contains the actual register, instead of the
             virtual register.   Do this by not passing an insn to
             subst_constants.  */
-         subst_constants (&DECL_RTL (d), NULL_RTX, map);
+         subst_constants (&DECL_RTL (d), NULL_RTX, map, 1);
          apply_change_group ();
        }
-      /* These args would always appear unused, if not for this.  */
-      TREE_USED (d) = 1;
 
-      if (DECL_LANG_SPECIFIC (d))
-       copy_lang_decl (d);
-
-      pushdecl (d);
+      /* Add this declaration to the list of variables in the new
+        block.  */
+      *next = d;
+      next = &TREE_CHAIN (d);
     }
 
-  for (t = BLOCK_SUBBLOCKS (let); t; t = TREE_CHAIN (t))
-    integrate_decl_tree (t, level + 1, map);
-
-  if (level > 0)
+  next = &BLOCK_SUBBLOCKS (new_block);
+  for (t = BLOCK_SUBBLOCKS (let); t; t = BLOCK_CHAIN (t))
     {
-      node = poplevel (1, 0, 0);
-      if (node)
-       {
-         TREE_USED (node) = TREE_USED (let);
-         BLOCK_ABSTRACT_ORIGIN (node) = let;
-       }
+      *next = integrate_decl_tree (t, map);
+      BLOCK_SUPERCONTEXT (*next) = new_block;
+      next = &BLOCK_CHAIN (*next);
     }
-}
-
-/* Given a BLOCK node LET, search for all DECL_RTL fields, and pass them
-   through save_constants.  */
 
-static void
-save_constants_in_decl_trees (let)
-     tree let;
-{
-  tree t;
-
-  for (t = BLOCK_VARS (let); t; t = TREE_CHAIN (t))
-    if (DECL_RTL (t) != 0)
-      save_constants (&DECL_RTL (t));
-
-  for (t = BLOCK_SUBBLOCKS (let); t; t = TREE_CHAIN (t))
-    save_constants_in_decl_trees (t);
+  TREE_USED (new_block) = TREE_USED (let);
+  BLOCK_ABSTRACT_ORIGIN (new_block) = let;
+  
+  return new_block;
 }
 \f
-/* Create a new copy of an rtx.
-   Recursively copies the operands of the rtx,
+/* Create a new copy of an rtx. Recursively copies the operands of the rtx,
    except for those few rtx codes that are sharable.
 
    We always return an rtx that is similar to that incoming rtx, with the
    exception of possibly changing a REG to a SUBREG or vice versa.  No
    rtl is ever emitted.
 
+   If FOR_LHS is nonzero, if means we are processing something that will
+   be the LHS of a SET.  In that case, we copy RTX_UNCHANGING_P even if
+   inlining since we need to be conservative in how it is set for
+   such cases.
+
    Handle constants that need to be placed in the constant pool by
    calling `force_const_mem'.  */
 
 rtx
-copy_rtx_and_substitute (orig, map)
+copy_rtx_and_substitute (orig, map, for_lhs)
      register rtx orig;
      struct inline_remap *map;
+     int for_lhs;
 {
   register rtx copy, temp;
   register int i, j;
   register RTX_CODE code;
   register enum machine_mode mode;
-  register char *format_ptr;
+  register const char *format_ptr;
   int regno;
 
   if (orig == 0)
@@ -2291,7 +1737,9 @@ copy_rtx_and_substitute (orig, map)
         Small hard registers are returned as-is.  Pseudo-registers
         go through their `reg_map'.  */
       regno = REGNO (orig);
-      if (regno <= LAST_VIRTUAL_REGISTER)
+      if (regno <= LAST_VIRTUAL_REGISTER
+         || (map->integrating
+             && DECL_SAVED_INSNS (map->fndecl)->internal_arg_pointer == orig))
        {
          /* Some hard registers are also mapped,
             but others are not translated.  */
@@ -2310,13 +1758,17 @@ copy_rtx_and_substitute (orig, map)
            {
              rtx loc, seq;
              int size = get_func_frame_size (DECL_SAVED_INSNS (map->fndecl));
-
 #ifdef FRAME_GROWS_DOWNWARD
+             int alignment
+               = (DECL_SAVED_INSNS (map->fndecl)->stack_alignment_needed
+                  / BITS_PER_UNIT);
+
              /* In this case, virtual_stack_vars_rtx points to one byte
                 higher than the top of the frame area.  So make sure we
                 allocate a big enough chunk to keep the frame pointer
                 aligned like a real one.  */
-             size = CEIL_ROUND (size, BIGGEST_ALIGNMENT / BITS_PER_UNIT);
+             if (alignment)
+               size = CEIL_ROUND (size, alignment);
 #endif
              start_sequence ();
              loc = assign_stack_temp (BLKmode, size, 1);
@@ -2331,8 +1783,7 @@ copy_rtx_and_substitute (orig, map)
                = force_reg (Pmode, force_operand (loc, NULL_RTX));
 
 #ifdef STACK_BOUNDARY
-             mark_reg_pointer (map->reg_map[regno],
-                               STACK_BOUNDARY / BITS_PER_UNIT);
+             mark_reg_pointer (map->reg_map[regno], STACK_BOUNDARY);
 #endif
 
              SET_CONST_EQUIV_DATA (map, temp, loc, CONST_AGE_PARM);
@@ -2342,7 +1793,10 @@ copy_rtx_and_substitute (orig, map)
              emit_insn_after (seq, map->insns_at_start);
              return temp;
            }
-         else if (regno == VIRTUAL_INCOMING_ARGS_REGNUM)
+         else if (regno == VIRTUAL_INCOMING_ARGS_REGNUM
+                  || (map->integrating
+                      && (DECL_SAVED_INSNS (map->fndecl)->internal_arg_pointer
+                          == orig)))
            {
              /* Do the same for a block to contain any arguments referenced
                 in memory.  */
@@ -2362,8 +1816,7 @@ copy_rtx_and_substitute (orig, map)
                = force_reg (Pmode, force_operand (loc, NULL_RTX));
 
 #ifdef STACK_BOUNDARY
-             mark_reg_pointer (map->reg_map[regno],
-                               STACK_BOUNDARY / BITS_PER_UNIT);
+             mark_reg_pointer (map->reg_map[regno], STACK_BOUNDARY);
 #endif
 
              SET_CONST_EQUIV_DATA (map, temp, loc, CONST_AGE_PARM);
@@ -2405,7 +1858,7 @@ copy_rtx_and_substitute (orig, map)
       return map->reg_map[regno];
 
     case SUBREG:
-      copy = copy_rtx_and_substitute (SUBREG_REG (orig), map);
+      copy = copy_rtx_and_substitute (SUBREG_REG (orig), map, for_lhs);
       /* SUBREG is ordinary, but don't make nested SUBREGs.  */
       if (GET_CODE (copy) == SUBREG)
        return gen_rtx_SUBREG (GET_MODE (orig), SUBREG_REG (copy),
@@ -2428,8 +1881,9 @@ copy_rtx_and_substitute (orig, map)
 
     case ADDRESSOF:
       copy = gen_rtx_ADDRESSOF (mode,
-                       copy_rtx_and_substitute (XEXP (orig, 0), map), 0);
-      SET_ADDRESSOF_DECL (copy, ADDRESSOF_DECL (orig));
+                               copy_rtx_and_substitute (XEXP (orig, 0),
+                                                        map, for_lhs),
+                               0, ADDRESSOF_DECL(orig));
       regno = ADDRESSOF_REGNO (orig);
       if (map->reg_map[regno])
        regno = REGNO (map->reg_map[regno]);
@@ -2456,7 +1910,7 @@ copy_rtx_and_substitute (orig, map)
         to (use foo) if the original insn didn't have a subreg.
         Removing the subreg distorts the VAX movstrhi pattern
         by changing the mode of an operand.  */
-      copy = copy_rtx_and_substitute (XEXP (orig, 0), map);
+      copy = copy_rtx_and_substitute (XEXP (orig, 0), map, code == CLOBBER);
       if (GET_CODE (copy) == SUBREG && GET_CODE (XEXP (orig, 0)) != SUBREG)
        copy = SUBREG_REG (copy);
       return gen_rtx_fmt_e (code, VOIDmode, copy);
@@ -2466,11 +1920,20 @@ copy_rtx_and_substitute (orig, map)
        = LABEL_PRESERVE_P (orig);
       return get_label_from_map (map, CODE_LABEL_NUMBER (orig));
 
+    /* We need to handle "deleted" labels that appear in the DECL_RTL
+       of a LABEL_DECL.  */
+    case NOTE:
+      if (NOTE_LINE_NUMBER (orig) == NOTE_INSN_DELETED_LABEL)
+       return map->insn_map[INSN_UID (orig)];
+      break;
+
     case LABEL_REF:
-      copy = gen_rtx_LABEL_REF (mode,
-                               LABEL_REF_NONLOCAL_P (orig) ? XEXP (orig, 0)
-                               : get_label_from_map (map, 
-                                                     CODE_LABEL_NUMBER (XEXP (orig, 0))));
+      copy
+       = gen_rtx_LABEL_REF
+         (mode,
+          LABEL_REF_NONLOCAL_P (orig) ? XEXP (orig, 0)
+          : get_label_from_map (map, CODE_LABEL_NUMBER (XEXP (orig, 0))));
+
       LABEL_OUTSIDE_LOOP_P (copy) = LABEL_OUTSIDE_LOOP_P (orig);
 
       /* The fact that this label was previously nonlocal does not mean
@@ -2501,11 +1964,43 @@ copy_rtx_and_substitute (orig, map)
         remapped label.  Otherwise, symbols are returned unchanged.  */
       if (CONSTANT_POOL_ADDRESS_P (orig))
        {
-         rtx constant = get_pool_constant (orig);
-         if (GET_CODE (constant) == LABEL_REF)
-           return XEXP (force_const_mem (GET_MODE (orig),
+         struct function *f = inlining ? inlining : cfun;
+         rtx constant = get_pool_constant_for_function (f, orig);
+         enum machine_mode const_mode = get_pool_mode_for_function (f, orig);
+         if (inlining)
+           {
+             rtx temp = force_const_mem (const_mode,
                                          copy_rtx_and_substitute (constant,
-                                                                  map)),
+                                                                  map, 0));
+
+#if 0
+             /* Legitimizing the address here is incorrect.
+
+                Since we had a SYMBOL_REF before, we can assume it is valid
+                to have one in this position in the insn.
+
+                Also, change_address may create new registers.  These
+                registers will not have valid reg_map entries.  This can
+                cause try_constants() to fail because assumes that all
+                registers in the rtx have valid reg_map entries, and it may
+                end up replacing one of these new registers with junk.  */
+
+             if (! memory_address_p (GET_MODE (temp), XEXP (temp, 0)))
+               temp = change_address (temp, GET_MODE (temp), XEXP (temp, 0));
+#endif
+
+             temp = XEXP (temp, 0);
+
+#ifdef POINTERS_EXTEND_UNSIGNED
+             if (GET_MODE (temp) != GET_MODE (orig))
+               temp = convert_memory_address (GET_MODE (orig), temp);
+#endif
+             return temp;
+           }
+         else if (GET_CODE (constant) == LABEL_REF)
+           return XEXP (force_const_mem
+                        (GET_MODE (orig),
+                         copy_rtx_and_substitute (constant, map, for_lhs)),
                         0);
        }
       else
@@ -2537,62 +2032,8 @@ copy_rtx_and_substitute (orig, map)
       /* Make new constant pool entry for a constant
         that was in the pool of the inline function.  */
       if (RTX_INTEGRATED_P (orig))
-       {
-         /* If this was an address of a constant pool entry that itself
-            had to be placed in the constant pool, it might not be a
-            valid address.  So the recursive call below might turn it
-            into a register.  In that case, it isn't a constant any
-            more, so return it.  This has the potential of changing a
-            MEM into a REG, but we'll assume that it safe.  */
-         temp = copy_rtx_and_substitute (XEXP (orig, 0), map);
-         if (! CONSTANT_P (temp))
-           return temp;
-         return validize_mem (force_const_mem (GET_MODE (orig), temp));
-       }
-      break;
-
-    case ADDRESS:
-      /* If from constant pool address, make new constant pool entry and
-        return its address.  */
-      if (! RTX_INTEGRATED_P (orig))
        abort ();
-
-      temp
-       = force_const_mem (GET_MODE (XEXP (orig, 0)),
-                          copy_rtx_and_substitute (XEXP (XEXP (orig, 0), 0),
-                                                   map));
-
-#if 0
-      /* Legitimizing the address here is incorrect.
-
-        The only ADDRESS rtx's that can reach here are ones created by
-        save_constants.  Hence the operand of the ADDRESS is always valid
-        in this position of the instruction, since the original rtx without
-        the ADDRESS was valid.
-
-        The reason we don't legitimize the address here is that on the
-        Sparc, the caller may have a (high ...) surrounding this ADDRESS.
-        This code forces the operand of the address to a register, which
-        fails because we can not take the HIGH part of a register.
-
-        Also, change_address may create new registers.  These registers
-        will not have valid reg_map entries.  This can cause try_constants()
-        to fail because assumes that all registers in the rtx have valid
-        reg_map entries, and it may end up replacing one of these new
-        registers with junk.  */
-
-      if (! memory_address_p (GET_MODE (temp), XEXP (temp, 0)))
-       temp = change_address (temp, GET_MODE (temp), XEXP (temp, 0));
-#endif
-
-      temp = XEXP (temp, 0);
-
-#ifdef POINTERS_EXTEND_UNSIGNED
-      if (GET_MODE (temp) != GET_MODE (orig))
-       temp = convert_memory_address (GET_MODE (orig), temp);
-#endif
-
-      return temp;
+      break;
 
     case ASM_OPERANDS:
       /* If a single asm insn contains multiple output operands
@@ -2621,10 +2062,13 @@ copy_rtx_and_substitute (orig, map)
 #ifndef NO_FUNCTION_CSE
       if (! (optimize && ! flag_no_function_cse))
 #endif
-       return gen_rtx_CALL (GET_MODE (orig),
-                            gen_rtx_MEM (GET_MODE (XEXP (orig, 0)),
-                                         copy_rtx_and_substitute (XEXP (XEXP (orig, 0), 0), map)),
-                       copy_rtx_and_substitute (XEXP (orig, 1), map));
+       return
+         gen_rtx_CALL
+           (GET_MODE (orig),
+            gen_rtx_MEM (GET_MODE (XEXP (orig, 0)),
+                         copy_rtx_and_substitute (XEXP (XEXP (orig, 0), 0),
+                                                  map, 0)),
+            copy_rtx_and_substitute (XEXP (orig, 1), map, 0));
       break;
 
 #if 0
@@ -2646,35 +2090,55 @@ copy_rtx_and_substitute (orig, map)
          rtx equiv_loc;
          HOST_WIDE_INT loc_offset;
 
-         copy_rtx_and_substitute (SET_DEST (orig), map);
+         copy_rtx_and_substitute (SET_DEST (orig), map, for_lhs);
          equiv_reg = map->reg_map[REGNO (SET_DEST (orig))];
-         equiv_loc = VARRAY_CONST_EQUIV (map->const_equiv_varray, REGNO (equiv_reg)).rtx;
+         equiv_loc = VARRAY_CONST_EQUIV (map->const_equiv_varray,
+                                         REGNO (equiv_reg)).rtx;
          loc_offset
            = GET_CODE (equiv_loc) == REG ? 0 : INTVAL (XEXP (equiv_loc, 1));
+             
          return gen_rtx_SET (VOIDmode, SET_DEST (orig),
                              force_operand
                              (plus_constant
-                              (copy_rtx_and_substitute (SET_SRC (orig), map),
+                              (copy_rtx_and_substitute (SET_SRC (orig),
+                                                        map, 0),
                                - loc_offset),
                               NULL_RTX));
        }
+      else
+       return gen_rtx_SET (VOIDmode,
+                           copy_rtx_and_substitute (SET_DEST (orig), map, 1),
+                           copy_rtx_and_substitute (SET_SRC (orig), map, 0));
       break;
 
     case MEM:
+      if (inlining
+         && GET_CODE (XEXP (orig, 0)) == SYMBOL_REF
+         && CONSTANT_POOL_ADDRESS_P (XEXP (orig, 0)))
+       {
+         enum machine_mode const_mode
+           = get_pool_mode_for_function (inlining, XEXP (orig, 0));
+         rtx constant
+           = get_pool_constant_for_function (inlining, XEXP (orig, 0));
+
+         constant = copy_rtx_and_substitute (constant, map, 0);
+
+         /* If this was an address of a constant pool entry that itself
+            had to be placed in the constant pool, it might not be a
+            valid address.  So the recursive call might have turned it
+            into a register.  In that case, it isn't a constant any
+            more, so return it.  This has the potential of changing a
+            MEM into a REG, but we'll assume that it safe.  */
+         if (! CONSTANT_P (constant))
+           return constant;
+
+         return validize_mem (force_const_mem (const_mode, constant));
+       }
+
       copy = rtx_alloc (MEM);
       PUT_MODE (copy, mode);
-      XEXP (copy, 0) = copy_rtx_and_substitute (XEXP (orig, 0), map);
+      XEXP (copy, 0) = copy_rtx_and_substitute (XEXP (orig, 0), map, 0);
       MEM_COPY_ATTRIBUTES (copy, orig);
-      MEM_ALIAS_SET (copy) = MEM_ALIAS_SET (orig);
-
-      /* If doing function inlining, this MEM might not be const in the
-        function that it is being inlined into, and thus may not be
-        unchanging after function inlining.  Constant pool references are
-        handled elsewhere, so this doesn't lose RTX_UNCHANGING_P bits
-        for them.  */
-      if (! map->integrating)
-       RTX_UNCHANGING_P (copy) = RTX_UNCHANGING_P (orig);
-
       return copy;
       
     default:
@@ -2694,11 +2158,13 @@ copy_rtx_and_substitute (orig, map)
       switch (*format_ptr++)
        {
        case '0':
-         XEXP (copy, i) = XEXP (orig, i);
+         /* Copy this through the wide int field; that's safest.  */
+         X0WINT (copy, i) = X0WINT (orig, i);
          break;
 
        case 'e':
-         XEXP (copy, i) = copy_rtx_and_substitute (XEXP (orig, i), map);
+         XEXP (copy, i)
+           = copy_rtx_and_substitute (XEXP (orig, i), map, for_lhs);
          break;
 
        case 'u':
@@ -2714,7 +2180,8 @@ copy_rtx_and_substitute (orig, map)
              XVEC (copy, i) = rtvec_alloc (XVECLEN (orig, i));
              for (j = 0; j < XVECLEN (copy, i); j++)
                XVECEXP (copy, i, j)
-                 = copy_rtx_and_substitute (XVECEXP (orig, i, j), map);
+                 = copy_rtx_and_substitute (XVECEXP (orig, i, j),
+                                            map, for_lhs);
            }
          break;
 
@@ -2730,6 +2197,10 @@ copy_rtx_and_substitute (orig, map)
          XSTR (copy, i) = XSTR (orig, i);
          break;
 
+       case 't':
+         XTREE (copy, i) = XTREE (orig, i);
+         break;
+
        default:
          abort ();
        }
@@ -2755,13 +2226,18 @@ try_constants (insn, map)
   int i;
 
   map->num_sets = 0;
-  subst_constants (&PATTERN (insn), insn, map);
 
-  /* Apply the changes if they are valid; otherwise discard them.  */
+  /* First try just updating addresses, then other things.  This is
+     important when we have something like the store of a constant
+     into memory and we can update the memory address but the machine
+     does not support a constant source.  */
+  subst_constants (&PATTERN (insn), insn, map, 1);
+  apply_change_group ();
+  subst_constants (&PATTERN (insn), insn, map, 0);
   apply_change_group ();
 
   /* Show we don't know the value of anything stored or clobbered.  */
-  note_stores (PATTERN (insn), mark_stores);
+  note_stores (PATTERN (insn), mark_stores, NULL);
   map->last_pc_value = 0;
 #ifdef HAVE_cc0
   map->last_cc0_value = 0;
@@ -2804,18 +2280,21 @@ try_constants (insn, map)
    into insns; cse will do the latter task better.
 
    This function is also used to adjust address of items previously addressed
-   via the virtual stack variable or virtual incoming arguments registers.  */
+   via the virtual stack variable or virtual incoming arguments registers. 
+
+   If MEMONLY is nonzero, only make changes inside a MEM.  */
 
 static void
-subst_constants (loc, insn, map)
+subst_constants (loc, insn, map, memonly)
      rtx *loc;
      rtx insn;
      struct inline_remap *map;
+     int memonly;
 {
   rtx x = *loc;
-  register int i;
+  register int i, j;
   register enum rtx_code code;
-  register char *format_ptr;
+  register const char *format_ptr;
   int num_changes = num_validated_changes ();
   rtx new = 0;
   enum machine_mode op0_mode = MAX_MACHINE_MODE;
@@ -2835,7 +2314,8 @@ subst_constants (loc, insn, map)
 
 #ifdef HAVE_cc0
     case CC0:
-      validate_change (insn, loc, map->last_cc0_value, 1);
+      if (! memonly)
+       validate_change (insn, loc, map->last_cc0_value, 1);
       return;
 #endif
 
@@ -2844,24 +2324,25 @@ subst_constants (loc, insn, map)
       /* The only thing we can do with a USE or CLOBBER is possibly do
         some substitutions in a MEM within it.  */
       if (GET_CODE (XEXP (x, 0)) == MEM)
-       subst_constants (&XEXP (XEXP (x, 0), 0), insn, map);
+       subst_constants (&XEXP (XEXP (x, 0), 0), insn, map, 0);
       return;
 
     case REG:
       /* Substitute for parms and known constants.  Don't replace
         hard regs used as user variables with constants.  */
-      {
-       int regno = REGNO (x);
-       struct const_equiv_data *p;
-
-       if (! (regno < FIRST_PSEUDO_REGISTER && REG_USERVAR_P (x))
-           && regno < VARRAY_SIZE (map->const_equiv_varray)
-           && (p = &VARRAY_CONST_EQUIV (map->const_equiv_varray, regno),
-               p->rtx != 0)
-           && p->age >= map->const_age)
-         validate_change (insn, loc, p->rtx, 1);
-       return;
-      }
+      if (! memonly)
+       {
+         int regno = REGNO (x);
+         struct const_equiv_data *p;
+
+         if (! (regno < FIRST_PSEUDO_REGISTER && REG_USERVAR_P (x))
+             && (size_t) regno < VARRAY_SIZE (map->const_equiv_varray)
+             && (p = &VARRAY_CONST_EQUIV (map->const_equiv_varray, regno),
+                 p->rtx != 0)
+             && p->age >= map->const_age)
+           validate_change (insn, loc, p->rtx, 1);
+       }
+      return;
 
     case SUBREG:
       /* SUBREG applied to something other than a reg
@@ -2869,7 +2350,7 @@ subst_constants (loc, insn, map)
         be a special hack and we don't know how to treat it specially.
         Consider for example mulsidi3 in m68k.md.
         Ordinary SUBREG of a REG needs this special treatment.  */
-      if (GET_CODE (SUBREG_REG (x)) == REG)
+      if (! memonly && GET_CODE (SUBREG_REG (x)) == REG)
        {
          rtx inner = SUBREG_REG (x);
          rtx new = 0;
@@ -2879,7 +2360,7 @@ subst_constants (loc, insn, map)
             see what is inside, try to form the new SUBREG and see if that is
             valid.  We handle two cases: extracting a full word in an 
             integral mode and extracting the low part.  */
-         subst_constants (&inner, NULL_RTX, map);
+         subst_constants (&inner, NULL_RTX, map, 0);
 
          if (GET_MODE_CLASS (GET_MODE (x)) == MODE_INT
              && GET_MODE_SIZE (GET_MODE (x)) == UNITS_PER_WORD
@@ -2899,11 +2380,11 @@ subst_constants (loc, insn, map)
       break;
 
     case MEM:
-      subst_constants (&XEXP (x, 0), insn, map);
+      subst_constants (&XEXP (x, 0), insn, map, 0);
 
       /* If a memory address got spoiled, change it back.  */
-      if (insn != 0 && num_validated_changes () != num_changes
-         && !memory_address_p (GET_MODE (x), XEXP (x, 0)))
+      if (! memonly && insn != 0 && num_validated_changes () != num_changes
+         && ! memory_address_p (GET_MODE (x), XEXP (x, 0)))
        cancel_changes (num_changes);
       return;
 
@@ -2916,7 +2397,7 @@ subst_constants (loc, insn, map)
        rtx dest = *dest_loc;
        rtx src, tem;
 
-       subst_constants (&SET_SRC (x), insn, map);
+       subst_constants (&SET_SRC (x), insn, map, memonly);
        src = SET_SRC (x);
 
        while (GET_CODE (*dest_loc) == ZERO_EXTRACT
@@ -2925,15 +2406,15 @@ subst_constants (loc, insn, map)
          {
            if (GET_CODE (*dest_loc) == ZERO_EXTRACT)
              {
-               subst_constants (&XEXP (*dest_loc, 1), insn, map);
-               subst_constants (&XEXP (*dest_loc, 2), insn, map);
+               subst_constants (&XEXP (*dest_loc, 1), insn, map, memonly);
+               subst_constants (&XEXP (*dest_loc, 2), insn, map, memonly);
              }
            dest_loc = &XEXP (*dest_loc, 0);
          }
 
        /* Do substitute in the address of a destination in memory.  */
        if (GET_CODE (*dest_loc) == MEM)
-         subst_constants (&XEXP (*dest_loc, 0), insn, map);
+         subst_constants (&XEXP (*dest_loc, 0), insn, map, 0);
 
        /* Check for the case of DEST a SUBREG, both it and the underlying
           register are less than one word, and the SUBREG has the wider mode.
@@ -2995,22 +2476,22 @@ subst_constants (loc, insn, map)
 
        case 'e':
          if (XEXP (x, i))
-           subst_constants (&XEXP (x, i), insn, map);
+           subst_constants (&XEXP (x, i), insn, map, memonly);
          break;
 
        case 'u':
        case 'i':
        case 's':
        case 'w':
+       case 'n':
+       case 't':
          break;
 
        case 'E':
          if (XVEC (x, i) != NULL && XVECLEN (x, i) != 0)
-           {
-             int j;
-             for (j = 0; j < XVECLEN (x, i); j++)
-               subst_constants (&XVECEXP (x, i, j), insn, map);
-           }
+           for (j = 0; j < XVECLEN (x, i); j++)
+             subst_constants (&XVECEXP (x, i, j), insn, map, memonly);
+
          break;
 
        default:
@@ -3020,7 +2501,8 @@ subst_constants (loc, insn, map)
 
   /* If this is a commutative operation, move a constant to the second
      operand unless the second operand is already a CONST_INT.  */
-  if ((GET_RTX_CLASS (code) == 'c' || code == NE || code == EQ)
+  if (! memonly
+      && (GET_RTX_CLASS (code) == 'c' || code == NE || code == EQ)
       && CONSTANT_P (XEXP (x, 0)) && GET_CODE (XEXP (x, 1)) != CONST_INT)
     {
       rtx tem = XEXP (x, 0);
@@ -3029,45 +2511,56 @@ subst_constants (loc, insn, map)
     }
 
   /* Simplify the expression in case we put in some constants.  */
-  switch (GET_RTX_CLASS (code))
-    {
-    case '1':
-      if (op0_mode == MAX_MACHINE_MODE)
-       abort ();
-      new = simplify_unary_operation (code, GET_MODE (x),
-                                     XEXP (x, 0), op0_mode);
-      break;
-
-    case '<':
+  if (! memonly)
+    switch (GET_RTX_CLASS (code))
       {
-       enum machine_mode op_mode = GET_MODE (XEXP (x, 0));
-       if (op_mode == VOIDmode)
-         op_mode = GET_MODE (XEXP (x, 1));
-       new = simplify_relational_operation (code, op_mode,
-                                            XEXP (x, 0), XEXP (x, 1));
+      case '1':
+       if (op0_mode == MAX_MACHINE_MODE)
+         abort ();
+       new = simplify_unary_operation (code, GET_MODE (x),
+                                       XEXP (x, 0), op0_mode);
+       break;
+
+      case '<':
+       {
+         enum machine_mode op_mode = GET_MODE (XEXP (x, 0));
+
+         if (op_mode == VOIDmode)
+           op_mode = GET_MODE (XEXP (x, 1));
+         new = simplify_relational_operation (code, op_mode,
+                                              XEXP (x, 0), XEXP (x, 1));
 #ifdef FLOAT_STORE_FLAG_VALUE
-       if (new != 0 && GET_MODE_CLASS (GET_MODE (x)) == MODE_FLOAT)
-         new = ((new == const0_rtx) ? CONST0_RTX (GET_MODE (x))
-                : CONST_DOUBLE_FROM_REAL_VALUE (FLOAT_STORE_FLAG_VALUE,
-                                                GET_MODE (x)));
+         if (new != 0 && GET_MODE_CLASS (GET_MODE (x)) == MODE_FLOAT)
+           {
+             enum machine_mode mode = GET_MODE (x);
+             if (new == const0_rtx)
+               new = CONST0_RTX (mode);
+             else
+               {
+                 REAL_VALUE_TYPE val = FLOAT_STORE_FLAG_VALUE (mode);
+                 new = CONST_DOUBLE_FROM_REAL_VALUE (val, mode);
+               }
+           }
 #endif
-       break;
+         break;
       }
 
-    case '2':
-    case 'c':
-      new = simplify_binary_operation (code, GET_MODE (x),
-                                      XEXP (x, 0), XEXP (x, 1));
-      break;
+      case '2':
+      case 'c':
+       new = simplify_binary_operation (code, GET_MODE (x),
+                                        XEXP (x, 0), XEXP (x, 1));
+       break;
 
-    case 'b':
-    case '3':
-      if (op0_mode == MAX_MACHINE_MODE)
-       abort ();
-      new = simplify_ternary_operation (code, GET_MODE (x), op0_mode,
-                                       XEXP (x, 0), XEXP (x, 1), XEXP (x, 2));
-      break;
-    }
+      case 'b':
+      case '3':
+       if (op0_mode == MAX_MACHINE_MODE)
+         abort ();
+
+       new = simplify_ternary_operation (code, GET_MODE (x), op0_mode,
+                                         XEXP (x, 0), XEXP (x, 1),
+                                         XEXP (x, 2));
+       break;
+      }
 
   if (new)
     validate_change (insn, loc, new, 1);
@@ -3076,13 +2569,14 @@ subst_constants (loc, insn, map)
 /* Show that register modified no longer contain known constants.  We are
    called from note_stores with parts of the new insn.  */
 
-void
-mark_stores (dest, x)
+static void
+mark_stores (dest, x, data)
      rtx dest;
      rtx x ATTRIBUTE_UNUSED;
+     void *data ATTRIBUTE_UNUSED;
 {
   int regno = -1;
-  enum machine_mode mode;
+  enum machine_mode mode = VOIDmode;
 
   /* DEST is always the innermost thing set, except in the case of
      SUBREGs of hard registers.  */
@@ -3097,102 +2591,21 @@ mark_stores (dest, x)
 
   if (regno >= 0)
     {
-      int last_reg = (regno >= FIRST_PSEUDO_REGISTER ? regno
-                     : regno + HARD_REGNO_NREGS (regno, mode) - 1);
-      int i;
+      unsigned int uregno = regno;
+      unsigned int last_reg = (uregno >= FIRST_PSEUDO_REGISTER ? uregno
+                             : uregno + HARD_REGNO_NREGS (uregno, mode) - 1);
+      unsigned int i;
 
       /* Ignore virtual stack var or virtual arg register since those
         are handled separately.  */
-      if (regno != VIRTUAL_INCOMING_ARGS_REGNUM
-         && regno != VIRTUAL_STACK_VARS_REGNUM)
-       for (i = regno; i <= last_reg; i++)
-         if (i < VARRAY_SIZE (global_const_equiv_varray))
+      if (uregno != VIRTUAL_INCOMING_ARGS_REGNUM
+         && uregno != VIRTUAL_STACK_VARS_REGNUM)
+       for (i = uregno; i <= last_reg; i++)
+         if ((size_t) i < VARRAY_SIZE (global_const_equiv_varray))
            VARRAY_CONST_EQUIV (global_const_equiv_varray, i).rtx = 0;
     }
 }
 \f
-/* If any CONST expressions with RTX_INTEGRATED_P are present in the rtx
-   pointed to by PX, they represent constants in the constant pool.
-   Replace these with a new memory reference obtained from force_const_mem.
-   Similarly, ADDRESS expressions with RTX_INTEGRATED_P represent the
-   address of a constant pool entry.  Replace them with the address of
-   a new constant pool entry obtained from force_const_mem.  */
-
-static void
-restore_constants (px)
-     rtx *px;
-{
-  rtx x = *px;
-  int i, j;
-  char *fmt;
-
-  if (x == 0)
-    return;
-
-  if (GET_CODE (x) == CONST_DOUBLE)
-    {
-      /* We have to make a new CONST_DOUBLE to ensure that we account for
-        it correctly.  Using the old CONST_DOUBLE_MEM data is wrong.  */
-      if (GET_MODE_CLASS (GET_MODE (x)) == MODE_FLOAT)
-       {
-         REAL_VALUE_TYPE d;
-
-         REAL_VALUE_FROM_CONST_DOUBLE (d, x);
-         *px = CONST_DOUBLE_FROM_REAL_VALUE (d, GET_MODE (x));
-       }
-      else
-       *px = immed_double_const (CONST_DOUBLE_LOW (x), CONST_DOUBLE_HIGH (x),
-                                 VOIDmode);
-    }
-
-  else if (RTX_INTEGRATED_P (x) && GET_CODE (x) == CONST)
-    {
-      restore_constants (&XEXP (x, 0));
-      *px = validize_mem (force_const_mem (GET_MODE (x), XEXP (x, 0)));
-    }
-  else if (RTX_INTEGRATED_P (x) && GET_CODE (x) == SUBREG)
-    {
-      /* This must be (subreg/i:M1 (const/i:M2 ...) 0).  */
-      rtx new = XEXP (SUBREG_REG (x), 0);
-
-      restore_constants (&new);
-      new = force_const_mem (GET_MODE (SUBREG_REG (x)), new);
-      PUT_MODE (new, GET_MODE (x));
-      *px = validize_mem (new);
-    }
-  else if (RTX_INTEGRATED_P (x) && GET_CODE (x) == ADDRESS)
-    {
-      rtx new = XEXP (force_const_mem (GET_MODE (XEXP (x, 0)),
-                                      XEXP (XEXP (x, 0), 0)),
-                     0);
-
-#ifdef POINTERS_EXTEND_UNSIGNED
-      if (GET_MODE (new) != GET_MODE (x))
-       new = convert_memory_address (GET_MODE (x), new);
-#endif
-
-      *px = new;
-    }
-  else
-    {
-      fmt = GET_RTX_FORMAT (GET_CODE (x));
-      for (i = 0; i < GET_RTX_LENGTH (GET_CODE (x)); i++)
-       {
-         switch (*fmt++)
-           {
-           case 'E':
-             for (j = 0; j < XVECLEN (x, i); j++)
-               restore_constants (&XVECEXP (x, i, j));
-             break;
-
-           case 'e':
-             restore_constants (&XEXP (x, i));
-             break;
-           }
-       }
-    }
-}
-\f
 /* Given a pointer to some BLOCK node, if the BLOCK_ABSTRACT_ORIGIN for the
    given BLOCK node is NULL, set the BLOCK_ABSTRACT_ORIGIN for the node so
    that it points to the node itself, thus indicating that the node is its
@@ -3242,7 +2655,7 @@ set_block_origin_self (stmt)
    set *their* DECL_ABSTRACT_ORIGIN or BLOCK_ABSTRACT_ORIGIN values to
    point to themselves.  */
 
-static void
+void
 set_decl_origin_self (decl)
      register tree decl;
 {
@@ -3320,50 +2733,29 @@ void
 output_inline_function (fndecl)
      tree fndecl;
 {
+  struct function *old_cfun = cfun;
   struct function *f = DECL_SAVED_INSNS (fndecl);
-  rtx last;
 
-  /* Things we allocate from here on are part of this function, not
-     permanent.  */
-  temporary_allocation ();
-  current_function = f;
+  cfun = f;
   current_function_decl = fndecl;
   clear_emit_caches ();
 
-  /* Find last insn and rebuild the constant pool.  */
-  init_const_rtx_hash_table ();
-  for (last = get_insns (); NEXT_INSN (last); last = NEXT_INSN (last))
-    {
-      if (GET_RTX_CLASS (GET_CODE (last)) == 'i')
-       {
-         restore_constants (&PATTERN (last));
-         restore_constants (&REG_NOTES (last));
-       }
-    }
+  /* Things we allocate from here on are part of this function, not
+     permanent.  */
+  temporary_allocation ();
 
   set_new_last_label_num (f->inl_max_label_num);
 
-  /* We must have already output DWARF debugging information for the
-     original (abstract) inline function declaration/definition, so
-     we want to make sure that the debugging information we generate
-     for this special instance of the inline function refers back to
-     the information we already generated.  To make sure that happens,
-     we simply have to set the DECL_ABSTRACT_ORIGIN for the function
-     node (and for all of the local ..._DECL nodes which are its children)
-     so that they all point to themselves.  */
-
-  set_decl_origin_self (fndecl);
-
   /* We're not deferring this any longer.  */
   DECL_DEFER_OUTPUT (fndecl) = 0;
 
+  /* Compile this function all the way down to assembly code.  */
+  rest_of_compilation (fndecl);
+
   /* We can't inline this anymore.  */
   f->inlinable = 0;
   DECL_INLINE (fndecl) = 0;
 
-  /* Compile this function all the way down to assembly code.  */
-  rest_of_compilation (fndecl);
-
-  current_function = 0;
-  current_function_decl = 0;
+  cfun = old_cfun;
+  current_function_decl = old_cfun ? old_cfun->decl : 0;
 }