* 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

diff --git a/gcc/ChangeLog b/gcc/ChangeLog
index a94318e..4dd7008 100644 (file)
--- 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 dde9aee..0c36f3e 100644 (file)
--- 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