X-Git-Url: http://nsz.repo.hu/git/?a=blobdiff_plain;f=ir%2Fopt%2Fcfopt.c;h=e9497a596aeb1c396e8088324048287f8e5cb9c8;hb=04906225f6fe1bc22f73b5d6a98eb9c0992a9b54;hp=e3a14b48dcfda4af895d9c4e73faed5aeb5fd021;hpb=b597c7fd473086ca6374b2abbdf129f595c156d0;p=libfirm

diff --git a/ir/opt/cfopt.c b/ir/opt/cfopt.c
index e3a14b48d..e9497a596 100644
--- a/ir/opt/cfopt.c
+++ b/ir/opt/cfopt.c
@@ -23,9 +23,7 @@
  * @author  Goetz Lindenmaier, Michael Beck, Sebastian Hack
  * @version $Id$
  */
-#ifdef HAVE_CONFIG_H
-# include "config.h"
-#endif
+#include "config.h"
 
 #include "iroptimize.h"
 
@@ -45,7 +43,7 @@
 #include "irvrfy.h"
 #include "iredges.h"
 
-#include "array.h"
+#include "array_t.h"
 
 #include "irouts.h"
 #include "irbackedge_t.h"
@@ -165,7 +163,7 @@ static void merge_blocks(ir_node *node, void *ctx) {
 		if (!is_Block_dead(b)) {
 			new_block = equivalent_node(b);
 
-			while (irn_not_visited(b) && (!is_Block_dead(new_block)) && (new_block != b)) {
+			while (!irn_visited(b) && !is_Block_dead(new_block) && new_block != b) {
 				/* We would have to run gigo() if new is bad, so we
 				   promote it directly below. Nevertheless, we sometimes reach a block
 				   the first time through a dataflow node.  In this case we optimized the
@@ -421,7 +419,7 @@ static void optimize_blocks(ir_node *b, void *ctx) {
 	for (i = 0, k = get_Block_n_cfgpreds(b); i < k; ++i) {
 		max_preds += test_whether_dispensable(b, i);
 	}
-	in = xmalloc(max_preds * sizeof(*in));
+	in = XMALLOCN(ir_node*, max_preds);
 
 	/*- Fix the Phi nodes of the current block -*/
 	for (phi = get_irn_link(b); phi; ) {
@@ -434,8 +432,7 @@ static void optimize_blocks(ir_node *b, void *ctx) {
 
 			if (is_Block_dead(pred)) {
 				/* case Phi 1: Do nothing */
-			}
-			else if (is_Block_removable(pred) && Block_not_block_visited(pred)) {
+			} else if (is_Block_removable(pred) && !Block_block_visited(pred)) {
 				/* case Phi 2: It's an empty block and not yet visited. */
 				ir_node *phi_pred = get_Phi_pred(phi, i);
 
@@ -478,7 +475,7 @@ static void optimize_blocks(ir_node *b, void *ctx) {
 	for (k = 0, n = get_Block_n_cfgpreds(b); k < n; ++k) {
 		ir_node *predb = get_nodes_block(get_Block_cfgpred(b, k));
 
-		if (is_Block_removable(predb) && Block_not_block_visited(predb)) {
+		if (is_Block_removable(predb) && !Block_block_visited(predb)) {
 			ir_node *next_phi;
 
 			/* we found a predecessor block at position k that will be removed */
@@ -505,7 +502,7 @@ static void optimize_blocks(ir_node *b, void *ctx) {
 
 						if (is_Block_dead(pred)) {
 							/* Do nothing */
-						} else if (is_Block_removable(pred) && Block_not_block_visited(pred)) {
+						} else if (is_Block_removable(pred) && !Block_block_visited(pred)) {
 							/* It's an empty block and not yet visited. */
 							for (j = 0; j < get_Block_n_cfgpreds(pred); j++) {
 								if (! is_Bad(get_Block_cfgpred(pred, j)))
@@ -529,7 +526,7 @@ static void optimize_blocks(ir_node *b, void *ctx) {
 
 						if (is_Block_dead(pred)) {
 							/* Do nothing */
-						} else if (is_Block_removable(pred) && Block_not_block_visited(pred)) {
+						} else if (is_Block_removable(pred) && !Block_block_visited(pred)) {
 							/* It's an empty block and not yet visited. */
 							for (j = 0; j < get_Block_n_cfgpreds(pred); j++) {
 								if (! is_Bad(get_Block_cfgpred(pred, j)))
@@ -561,7 +558,7 @@ static void optimize_blocks(ir_node *b, void *ctx) {
 
 		if (is_Block_dead(pred)) {
 			/* case 1: Do nothing */
-		} else if (is_Block_removable(pred) && Block_not_block_visited(pred)) {
+		} else if (is_Block_removable(pred) && !Block_block_visited(pred)) {
 			/* case 2: It's an empty block and not yet visited. */
 			assert(get_Block_n_cfgpreds(b) > 1);
 			/* Else it should be optimized by equivalent_node. */
@@ -788,20 +785,20 @@ restart:
 	for (i = j = 0; i < n; i++) {
 		ir_node *ka = get_End_keepalive(end, i);
 
-		if (irn_not_visited(ka)) {
-			ir_op *op = get_irn_op(ka);
-
-			if ((op == op_Block) && Block_not_block_visited(ka)) {
-				/* irg_block_walk() will increase the block visited flag, but we must visit only
-				   these blocks that are not visited yet, so decrease it first. */
-				set_irg_block_visited(irg, get_irg_block_visited(irg) - 1);
-				irg_block_walk(ka, optimize_blocks, remove_simple_blocks, &env.changed);
-				mark_irn_visited(ka);
-				in[j++] = ka;
+		if (!irn_visited(ka)) {
+			if (is_Block(ka)) {
+				if (!Block_block_visited(ka)) {
+					/* irg_block_walk() will increase the block visited flag, but we must visit only
+					   these blocks that are not visited yet, so decrease it first. */
+					set_irg_block_visited(irg, get_irg_block_visited(irg) - 1);
+					irg_block_walk(ka, optimize_blocks, remove_simple_blocks, &env.changed);
+					mark_irn_visited(ka);
+					in[j++] = ka;
+				}
 			} else {
 				mark_irn_visited(ka);
 				/* don't keep alive dead blocks */
-				if (! is_Block_dead(get_nodes_block(ka)))
+				if (!is_Bad(ka) && !is_Block_dead(get_nodes_block(ka)))
 					in[j++] = ka;
 			}
 		}