2009-04-04 Richard Guenther <rguenther@suse.de>

	* tree-ssa-operands.c (pop_stmt_changes): Remove automatic
	renaming code.

git-svn-id: svn+ssh://gcc.gnu.org/svn/gcc/trunk@145547 138bc75d-0d04-0410-961f-82ee72b054a4

diff --git a/gcc/ChangeLog b/gcc/ChangeLog
index bf203d9..6723da5 100644 (file)
--- a/gcc/ChangeLog
+++ b/gcc/ChangeLog
@@ -1,3 +1,8 @@
+2009-04-04  Richard Guenther  <rguenther@suse.de>
+
+       * tree-ssa-operands.c (pop_stmt_changes): Remove automatic
+       renaming code.
+
 2009-04-04  Jan Hubicka  <jh@suse.cz>
 
        * tree-ssa-uncprop.c (associate_equivalences_with_edges): Use

diff --git a/gcc/tree-ssa-operands.c b/gcc/tree-ssa-operands.c
index f4df750..c9473a8 100644 (file)
--- a/gcc/tree-ssa-operands.c
+++ b/gcc/tree-ssa-operands.c
@@ -1380,8 +1380,6 @@ void
 pop_stmt_changes (gimple *stmt_p)
 {
   gimple *stmt2_p, stmt = *stmt_p;
-  ssa_op_iter iter;
-  tree op;
 
   /* It makes no sense to keep track of PHI nodes.  */
   if (gimple_code (stmt) == GIMPLE_PHI)
@@ -1394,15 +1392,6 @@ pop_stmt_changes (gimple *stmt_p)
      exposed variables.  This also will mark the virtual operand
      for renaming if necessary.  */
   update_stmt (stmt);
-
-  /* Mark all the naked GIMPLE register operands for renaming.
-     ???  Especially this is considered bad behavior of the caller,
-     it should have updated SSA form manually.  Even more so as
-     we do not have a way to verify that no SSA names for op are
-     already in use.  */
-  FOR_EACH_SSA_TREE_OPERAND (op, stmt, iter, SSA_OP_DEF|SSA_OP_USE)
-    if (DECL_P (op))
-      mark_sym_for_renaming (op);
 }
 
 /* Discard the topmost stmt from SCB_STACK.  This is useful