diff options
| author | kazu <kazu@138bc75d-0d04-0410-961f-82ee72b054a4> | 2004-10-05 18:48:44 +0000 |
|---|---|---|
| committer | kazu <kazu@138bc75d-0d04-0410-961f-82ee72b054a4> | 2004-10-05 18:48:44 +0000 |
| commit | 2d0a22bf0c21bba1fa730995f66a0e38c20cae27 (patch) | |
| tree | eddfa4d02f3e42ac752f34ead71788672ddbf493 | |
| parent | 7e72af53dcdaa380c45a0195b691cb47ab6866ef (diff) | |
* tree-cfg.c (cleanup_tree_cfg): Remove variable
something_changed. Simplify the while loop.
git-svn-id: svn+ssh://gcc.gnu.org/svn/gcc/trunk@88564 138bc75d-0d04-0410-961f-82ee72b054a4
| -rw-r--r-- | gcc/ChangeLog | 5 | ||||
| -rw-r--r-- | gcc/tree-cfg.c | 11 |
2 files changed, 8 insertions, 8 deletions
diff --git a/gcc/ChangeLog b/gcc/ChangeLog index a94318e9d78..4dd70086bc5 100644 --- a/gcc/ChangeLog +++ b/gcc/ChangeLog @@ -1,3 +1,8 @@ +2004-10-05 Kazu Hirata <kazu@cs.umass.edu> + + * tree-cfg.c (cleanup_tree_cfg): Remove variable + something_changed. Simplify the while loop. + 2004-10-05 Kaveh R. Ghazi <ghazi@caip.rutgers.edu> * builtins.c (expand_builtin_memmove): Delete duplicate code diff --git a/gcc/tree-cfg.c b/gcc/tree-cfg.c index dde9aee0d0f..0c36f3e5d6a 100644 --- a/gcc/tree-cfg.c +++ b/gcc/tree-cfg.c @@ -712,7 +712,6 @@ make_goto_expr_edges (basic_block bb) bool cleanup_tree_cfg (void) { - bool something_changed = true; bool retval = false; timevar_push (TV_TREE_CLEANUP_CFG); @@ -723,16 +722,12 @@ cleanup_tree_cfg (void) /* thread_jumps() sometimes leaves further transformation opportunities for itself, so iterate on it until nothing changes. */ - while (something_changed) + while (thread_jumps ()) { - something_changed = thread_jumps (); - /* delete_unreachable_blocks() does its job only when thread_jumps() produces more unreachable blocks. */ - if (something_changed) - delete_unreachable_blocks (); - - retval |= something_changed; + delete_unreachable_blocks (); + retval = true; } #ifdef ENABLE_CHECKING |