This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

[PATCH] Kill more of pop_stmt_changes


As it happens to work right now and nothing seems to depend on it this
removes automatic renaming of all bare decls from pop_stmt_changes.

Bootstrapped and tested on x86_64-unknown-linux-gnu, applied to trunk.

Richard.

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

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

Index: gcc/tree-ssa-operands.c
===================================================================
*** gcc/tree-ssa-operands.c	(revision 145539)
--- gcc/tree-ssa-operands.c	(working copy)
*************** void
*** 1380,1387 ****
  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)
--- 1380,1385 ----
*************** pop_stmt_changes (gimple *stmt_p)
*** 1394,1408 ****
       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
--- 1392,1397 ----


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]