2011-07-12 Richard Guenther <rguenther@suse.de>

	* tree-ssa-copyrename.c (rename_ssa_copies): Zero statistics.
	Do not perform no-op changes.

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

diff --git a/gcc/ChangeLog b/gcc/ChangeLog
index 792508d..d519a80 100644 (file)
--- a/gcc/ChangeLog
+++ b/gcc/ChangeLog
@@ -1,3 +1,8 @@
+2011-07-12  Richard Guenther  <rguenther@suse.de>
+
+       * tree-ssa-copyrename.c (rename_ssa_copies): Zero statistics.
+       Do not perform no-op changes.
+
 2011-07-12  Richard Sandiford  <richard.sandiford@linaro.org>
 
        * config/arm/predicates.md (neon_struct_operand): Make a normal

diff --git a/gcc/tree-ssa-copyrename.c b/gcc/tree-ssa-copyrename.c
index 88a156b..3e01b6e 100644 (file)
--- a/gcc/tree-ssa-copyrename.c
+++ b/gcc/tree-ssa-copyrename.c
@@ -296,6 +296,8 @@ rename_ssa_copies (void)
   FILE *debug;
   bool updated = false;
 
+  memset (&stats, 0, sizeof (stats));
+
   if (dump_file && (dump_flags & TDF_DETAILS))
     debug = dump_file;
   else
@@ -355,16 +357,15 @@ rename_ssa_copies (void)
       if (!part_var)
         continue;
       var = ssa_name (x);
+      if (SSA_NAME_VAR (var) == SSA_NAME_VAR (part_var))
+       continue;
       if (debug)
         {
-         if (SSA_NAME_VAR (var) != SSA_NAME_VAR (part_var))
-           {
-             fprintf (debug, "Coalesced ");
-             print_generic_expr (debug, var, TDF_SLIM);
-             fprintf (debug, " to ");
-             print_generic_expr (debug, part_var, TDF_SLIM);
-             fprintf (debug, "\n");
-           }
+         fprintf (debug, "Coalesced ");
+         print_generic_expr (debug, var, TDF_SLIM);
+         fprintf (debug, " to ");
+         print_generic_expr (debug, part_var, TDF_SLIM);
+         fprintf (debug, "\n");
        }
       stats.coalesced++;
       replace_ssa_name_symbol (var, SSA_NAME_VAR (part_var));