diff options
author | Diego Novillo <dnovillo@redhat.com> | 2005-07-29 16:32:00 +0000 |
---|---|---|
committer | Diego Novillo <dnovillo@gcc.gnu.org> | 2005-07-29 12:32:00 -0400 |
commit | 89e80dd49573d8eda808fb2a5dd09546329f8e82 (patch) | |
tree | b726ec8795790e3c447d315a30f79bfbbd34ed85 /gcc/tree-cfgcleanup.c | |
parent | d2f3ffba9a32449f59973e11b846de08c2683583 (diff) | |
download | gcc-89e80dd49573d8eda808fb2a5dd09546329f8e82.zip gcc-89e80dd49573d8eda808fb2a5dd09546329f8e82.tar.gz gcc-89e80dd49573d8eda808fb2a5dd09546329f8e82.tar.bz2 |
re PR tree-optimization/22550 (ICE in vrp_evaluate_conditional)
PR 22550
* tree-cfgcleanup.c (cleanup_tree_cfg_1): Extract from ...
(cleanup_tree_cfg): ... here.
Call cleanup_tree_cfg_1 until there are no more cleanups to
do.
testsuite/ChangeLog
PR 22550
* g++.dg/tree-ssa/pr22550.C: New test.
From-SVN: r102559
Diffstat (limited to 'gcc/tree-cfgcleanup.c')
-rw-r--r-- | gcc/tree-cfgcleanup.c | 38 |
1 files changed, 28 insertions, 10 deletions
diff --git a/gcc/tree-cfgcleanup.c b/gcc/tree-cfgcleanup.c index 51b2fc5..f2454b0 100644 --- a/gcc/tree-cfgcleanup.c +++ b/gcc/tree-cfgcleanup.c @@ -489,14 +489,12 @@ cleanup_forwarder_blocks (void) return changed; } -/* Remove unreachable blocks and other miscellaneous clean up work. */ +/* Do one round of CFG cleanup. */ -bool -cleanup_tree_cfg (void) +static bool +cleanup_tree_cfg_1 (void) { - bool retval = false; - - timevar_push (TV_TREE_CLEANUP_CFG); + bool retval; retval = cleanup_control_flow (); retval |= delete_unreachable_blocks (); @@ -516,6 +514,28 @@ cleanup_tree_cfg (void) end_recording_case_labels (); } + /* Merging the blocks may create new opportunities for folding + conditional branches (due to the elimination of single-valued PHI + nodes). */ + retval |= merge_seq_blocks (); + + return retval; +} + + +/* Remove unreachable blocks and other miscellaneous clean up work. */ + +bool +cleanup_tree_cfg (void) +{ + bool retval; + int i; + + timevar_push (TV_TREE_CLEANUP_CFG); + + for (retval = true, i = 0; i < 5 && retval; i++) + retval = cleanup_tree_cfg_1 (); + #ifdef ENABLE_CHECKING if (retval) { @@ -526,16 +546,14 @@ cleanup_tree_cfg (void) } #endif - /* Merging the blocks creates no new opportunities for the other - optimizations, so do it here. */ - retval |= merge_seq_blocks (); - compact_blocks (); #ifdef ENABLE_CHECKING verify_flow_info (); #endif + timevar_pop (TV_TREE_CLEANUP_CFG); + return retval; } |