From 9fc49e8fb8866f8b2129bdeeec18e3cd2afa7397 Mon Sep 17 00:00:00 2001 From: spop Date: Tue, 6 Apr 2010 19:20:47 +0000 Subject: [PATCH] Avoid calling verify_ssa twice in verify_loop_closed_ssa. 2010-04-06 Sebastian Pop * graphite-clast-to-gimple.c (graphite_verify): Remove redundant call to verify_ssa. Invoke verify_loop_closed_ssa with an extra argument. * graphite-scop-detection.c (canonicalize_loop_closed_ssa_form): Same. * graphite-sese-to-poly.c (rewrite_reductions_out_of_ssa): Same. (rewrite_commutative_reductions_out_of_ssa): Same. * passes.c (execute_function_todo): Call verify_ssa for every pass in the LNO. Invoke verify_loop_closed_ssa with an extra argument. * tree-flow.h (verify_loop_closed_ssa): Update declaration. * tree-parloops.c (parallelize_loops): Invoke verify_loop_closed_ssa with an extra argument. * tree-ssa-loop-manip.c (check_loop_closed_ssa_stmt): Same. Call verify_ssa only when the extra argument is true. (gimple_duplicate_loop_to_header_edge): Invoke verify_loop_closed_ssa with an extra argument. (tree_transform_and_unroll_loop): Same. git-svn-id: svn+ssh://gcc.gnu.org/svn/gcc/trunk@158021 138bc75d-0d04-0410-961f-82ee72b054a4 --- gcc/ChangeLog | 19 +++++++++++++++++++ gcc/graphite-clast-to-gimple.c | 3 +-- gcc/graphite-sese-to-poly.c | 9 +++------ gcc/passes.c | 5 +++-- 4 files changed, 26 insertions(+), 10 deletions(-) diff --git a/gcc/ChangeLog b/gcc/ChangeLog index 14898408028..419895f4191 100644 --- a/gcc/ChangeLog +++ b/gcc/ChangeLog @@ -1,5 +1,24 @@ 2010-04-06 Sebastian Pop + * graphite-clast-to-gimple.c (graphite_verify): Remove redundant + call to verify_ssa. Invoke verify_loop_closed_ssa with an extra + argument. + * graphite-scop-detection.c (canonicalize_loop_closed_ssa_form): Same. + * graphite-sese-to-poly.c (rewrite_reductions_out_of_ssa): Same. + (rewrite_commutative_reductions_out_of_ssa): Same. + * passes.c (execute_function_todo): Call verify_ssa for every pass + in the LNO. Invoke verify_loop_closed_ssa with an extra argument. + * tree-flow.h (verify_loop_closed_ssa): Update declaration. + * tree-parloops.c (parallelize_loops): Invoke verify_loop_closed_ssa + with an extra argument. + * tree-ssa-loop-manip.c (check_loop_closed_ssa_stmt): Same. Call + verify_ssa only when the extra argument is true. + (gimple_duplicate_loop_to_header_edge): Invoke verify_loop_closed_ssa + with an extra argument. + (tree_transform_and_unroll_loop): Same. + +2010-04-06 Sebastian Pop + * passes.c (execute_function_todo): Call verify_loop_closed_ssa for all the passes of the LNO having LOOP_CLOSED_SSA. * tree-if-conv.c (pass_if_conversion): Remove TODO_verify_loops. diff --git a/gcc/graphite-clast-to-gimple.c b/gcc/graphite-clast-to-gimple.c index c01d899fe1a..819b9247533 100644 --- a/gcc/graphite-clast-to-gimple.c +++ b/gcc/graphite-clast-to-gimple.c @@ -65,8 +65,7 @@ graphite_verify (void) verify_loop_structure (); verify_dominators (CDI_DOMINATORS); verify_dominators (CDI_POST_DOMINATORS); - verify_ssa (false); - verify_loop_closed_ssa (); + verify_loop_closed_ssa (true); #endif } diff --git a/gcc/graphite-sese-to-poly.c b/gcc/graphite-sese-to-poly.c index d4bb8a37c25..17fb5a2d6b4 100644 --- a/gcc/graphite-sese-to-poly.c +++ b/gcc/graphite-sese-to-poly.c @@ -2402,8 +2402,7 @@ rewrite_reductions_out_of_ssa (scop_p scop) update_ssa (TODO_update_ssa); #ifdef ENABLE_CHECKING - verify_ssa (false); - verify_loop_closed_ssa (); + verify_loop_closed_ssa (true); #endif FOR_EACH_BB (bb) @@ -2413,8 +2412,7 @@ rewrite_reductions_out_of_ssa (scop_p scop) update_ssa (TODO_update_ssa); #ifdef ENABLE_CHECKING - verify_ssa (false); - verify_loop_closed_ssa (); + verify_loop_closed_ssa (true); #endif } @@ -2868,8 +2866,7 @@ rewrite_commutative_reductions_out_of_ssa (sese region, sbitmap reductions) gsi_commit_edge_inserts (); update_ssa (TODO_update_ssa); #ifdef ENABLE_CHECKING - verify_ssa (false); - verify_loop_closed_ssa (); + verify_loop_closed_ssa (true); #endif } diff --git a/gcc/passes.c b/gcc/passes.c index 0d42f9c4af4..cbd605fb430 100644 --- a/gcc/passes.c +++ b/gcc/passes.c @@ -1256,14 +1256,15 @@ execute_function_todo (void *data) } #if defined ENABLE_CHECKING - if (flags & TODO_verify_ssa) + if (flags & TODO_verify_ssa + || (current_loops && loops_state_satisfies_p (LOOP_CLOSED_SSA))) verify_ssa (true); if (flags & TODO_verify_flow) verify_flow_info (); if (flags & TODO_verify_stmts) verify_stmts (); if (current_loops && loops_state_satisfies_p (LOOP_CLOSED_SSA)) - verify_loop_closed_ssa (); + verify_loop_closed_ssa (false); if (flags & TODO_verify_rtl_sharing) verify_rtl_sharing (); #endif -- 2.11.0