aboutsummaryrefslogtreecommitdiff
path: root/gcc/tree-cfgcleanup.c
diff options
context:
space:
mode:
authorDiego Novillo <dnovillo@redhat.com>2005-07-29 16:32:00 +0000
committerDiego Novillo <dnovillo@gcc.gnu.org>2005-07-29 12:32:00 -0400
commit89e80dd49573d8eda808fb2a5dd09546329f8e82 (patch)
treeb726ec8795790e3c447d315a30f79bfbbd34ed85 /gcc/tree-cfgcleanup.c
parentd2f3ffba9a32449f59973e11b846de08c2683583 (diff)
downloadgcc-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.c38
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;
}