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] PR/28218, thinko in tree-ssa-math-opts.c


I don't know where I got the idea that you can do

calculate_dominance_info (CDI_DOMINATORS | CDI_POST_DOMINATORS);

instead of

      calculate_dominance_info (CDI_DOMINATORS);
      calculate_dominance_info (CDI_POST_DOMINATORS);

Will commit as obvious tomorrow.

Paolo

2006-07-04 Paolo Bonzini <bonzini@gnu.org>

      * tree-ssa-math-opts.c (execute_cse_reciprocals): Fix calls
      to calculate and free the dominator information.

Index: ../../gcc/tree-ssa-math-opts.c
===================================================================
--- ../../gcc/tree-ssa-math-opts.c      (revision 115179)
+++ ../../gcc/tree-ssa-math-opts.c      (working copy)
@@ -486,7 +486,8 @@ execute_cse_reciprocals (void)
                               sizeof (struct occurrence),
                               n_basic_blocks / 3 + 1);

-  calculate_dominance_info (CDI_DOMINATORS | CDI_POST_DOMINATORS);
+  calculate_dominance_info (CDI_DOMINATORS);
+  calculate_dominance_info (CDI_POST_DOMINATORS);

#ifdef ENABLE_CHECKING
  FOR_EACH_BB (bb)
@@ -523,7 +524,8 @@ execute_cse_reciprocals (void)
       }
    }

-  free_dominance_info (CDI_DOMINATORS | CDI_POST_DOMINATORS);
+  free_dominance_info (CDI_DOMINATORS);
+  free_dominance_info (CDI_POST_DOMINATORS);
  free_alloc_pool (occ_pool);
}



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