X-Git-Url: http://nsz.repo.hu/git/?a=blobdiff_plain;f=ir%2Fbe%2Fbesched.c;h=a2c82b9d9fcca44efefe7760d6a9f256b4744f7a;hb=cbfbedae75798a9830fb0ef090189345ede85dc8;hp=a3c0e029f373a12b2c2fbd6afa6f5d6c2715b955;hpb=4cdd9d655a6d671f90f04d964769d2a664c4d734;p=libfirm

diff --git a/ir/be/besched.c b/ir/be/besched.c
index a3c0e029f..a2c82b9d9 100644
--- a/ir/be/besched.c
+++ b/ir/be/besched.c
@@ -1,3 +1,5 @@
+/* $Id$ */
+
 #ifdef HAVE_CONFIG_H
 # include "config.h"
 #endif
@@ -12,6 +14,7 @@
 #include "irnode_t.h"
 #include "irgraph_t.h"
 #include "iredges_t.h"
+#include "ircons.h"
 #include "debug.h"
 
 #include "bearch.h"
@@ -54,111 +57,110 @@ void be_sched_dump(FILE *f, ir_graph *irg)
 void be_sched_init(void)
 {
 	sched_irn_data_offset = register_additional_node_data(sizeof(sched_info_t));
-	firm_dbg_register("be.sched");
 }
 
 void sched_renumber(const ir_node *block)
 {
-  ir_node *irn;
-  sched_info_t *inf;
-  sched_timestep_t step = 0;
-
-  sched_foreach(block, irn) {
-    inf = get_irn_sched_info(irn);
-    inf->time_step = step;
-    step += SCHED_INITIAL_GRANULARITY;
-  }
+	ir_node *irn;
+	sched_info_t *inf;
+	sched_timestep_t step = SCHED_INITIAL_GRANULARITY;
+
+	sched_foreach(block, irn) {
+		inf = get_irn_sched_info(irn);
+		inf->time_step = step;
+		step += SCHED_INITIAL_GRANULARITY;
+	}
 }
 
 /* Verify a schedule. */
 int sched_verify(const ir_node *block)
 {
-  firm_dbg_module_t *dbg_sched = firm_dbg_register("be.sched");
-  int res = 1;
-  const ir_node *irn;
-  int i, n;
-  int *save_time_step;
-  const ir_node **save_nodes;
-  const ir_edge_t *edge;
-  pset *scheduled_nodes = pset_new_ptr_default();
-
-  firm_dbg_set_mask(dbg_sched, -1);
-
-  /* Count the number of nodes in the schedule. */
-  n = 0;
-  sched_foreach(block, irn)
-    n++;
-
-  if(n <= 0)
-    return 1;
-
-  save_time_step = xmalloc(n * sizeof(save_time_step[0]));
-  save_nodes = xmalloc(n * sizeof(save_nodes[0]));
-
-  i = 0;
-  sched_foreach(block, irn) {
-    sched_info_t *info = get_irn_sched_info(irn);
-    save_time_step[i] = info->time_step;
-    save_nodes[i] = (ir_node *)irn;
-    info->time_step = i;
-    pset_insert_ptr(scheduled_nodes, irn);
-
-    i += 1;
-  }
-
-  /*
-   * Check if each relevant operand of a node is scheduled before
-   * the node itself.
-   */
-  sched_foreach(block, irn) {
-    int i, n;
-    int step = sched_get_time_step(irn);
-
-    for(i = 0, n = get_irn_arity(irn); i < n; i++) {
-      ir_node *op = get_irn_n(irn, i);
-
-      if(to_appear_in_schedule(op)
-          && !is_Phi(irn)
-          && get_nodes_block(op) == block
-          && sched_get_time_step(op) > step) {
-
-          DBG((dbg_sched, LEVEL_DEFAULT,
-                "%+F: %+F is operand of %+F but scheduled after\n", block, op, irn));
-          res = 0;
-      }
-    }
-  }
-
-  /* Check, if the time steps are correct */
-  for(i = 1; i < n; ++i) {
-    if(save_time_step[i] - save_time_step[i - 1] <= 0) {
-      DBG((dbg_sched, LEVEL_DEFAULT,
-            "%+F from %+F(%d) -> %+F(%d) step shrinks from %d -> %d\n",
-            block, save_nodes[i - 1], i - 1, save_nodes[i], i,
-            save_time_step[i - 1], save_time_step[i]));
-      res = 0;
-    }
-  }
-
-  /* Restore the old time steps */
-  i = 0;
-  sched_foreach(block, irn) {
-    sched_info_t *info = get_irn_sched_info(irn);
-    info->time_step = save_time_step[i++];
-  }
-
-  /* Check for all nodes in the block if they are scheduled. */
-  foreach_out_edge(block, edge) {
-    ir_node *irn = get_edge_src_irn(edge);
-    if(to_appear_in_schedule(irn) && !pset_find_ptr(scheduled_nodes, irn))
-      DBG((dbg_sched, LEVEL_DEFAULT,
-            "%+F: %+F is in block but not scheduled\n", block, irn));
-  }
-
-  del_pset(scheduled_nodes);
-  free(save_time_step);
-  free((void *) save_nodes);
-  return res;
+	int res = 1;
+	const ir_node *irn;
+	int i, n;
+	int *save_time_step;
+	const ir_node **save_nodes;
+	const ir_edge_t *edge;
+	pset *scheduled_nodes = pset_new_ptr_default();
+	FIRM_DBG_REGISTER(firm_dbg_module_t *dbg_sched, "firm.be.sched");
+
+	/* Count the number of nodes in the schedule. */
+	n = 0;
+	sched_foreach(block, irn)
+		n++;
+
+	if(n <= 0)
+		return 1;
+
+	save_time_step = xmalloc(n * sizeof(save_time_step[0]));
+	save_nodes = xmalloc(n * sizeof(save_nodes[0]));
+
+	i = 0;
+	sched_foreach(block, irn) {
+		sched_info_t *info = get_irn_sched_info(irn);
+		save_time_step[i] = info->time_step;
+		save_nodes[i] = (ir_node *)irn;
+		info->time_step = i;
+		pset_insert_ptr(scheduled_nodes, irn);
+
+		i += 1;
+	}
+
+	/*
+	 * Check if each relevant operand of a node is scheduled before
+	 * the node itself.
+	 */
+	sched_foreach(block, irn) {
+		int i, n;
+		int step = sched_get_time_step(irn);
+
+		for(i = 0, n = get_irn_arity(irn); i < n; i++) {
+			ir_node *op = get_irn_n(irn, i);
+
+			if(to_appear_in_schedule(op)
+				&& !is_Phi(irn)
+				&& get_nodes_block(op) == block
+				&& sched_get_time_step(op) > step) {
+
+				DBG((dbg_sched, LEVEL_DEFAULT,
+					"%+F: %+F is operand of %+F but scheduled after\n", block, op, irn));
+				res = 0;
+			}
+		}
+	}
+
+	/* Check, if the time steps are correct */
+	for(i = 1; i < n; ++i) {
+		if(save_time_step[i] - save_time_step[i - 1] <= 0) {
+			DBG((dbg_sched, LEVEL_DEFAULT,
+				"%+F from %+F(%d) -> %+F(%d) step shrinks from %d -> %d\n",
+				block, save_nodes[i - 1], i - 1, save_nodes[i], i,
+				save_time_step[i - 1], save_time_step[i]));
+			res = 0;
+		}
+	}
+
+	/* Restore the old time steps */
+	i = 0;
+	sched_foreach(block, irn) {
+		sched_info_t *info = get_irn_sched_info(irn);
+		info->time_step = save_time_step[i++];
+	}
+
+	/* Check for all nodes in the block if they are scheduled. */
+	foreach_out_edge(block, edge) {
+		ir_node *irn = get_edge_src_irn(edge);
+		if(to_appear_in_schedule(irn) && !pset_find_ptr(scheduled_nodes, irn)) {
+			DBG((dbg_sched, LEVEL_DEFAULT,
+				"%+F: %+F is in block but not scheduled\n", block, irn));
+			res = 0;
+		}
+	}
+
+	del_pset(scheduled_nodes);
+	free(save_time_step);
+	free((void *) save_nodes);
+	return res;
 }
 
 /**
@@ -166,22 +168,22 @@ int sched_verify(const ir_node *block)
  */
 static void sched_verify_walker(ir_node *block, void *data)
 {
-  int *res = data;
-  *res &= sched_verify(block);
+	int *res = data;
+	*res &= sched_verify(block);
 }
 
 /* Verify the schedules in all blocks of the irg. */
 int sched_verify_irg(ir_graph *irg)
 {
-  int res = 1;
-  irg_block_walk_graph(irg, sched_verify_walker, NULL, &res);
+	int res = 1;
+	irg_block_walk_graph(irg, sched_verify_walker, NULL, &res);
 
-  return res;
+	return res;
 }
 
 int sched_skip_cf_predicator(const ir_node *irn, void *data) {
-  arch_env_t *ae = data;
-  return arch_irn_classify(ae, irn) == arch_irn_class_branch;
+	arch_env_t *ae = data;
+	return arch_irn_class_is(ae, irn, branch);
 }
 
 int sched_skip_phi_predicator(const ir_node *irn, void *data) {
@@ -259,3 +261,49 @@ ir_node **sched_create_block_schedule(ir_graph *irg)
 	}
 	return blk_list;
 }
+
+typedef struct remove_dead_nodes_env_t_ {
+	ir_graph *irg;
+	bitset_t *reachable;
+} remove_dead_nodes_env_t;
+
+static void mark_dead_nodes_walker(ir_node *node, void *data)
+{
+	remove_dead_nodes_env_t *env = (remove_dead_nodes_env_t*) data;
+	bitset_set(env->reachable, get_irn_idx(node));
+}
+
+static void remove_dead_nodes_walker(ir_node *block, void *data)
+{
+	remove_dead_nodes_env_t *env = (remove_dead_nodes_env_t*) data;
+	ir_node *node, *next;
+
+	for(node = sched_first(block); !sched_is_end(node); node = next) {
+		int i, arity;
+
+		// get next node now, as after calling sched_remove it will be invalid
+		next = sched_next(node);
+
+		if(bitset_is_set(env->reachable, get_irn_idx(node)))
+			continue;
+
+		arity = get_irn_arity(node);
+		for(i = 0; i < arity; ++i)
+			set_irn_n(node, i, new_r_Bad(env->irg));
+
+		sched_remove(node);
+	}
+}
+
+void be_remove_dead_nodes_from_schedule(ir_graph *irg)
+{
+	remove_dead_nodes_env_t env;
+	env.irg = irg;
+	env.reachable = bitset_alloca(get_irg_last_idx(irg));
+
+	// mark all reachable nodes
+	irg_walk_graph(irg, mark_dead_nodes_walker, NULL, &env);
+
+	// walk schedule and remove non-marked nodes
+	irg_block_walk_graph(irg, remove_dead_nodes_walker, NULL, &env);
+}