tree-cfg.c (cleanup_tree_cfg): Remove variable something_changed.

	* tree-cfg.c (cleanup_tree_cfg): Remove variable
	something_changed.  Simplify the while loop.

From-SVN: r88564

diff --git a/gcc/ChangeLog b/gcc/ChangeLog
index a94318e9d786f484d0f36af5bd0fc1d12989f10e..4dd70086bc54580e64e87b2c8f1cef0f509aeef2 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 dde9aee0d0fec6910bf3f582a4d96b4e11584c54..0c36f3e5d6aee6bdf647cc49eee3b2d90df65188 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