OSDN Git Service

* config/m32r/m32r.h (LEGITIMATE_CONSTANT_P): Treat LABEL_REFs in
[pf3gnuchains/gcc-fork.git] / gcc / tree-dfa.c
index 0a8f88d..df0be2d 100644 (file)
@@ -1,5 +1,6 @@
 /* Data flow functions for trees.
-   Copyright (C) 2001, 2002, 2003, 2004, 2005, 2007 Free Software Foundation, Inc.
+   Copyright (C) 2001, 2002, 2003, 2004, 2005, 2007, 2008 Free Software
+   Foundation, Inc.
    Contributed by Diego Novillo <dnovillo@redhat.com>
 
 This file is part of GCC.
@@ -38,7 +39,7 @@ along with GCC; see the file COPYING3.  If not see
 #include "function.h"
 #include "diagnostic.h"
 #include "tree-dump.h"
-#include "tree-gimple.h"
+#include "gimple.h"
 #include "tree-flow.h"
 #include "tree-inline.h"
 #include "tree-pass.h"
@@ -51,13 +52,12 @@ along with GCC; see the file COPYING3.  If not see
 /* Counters used to display DFA and SSA statistics.  */
 struct dfa_stats_d
 {
-  long num_stmt_anns;
   long num_var_anns;
   long num_defs;
   long num_uses;
   long num_phis;
   long num_phi_args;
-  int max_num_phi_args;
+  size_t max_num_phi_args;
   long num_vdefs;
   long num_vuses;
 };
@@ -65,7 +65,6 @@ struct dfa_stats_d
 
 /* Local functions.  */
 static void collect_dfa_stats (struct dfa_stats_d *);
-static tree collect_dfa_stats_r (tree *, int *, void *);
 static tree find_vars_r (tree *, int *, void *);
 
 
@@ -84,27 +83,28 @@ static unsigned int
 find_referenced_vars (void)
 {
   basic_block bb;
-  block_stmt_iterator si;
-  tree phi;
+  gimple_stmt_iterator si;
 
   FOR_EACH_BB (bb)
     {
-      for (si = bsi_start (bb); !bsi_end_p (si); bsi_next (&si))
+      for (si = gsi_start_bb (bb); !gsi_end_p (si); gsi_next (&si))
        {
-         tree *stmt_p = bsi_stmt_ptr (si);
-         walk_tree (stmt_p, find_vars_r, NULL, NULL);
+         size_t i;
+         gimple stmt = gsi_stmt (si);
+         for (i = 0; i < gimple_num_ops (stmt); i++)
+           walk_tree (gimple_op_ptr (stmt, i), find_vars_r, NULL, NULL);
        }
 
-      for (phi = phi_nodes (bb); phi; phi = PHI_CHAIN (phi))
+      for (si = gsi_start_phis (bb); !gsi_end_p (si); gsi_next (&si))
        {
-         int len = PHI_NUM_ARGS (phi);
-         int i;
+         gimple phi = gsi_stmt (si);
+         size_t i, len = gimple_phi_num_args (phi);
 
-         walk_tree (&phi, find_vars_r, NULL, NULL);
+         walk_tree (gimple_phi_result_ptr (phi), find_vars_r, NULL, NULL);
 
          for (i = 0; i < len; i++)
            {
-             tree arg = PHI_ARG_DEF (phi, i);
+             tree arg = gimple_phi_arg_def (phi, i);
              walk_tree (&arg, find_vars_r, NULL, NULL);
            }
        }
@@ -142,33 +142,14 @@ var_ann_t
 create_var_ann (tree t)
 {
   var_ann_t ann;
-  struct static_var_ann_d *sann = NULL;
 
   gcc_assert (t);
   gcc_assert (DECL_P (t));
   gcc_assert (!t->base.ann || t->base.ann->common.type == VAR_ANN);
 
-  if (!MTAG_P (t) && (TREE_STATIC (t) || DECL_EXTERNAL (t)))
-    {
-      sann = GGC_CNEW (struct static_var_ann_d);
-      ann = &sann->ann;
-    }
-  else
-    ann = GGC_CNEW (struct var_ann_d);
-
+  ann = GGC_CNEW (struct var_ann_d);
   ann->common.type = VAR_ANN;
-
-  if (!MTAG_P (t) && (TREE_STATIC (t) || DECL_EXTERNAL (t)))
-    {
-       void **slot;
-       sann->uid = DECL_UID (t);
-       slot = htab_find_slot_with_hash (gimple_var_anns (cfun),
-                                       t, DECL_UID (t), INSERT);
-       gcc_assert (!*slot);
-       *slot = sann;
-    }
-  else
-    t->base.ann = (tree_ann_t) ann;
+  t->base.ann = (tree_ann_t) ann;
 
   return ann;
 }
@@ -184,7 +165,7 @@ create_function_ann (tree t)
   gcc_assert (TREE_CODE (t) == FUNCTION_DECL);
   gcc_assert (!t->base.ann || t->base.ann->common.type == FUNCTION_ANN);
 
-  ann = ggc_alloc (sizeof (*ann));
+  ann = (function_ann_t) ggc_alloc (sizeof (*ann));
   memset ((void *) ann, 0, sizeof (*ann));
 
   ann->common.type = FUNCTION_ANN;
@@ -194,29 +175,6 @@ create_function_ann (tree t)
   return ann;
 }
 
-/* Create a new annotation for a statement node T.  */
-
-stmt_ann_t
-create_stmt_ann (tree t)
-{
-  stmt_ann_t ann;
-
-  gcc_assert (is_gimple_stmt (t));
-  gcc_assert (!t->base.ann || t->base.ann->common.type == STMT_ANN);
-
-  ann = GGC_CNEW (struct stmt_ann_d);
-
-  ann->common.type = STMT_ANN;
-
-  /* Since we just created the annotation, mark the statement modified.  */
-  ann->modified = true;
-
-  ann->uid = inc_gimple_stmt_max_uid (cfun);
-  t->base.ann = (tree_ann_t) ann;
-
-  return ann;
-}
-
 /* Renumber all of the gimple stmt uids.  */
 
 void 
@@ -227,17 +185,11 @@ renumber_gimple_stmt_uids (void)
   set_gimple_stmt_max_uid (cfun, 0);
   FOR_ALL_BB (bb)
     {
-      block_stmt_iterator bsi;
-      for (bsi = bsi_start (bb); !bsi_end_p (bsi); bsi_next (&bsi))
+      gimple_stmt_iterator bsi;
+      for (bsi = gsi_start_bb (bb); !gsi_end_p (bsi); gsi_next (&bsi))
        {
-         tree stmt = bsi_stmt (bsi);
-         /* If the stmt has an annotation, then overwrite it, if not,
-            the process of getting it will set the number
-            properly.  */
-         if (has_stmt_ann (stmt))
-           set_gimple_stmt_uid (stmt, inc_gimple_stmt_max_uid (cfun));
-         else
-           get_stmt_ann (stmt);
+         gimple stmt = gsi_stmt (bsi);
+         gimple_set_uid (stmt, inc_gimple_stmt_max_uid (cfun));
        }
     }
 }
@@ -255,6 +207,7 @@ create_tree_common_ann (tree t)
   ann = GGC_CNEW (struct tree_ann_common_d);
 
   ann->type = TREE_ANN_COMMON;
+  ann->rn = -1;
   t->base.ann = (tree_ann_t) ann;
 
   return ann;
@@ -466,11 +419,6 @@ dump_dfa_stats (FILE *file)
   fprintf (file, fmt_str_1, "Referenced variables", (unsigned long)num_referenced_vars,
           SCALE (size), LABEL (size));
 
-  size = dfa_stats.num_stmt_anns * sizeof (struct stmt_ann_d);
-  total += size;
-  fprintf (file, fmt_str_1, "Statements annotated", dfa_stats.num_stmt_anns,
-          SCALE (size), LABEL (size));
-
   size = dfa_stats.num_var_anns * sizeof (struct var_ann_d);
   total += size;
   fprintf (file, fmt_str_1, "Variables annotated", dfa_stats.num_var_anns,
@@ -496,7 +444,7 @@ dump_dfa_stats (FILE *file)
   fprintf (file, fmt_str_1, "VDEF operands", dfa_stats.num_vdefs,
           SCALE (size), LABEL (size));
 
-  size = dfa_stats.num_phis * sizeof (struct tree_phi_node);
+  size = dfa_stats.num_phis * sizeof (struct gimple_statement_phi);
   total += size;
   fprintf (file, fmt_str_1, "PHI nodes", dfa_stats.num_phis,
           SCALE (size), LABEL (size));
@@ -513,9 +461,9 @@ dump_dfa_stats (FILE *file)
   fprintf (file, "\n");
 
   if (dfa_stats.num_phis)
-    fprintf (file, "Average number of arguments per PHI node: %.1f (max: %d)\n",
+    fprintf (file, "Average number of arguments per PHI node: %.1f (max: %ld)\n",
             (float) dfa_stats.num_phi_args / (float) dfa_stats.num_phis,
-            dfa_stats.max_num_phi_args);
+            (long) dfa_stats.max_num_phi_args);
 
   fprintf (file, "\n");
 }
@@ -534,75 +482,44 @@ debug_dfa_stats (void)
    DFA_STATS_P.  */
 
 static void
-collect_dfa_stats (struct dfa_stats_d *dfa_stats_p)
+collect_dfa_stats (struct dfa_stats_d *dfa_stats_p ATTRIBUTE_UNUSED)
 {
-  struct pointer_set_t *pset;
   basic_block bb;
-  block_stmt_iterator i;
+  referenced_var_iterator vi;
+  tree var;
 
   gcc_assert (dfa_stats_p);
 
   memset ((void *)dfa_stats_p, 0, sizeof (struct dfa_stats_d));
 
-  /* Walk all the trees in the function counting references.  Start at
-     basic block NUM_FIXED_BLOCKS, but don't stop at block boundaries.  */
-  pset = pointer_set_create ();
-
-  for (i = bsi_start (BASIC_BLOCK (NUM_FIXED_BLOCKS));
-       !bsi_end_p (i); bsi_next (&i))
-    walk_tree (bsi_stmt_ptr (i), collect_dfa_stats_r, (void *) dfa_stats_p,
-              pset);
-
-  pointer_set_destroy (pset);
+  /* Count all the variable annotations.  */
+  FOR_EACH_REFERENCED_VAR (var, vi)
+    if (var_ann (var))
+      dfa_stats_p->num_var_anns++;
 
+  /* Walk all the statements in the function counting references.  */
   FOR_EACH_BB (bb)
     {
-      tree phi;
-      for (phi = phi_nodes (bb); phi; phi = PHI_CHAIN (phi))
+      gimple_stmt_iterator si;
+
+      for (si = gsi_start_phis (bb); !gsi_end_p (si); gsi_next (&si))
        {
+         gimple phi = gsi_stmt (si);
          dfa_stats_p->num_phis++;
-         dfa_stats_p->num_phi_args += PHI_NUM_ARGS (phi);
-         if (PHI_NUM_ARGS (phi) > dfa_stats_p->max_num_phi_args)
-           dfa_stats_p->max_num_phi_args = PHI_NUM_ARGS (phi);
+         dfa_stats_p->num_phi_args += gimple_phi_num_args (phi);
+         if (gimple_phi_num_args (phi) > dfa_stats_p->max_num_phi_args)
+           dfa_stats_p->max_num_phi_args = gimple_phi_num_args (phi);
        }
-    }
-}
-
-
-/* Callback for walk_tree to collect DFA statistics for a tree and its
-   children.  */
-
-static tree
-collect_dfa_stats_r (tree *tp, int *walk_subtrees ATTRIBUTE_UNUSED,
-                    void *data)
-{
-  tree t = *tp;
-  struct dfa_stats_d *dfa_stats_p = (struct dfa_stats_d *)data;
 
-  if (t->base.ann)
-    {
-      switch (ann_type (t->base.ann))
+      for (si = gsi_start_bb (bb); !gsi_end_p (si); gsi_next (&si))
        {
-       case STMT_ANN:
-         {
-           dfa_stats_p->num_stmt_anns++;
-           dfa_stats_p->num_defs += NUM_SSA_OPERANDS (t, SSA_OP_DEF);
-           dfa_stats_p->num_uses += NUM_SSA_OPERANDS (t, SSA_OP_USE);
-           dfa_stats_p->num_vdefs += NUM_SSA_OPERANDS (t, SSA_OP_VDEF);
-           dfa_stats_p->num_vuses += NUM_SSA_OPERANDS (t, SSA_OP_VUSE);
-           break;
-         }
-
-       case VAR_ANN:
-         dfa_stats_p->num_var_anns++;
-         break;
-
-       default:
-         break;
+         gimple stmt = gsi_stmt (si);
+         dfa_stats_p->num_defs += NUM_SSA_OPERANDS (stmt, SSA_OP_DEF);
+         dfa_stats_p->num_uses += NUM_SSA_OPERANDS (stmt, SSA_OP_USE);
+         dfa_stats_p->num_vdefs += NUM_SSA_OPERANDS (stmt, SSA_OP_VDEF);
+         dfa_stats_p->num_vuses += NUM_SSA_OPERANDS (stmt, SSA_OP_VUSE);
        }
     }
-
-  return NULL;
 }
 
 
@@ -743,7 +660,7 @@ add_referenced_var (tree var)
       /* Scan DECL_INITIAL for pointer variables as they may contain
         address arithmetic referencing the address of other
         variables.  
-        Even non-constant intializers need to be walked, because
+        Even non-constant initializers need to be walked, because
         IPA passes might prove that their are invariant later on.  */
       if (DECL_INITIAL (var)
          /* Initializers of external variables are not useful to the
@@ -764,9 +681,22 @@ remove_referenced_var (tree var)
   unsigned int uid = DECL_UID (var);
 
   clear_call_clobbered (var);
+  bitmap_clear_bit (gimple_call_used_vars (cfun), uid);
   if ((v_ann = var_ann (var)))
-    ggc_free (v_ann);
-  var->base.ann = NULL;
+    {
+      /* Preserve var_anns of globals, but clear their alias info.  */
+      if (MTAG_P (var)
+         || (!TREE_STATIC (var) && !DECL_EXTERNAL (var)))
+       {
+         ggc_free (v_ann);
+         var->base.ann = NULL;
+       }
+      else
+       {
+         v_ann->mpt = NULL_TREE;
+         v_ann->symbol_mem_tag = NULL_TREE;
+       }
+    }
   gcc_assert (DECL_P (var));
   in.uid = uid;
   loc = htab_find_slot_with_hash (gimple_referenced_vars (cfun), &in, uid,
@@ -805,7 +735,7 @@ get_virtual_var (tree var)
    combination push_stmt_changes/pop_stmt_changes.  */
 
 void
-mark_symbols_for_renaming (tree stmt)
+mark_symbols_for_renaming (gimple stmt)
 {
   tree op;
   ssa_op_iter iter;
@@ -819,8 +749,9 @@ mark_symbols_for_renaming (tree stmt)
 }
 
 
-/* Find all variables within the gimplified statement that were not previously
-   visible to the function and add them to the referenced variables list.  */
+/* Find all variables within the gimplified statement that were not
+   previously visible to the function and add them to the referenced
+   variables list.  */
 
 static tree
 find_new_referenced_vars_1 (tree *tp, int *walk_subtrees,
@@ -840,10 +771,13 @@ find_new_referenced_vars_1 (tree *tp, int *walk_subtrees,
   return NULL;
 }
 
+
+/* Find any new referenced variables in STMT.  */
+
 void
-find_new_referenced_vars (tree *stmt_p)
+find_new_referenced_vars (gimple stmt)
 {
-  walk_tree (stmt_p, find_new_referenced_vars_1, NULL, NULL);
+  walk_gimple_op (stmt, find_new_referenced_vars_1, NULL);
 }
 
 
@@ -918,7 +852,7 @@ get_ref_base_and_extent (tree exp, HOST_WIDE_INT *poffset,
              {
                tree csize = TYPE_SIZE (TREE_TYPE (TREE_OPERAND (exp, 0)));
                /* We need to adjust maxsize to the whole structure bitsize.
-                  But we can subtract any constant offset seen sofar,
+                  But we can subtract any constant offset seen so far,
                   because that would get us out of the structure otherwise.  */
                if (maxsize != -1 && csize && host_integerp (csize, 1))
                  maxsize = TREE_INT_CST_LOW (csize) - bit_offset;
@@ -956,7 +890,7 @@ get_ref_base_and_extent (tree exp, HOST_WIDE_INT *poffset,
              {
                tree asize = TYPE_SIZE (TREE_TYPE (TREE_OPERAND (exp, 0)));
                /* We need to adjust maxsize to the whole array bitsize.
-                  But we can subtract any constant offset seen sofar,
+                  But we can subtract any constant offset seen so far,
                   because that would get us outside of the array otherwise.  */
                if (maxsize != -1 && asize && host_integerp (asize, 1))
                  maxsize = TREE_INT_CST_LOW (asize) - bit_offset;
@@ -1018,7 +952,7 @@ get_ref_base_and_extent (tree exp, HOST_WIDE_INT *poffset,
    SSA_NAME_OCCURS_IN_ABNORMAL_PHI set, otherwise false.  */
 
 bool
-stmt_references_abnormal_ssa_name (tree stmt)
+stmt_references_abnormal_ssa_name (gimple stmt)
 {
   ssa_op_iter oi;
   use_operand_p use_p;
@@ -1041,6 +975,7 @@ refs_may_alias_p (tree ref1, tree ref2)
   HOST_WIDE_INT offset1 = 0, offset2 = 0;
   HOST_WIDE_INT size1 = -1, size2 = -1;
   HOST_WIDE_INT max_size1 = -1, max_size2 = -1;
+  bool strict_aliasing_applies;
 
   gcc_assert ((SSA_VAR_P (ref1)
               || handled_component_p (ref1)
@@ -1066,22 +1001,96 @@ refs_may_alias_p (tree ref1, tree ref2)
 
   /* If both references are based on different variables, they cannot alias.
      If both references are based on the same variable, they cannot alias if
-     if the accesses do not overlap.  */
+     the accesses do not overlap.  */
   if (SSA_VAR_P (base1)
-      && SSA_VAR_P (base2)
-      && (!operand_equal_p (base1, base2, 0)
-         || !ranges_overlap_p (offset1, max_size1, offset2, max_size2)))
-    return false;
+      && SSA_VAR_P (base2))
+    {
+      if (!operand_equal_p (base1, base2, 0))
+       return false;
+      return ranges_overlap_p (offset1, max_size1, offset2, max_size2);
+    }
 
-  /* If both references are through pointers and both pointers are equal
-     then they do not alias if the accesses do not overlap.  */
-  if (TREE_CODE (base1) == INDIRECT_REF
-      && TREE_CODE (base2) == INDIRECT_REF
-      && operand_equal_p (TREE_OPERAND (base1, 0),
-                         TREE_OPERAND (base2, 0), 0)
-      && !ranges_overlap_p (offset1, max_size1, offset2, max_size2))
+  /* If one base is a ref-all pointer weird things are allowed.  */
+  strict_aliasing_applies = (flag_strict_aliasing
+                            && (!INDIRECT_REF_P (base1)
+                                || get_alias_set (base1) != 0)
+                            && (!INDIRECT_REF_P (base2)
+                                || get_alias_set (base2) != 0));
+
+  /* If strict aliasing applies the only way to access a scalar variable
+     is through a pointer dereference or through a union (gcc extension).  */
+  if (strict_aliasing_applies
+      && ((SSA_VAR_P (ref2)
+          && !AGGREGATE_TYPE_P (TREE_TYPE (ref2))
+          && !INDIRECT_REF_P (ref1)
+          && TREE_CODE (TREE_TYPE (base1)) != UNION_TYPE)
+         || (SSA_VAR_P (ref1)
+             && !AGGREGATE_TYPE_P (TREE_TYPE (ref1))
+             && !INDIRECT_REF_P (ref2)
+             && TREE_CODE (TREE_TYPE (base2)) != UNION_TYPE)))
     return false;
 
+  /* If both references are through the same type, or if strict aliasing
+     doesn't apply they are through two same pointers, they do not alias
+     if the accesses do not overlap.  */
+  if ((strict_aliasing_applies
+       && (TYPE_MAIN_VARIANT (TREE_TYPE (base1))
+          == TYPE_MAIN_VARIANT (TREE_TYPE (base2))))
+      || (TREE_CODE (base1) == INDIRECT_REF
+         && TREE_CODE (base2) == INDIRECT_REF
+         && operand_equal_p (TREE_OPERAND (base1, 0),
+                             TREE_OPERAND (base2, 0), 0)))
+    return ranges_overlap_p (offset1, max_size1, offset2, max_size2);
+
+  /* If both are component references through pointers try to find a
+     common base and apply offset based disambiguation.  This handles
+     for example
+       struct A { int i; int j; } *q;
+       struct B { struct A a; int k; } *p;
+     disambiguating q->i and p->a.j.  */
+  if (strict_aliasing_applies
+      && (TREE_CODE (base1) == INDIRECT_REF
+         || TREE_CODE (base2) == INDIRECT_REF)
+      && handled_component_p (ref1)
+      && handled_component_p (ref2))
+    {
+      tree *refp;
+      /* Now search for the type of base1 in the access path of ref2.  This
+        would be a common base for doing offset based disambiguation on.  */
+      refp = &ref2;
+      while (handled_component_p (*refp)
+            /* Note that the following is only conservative if there are
+               never copies of types appearing as sub-structures.  */
+            && (TYPE_MAIN_VARIANT (TREE_TYPE (*refp))
+                != TYPE_MAIN_VARIANT (TREE_TYPE (base1))))
+       refp = &TREE_OPERAND (*refp, 0);
+      if (TYPE_MAIN_VARIANT (TREE_TYPE (*refp))
+         == TYPE_MAIN_VARIANT (TREE_TYPE (base1)))
+       {
+         HOST_WIDE_INT offadj, sztmp, msztmp;
+         get_ref_base_and_extent (*refp, &offadj, &sztmp, &msztmp);
+         offset2 -= offadj;
+         return ranges_overlap_p (offset1, max_size1, offset2, max_size2);
+       }
+      /* The other way around.  */
+      refp = &ref1;
+      while (handled_component_p (*refp)
+            && (TYPE_MAIN_VARIANT (TREE_TYPE (*refp))
+                != TYPE_MAIN_VARIANT (TREE_TYPE (base2))))
+       refp = &TREE_OPERAND (*refp, 0);
+      if (TYPE_MAIN_VARIANT (TREE_TYPE (*refp))
+         == TYPE_MAIN_VARIANT (TREE_TYPE (base2)))
+       {
+         HOST_WIDE_INT offadj, sztmp, msztmp;
+         get_ref_base_and_extent (*refp, &offadj, &sztmp, &msztmp);
+         offset1 -= offadj;
+         return ranges_overlap_p (offset1, max_size1, offset2, max_size2);
+       }
+      /* If we can be sure to catch all equivalent types in the search
+        for the common base then we could return false here.  In that
+        case we would be able to disambiguate q->i and p->k.  */
+    }
+
   return true;
 }
 
@@ -1093,25 +1102,25 @@ refs_may_alias_p (tree ref1, tree ref2)
    a PHI node as well.  Note that if all VUSEs are default definitions
    this function will return an empty statement.  */
 
-tree
-get_single_def_stmt (tree stmt)
+gimple
+get_single_def_stmt (gimple stmt)
 {
-  tree def_stmt = NULL_TREE;
+  gimple def_stmt = NULL;
   tree use;
   ssa_op_iter iter;
 
   FOR_EACH_SSA_TREE_OPERAND (use, stmt, iter, SSA_OP_VIRTUAL_USES)
     {
-      tree tmp = SSA_NAME_DEF_STMT (use);
+      gimple tmp = SSA_NAME_DEF_STMT (use);
 
       /* ???  This is too simplistic for multiple virtual operands
         reaching different PHI nodes of the same basic blocks or for
         reaching all default definitions.  */
       if (def_stmt
          && def_stmt != tmp
-         && !(IS_EMPTY_STMT (def_stmt)
-              && IS_EMPTY_STMT (tmp)))
-       return NULL_TREE;
+         && !(gimple_nop_p (def_stmt)
+              && gimple_nop_p (tmp)))
+       return NULL;
 
       def_stmt = tmp;
     }
@@ -1125,25 +1134,25 @@ get_single_def_stmt (tree stmt)
    from a non-backedge.  Returns NULL_TREE if such statement within
    the above conditions cannot be found.  */
 
-tree
-get_single_def_stmt_from_phi (tree ref, tree phi)
+gimple
+get_single_def_stmt_from_phi (tree ref, gimple phi)
 {
   tree def_arg = NULL_TREE;
-  int i;
+  unsigned i;
 
   /* Find the single PHI argument that is not flowing in from a
      back edge and verify that the loop-carried definitions do
      not alias the reference we look for.  */
-  for (i = 0; i < PHI_NUM_ARGS (phi); ++i)
+  for (i = 0; i < gimple_phi_num_args (phi); ++i)
     {
       tree arg = PHI_ARG_DEF (phi, i);
-      tree def_stmt;
+      gimple def_stmt;
 
-      if (!(PHI_ARG_EDGE (phi, i)->flags & EDGE_DFS_BACK))
+      if (!(gimple_phi_arg_edge (phi, i)->flags & EDGE_DFS_BACK))
        {
          /* Multiple non-back edges?  Do not try to handle this.  */
          if (def_arg)
-           return NULL_TREE;
+           return NULL;
          def_arg = arg;
          continue;
        }
@@ -1153,14 +1162,14 @@ get_single_def_stmt_from_phi (tree ref, tree phi)
       def_stmt = SSA_NAME_DEF_STMT (arg);
       do
        {
-         if (TREE_CODE (def_stmt) != GIMPLE_MODIFY_STMT
-             || refs_may_alias_p (ref, GIMPLE_STMT_OPERAND (def_stmt, 0)))
-           return NULL_TREE;
+         if (!is_gimple_assign (def_stmt)
+             || refs_may_alias_p (ref, gimple_assign_lhs (def_stmt)))
+           return NULL;
          /* ???  This will only work, reaching the PHI node again if
             there is a single virtual operand on def_stmt.  */
          def_stmt = get_single_def_stmt (def_stmt);
          if (!def_stmt)
-           return NULL_TREE;
+           return NULL;
        }
       while (def_stmt != phi);
     }
@@ -1173,8 +1182,8 @@ get_single_def_stmt_from_phi (tree ref, tree phi)
    Take into account only definitions that alias REF if following
    back-edges when looking through a loop PHI node.  */
 
-tree
-get_single_def_stmt_with_phi (tree ref, tree stmt)
+gimple
+get_single_def_stmt_with_phi (tree ref, gimple stmt)
 {
   switch (NUM_SSA_OPERANDS (stmt, SSA_OP_VIRTUAL_USES))
     {
@@ -1183,11 +1192,11 @@ get_single_def_stmt_with_phi (tree ref, tree stmt)
 
     case 1:
       {
-       tree def_stmt = SSA_NAME_DEF_STMT (SINGLE_SSA_TREE_OPERAND
+       gimple def_stmt = SSA_NAME_DEF_STMT (SINGLE_SSA_TREE_OPERAND
                                             (stmt, SSA_OP_VIRTUAL_USES));
        /* We can handle lookups over PHI nodes only for a single
           virtual operand.  */
-       if (TREE_CODE (def_stmt) == PHI_NODE)
+       if (gimple_code (def_stmt) == GIMPLE_PHI)
          return get_single_def_stmt_from_phi (ref, def_stmt);
        return def_stmt;
       }