tree-cfg.c (thread_jumps): Speed up by reordering the two conditions for entering...

	* tree-cfg.c (thread_jumps): Speed up by reordering the two
	conditions for entering basic blocks into worklist.

From-SVN: r89700

diff --git a/gcc/ChangeLog b/gcc/ChangeLog
index fcafd5f75bb476d01cba229665516a1a8a7c57ac..ca2253e9e2593b81648ef43a550d1bfe5bf90843 100644 (file)
--- a/gcc/ChangeLog
+++ b/gcc/ChangeLog
@@ -1,3 +1,8 @@
+2004-10-27  Kazu Hirata  <kazu@cs.umass.edu>
+
+       * tree-cfg.c (thread_jumps): Speed up by reordering the two
+       conditions for entering basic blocks into worklist.
+
 2004-10-27  Kazu Hirata  <kazu@cs.umass.edu>
 
        * tree-cfg.c (thread_jumps): Speed up by pretending to have

diff --git a/gcc/tree-cfg.c b/gcc/tree-cfg.c
index 4492daa1e726b940292c99d1a294122df0249746..e606749fa9d51909043b26295bc5bc2ce3bb6ac4 100644 (file)
--- a/gcc/tree-cfg.c
+++ b/gcc/tree-cfg.c
@@ -3974,11 +3974,11 @@ thread_jumps (void)
         among BB's predecessors.  */
       FOR_EACH_EDGE (e, ei, bb->preds)
        {
-         /* We are not interested in threading jumps from a forwarder
-            block.  */
-         if (!bb_ann (e->src)->forwardable
-             /* We don't want to put a duplicate into WORKLIST.  */
-             && (e->src->flags & BB_VISITED) == 0)
+         /* We don't want to put a duplicate into WORKLIST.  */
+         if ((e->src->flags & BB_VISITED) == 0
+             /* We are not interested in threading jumps from a forwarder
+                block.  */
+             && !bb_ann (e->src)->forwardable)
            {
              e->src->flags |= BB_VISITED;
              worklist[size] = e->src;
@@ -4013,11 +4013,11 @@ thread_jumps (void)
                 predecessors.  */
              FOR_EACH_EDGE (f, ej, bb->preds)
                {
-                 /* We are not interested in threading jumps from a
-                    forwarder block.  */
-                 if (!bb_ann (f->src)->forwardable
-                     /* We don't want to put a duplicate into WORKLIST.  */
-                     && (f->src->flags & BB_VISITED) == 0)
+                 /* We don't want to put a duplicate into WORKLIST.  */
+                 if ((f->src->flags & BB_VISITED) == 0
+                     /* We are not interested in threading jumps from a
+                        forwarder block.  */
+                     && !bb_ann (f->src)->forwardable)
                    {
                      f->src->flags |= BB_VISITED;
                      worklist[size] = f->src;