This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
[patch] tree-ssa-loop-ivopts.c: Remove a redundant call toupdate_stmt.
- From: Kazu Hirata <kazu at cs dot umass dot edu>
- To: gcc-patches at gcc dot gnu dot org
- Cc: dvorakz at suse dot cz
- Date: Wed, 04 May 2005 01:15:42 -0400 (EDT)
- Subject: [patch] tree-ssa-loop-ivopts.c: Remove a redundant call toupdate_stmt.
Hi,
Attached is a patch to remove a redundant call to update_stmt.
COPY shown in the patch is a newly created statement. As such, it
does not have a statement annotation yet. When we insert the new
statement with bsi_insert_before, it will eventually call
update_stmt_operands, so we don't need to explicitly call update_stmt
here.
Tested on i686-pc-linux-gnu. OK to apply?
Kazu Hirata
2005-05-04 Kazu Hirata <kazu@cs.umass.edu>
* tree-ssa-loop-ivopts.c (rewrite_address_base): Don't call
update_stmt.
Index: tree-ssa-loop-ivopts.c
===================================================================
RCS file: /cvs/gcc/gcc/gcc/tree-ssa-loop-ivopts.c,v
retrieving revision 2.67
diff -u -d -p -r2.67 tree-ssa-loop-ivopts.c
--- tree-ssa-loop-ivopts.c 1 May 2005 15:30:05 -0000 2.67
+++ tree-ssa-loop-ivopts.c 3 May 2005 23:05:52 -0000
@@ -5439,7 +5439,6 @@ rewrite_address_base (block_stmt_iterato
}
TREE_OPERAND (copy, 0) = new_name;
- update_stmt (copy);
bsi_insert_before (bsi, copy, BSI_SAME_STMT);
with = new_name;