X-Git-Url: http://nsz.repo.hu/git/?a=blobdiff_plain;f=ir%2Fopt%2Ftailrec.c;h=39e54aa3d338e0d4643fa12739d25d61db4837a1;hb=b7b24e372a1338ecd5eb26bdd285a8cbe7b1fec9;hp=551a8fce81e9c24a132d2ea9ffb2ba4491f7357f;hpb=5ecbb181a733f3f29f9abf3715b03606bb3fb9a2;p=libfirm

diff --git a/ir/opt/tailrec.c b/ir/opt/tailrec.c
index 551a8fce8..39e54aa3d 100644
--- a/ir/opt/tailrec.c
+++ b/ir/opt/tailrec.c
@@ -1,5 +1,5 @@
 /*
- * Copyright (C) 1995-2008 University of Karlsruhe.  All right reserved.
+ * Copyright (C) 1995-2011 University of Karlsruhe.  All right reserved.
  *
  * This file is part of libFirm.
  *
@@ -45,13 +45,14 @@
 #include "irouts.h"
 #include "irhooks.h"
 #include "ircons_t.h"
+#include "irpass.h"
 
 DEBUG_ONLY(static firm_dbg_module_t *dbg);
 
 /**
  * the environment for collecting data
  */
-typedef struct _collect_t {
+typedef struct collect_t {
 	ir_node *proj_X;      /**< initial exec proj */
 	ir_node *block;       /**< old first block */
 	int     blk_idx;      /**< cfgpred index of the initial exec in block */
@@ -62,8 +63,9 @@ typedef struct _collect_t {
 /**
  * walker for collecting data, fills a collect_t environment
  */
-static void collect_data(ir_node *node, void *env) {
-	collect_t *data = env;
+static void collect_data(ir_node *node, void *env)
+{
+	collect_t *data = (collect_t*)env;
 	ir_node *pred;
 	ir_op *op;
 
@@ -95,7 +97,7 @@ static void collect_data(ir_node *node, void *env) {
 		/*
 		 * the first block has the initial exec as cfg predecessor
 		 */
-		if (node != get_irg_start_block(current_ir_graph)) {
+		if (node != get_irg_start_block(get_irn_irg(node))) {
 			for (i = 0; i < n_pred; ++i) {
 				if (get_Block_cfgpred(node, i) == data->proj_X) {
 					data->block   = node;
@@ -134,7 +136,8 @@ typedef struct tr_env {
  * @param rets          linked list of all rets
  * @param n_tail_calls  number of tail-recursion calls
  */
-static void do_opt_tail_rec(ir_graph *irg, tr_env *env) {
+static void do_opt_tail_rec(ir_graph *irg, tr_env *env)
+{
 	ir_node *end_block = get_irg_end_block(irg);
 	ir_node *block, *jmp, *call, *calls;
 	ir_node **in;
@@ -146,9 +149,6 @@ static void do_opt_tail_rec(ir_graph *irg, tr_env *env) {
 	int rem            = get_optimize();
 	ir_entity *ent     = get_irg_entity(irg);
 	ir_type *method_tp = get_entity_type(ent);
-	ir_graph *old      = current_ir_graph;
-
-	current_ir_graph = irg;
 
 	assert(env->n_tail_calls > 0);
 
@@ -177,7 +177,7 @@ static void do_opt_tail_rec(ir_graph *irg, tr_env *env) {
 	irg_walk_graph(irg, NULL, collect_data, &data);
 
 	/* check number of arguments */
-	call = get_irn_link(end_block);
+	call     = (ir_node*)get_irn_link(end_block);
 	n_params = get_Call_n_params(call);
 
 	assert(data.proj_X && "Could not find initial exec from Start");
@@ -194,8 +194,8 @@ static void do_opt_tail_rec(ir_graph *irg, tr_env *env) {
 	for (i = 1, p = env->rets; p; p = n) {
 		ir_node *block = get_nodes_block(p);
 
-		n = get_irn_link(p);
-		in[i++] = new_r_Jmp(irg, block);
+		n = (ir_node*)get_irn_link(p);
+		in[i++] = new_r_Jmp(block);
 
 		// exchange(p, new_r_Bad(irg));
 
@@ -206,7 +206,7 @@ static void do_opt_tail_rec(ir_graph *irg, tr_env *env) {
 
 	/* create a new block at start */
 	block = new_r_Block(irg, env->n_tail_calls + 1, in);
-	jmp   = new_r_Jmp(irg, block);
+	jmp   = new_r_Jmp(block);
 
 	/* the old first block is now the second one */
 	set_Block_cfgpred(data.block, data.blk_idx, jmp);
@@ -216,17 +216,17 @@ static void do_opt_tail_rec(ir_graph *irg, tr_env *env) {
 
 	/* build the memory phi */
 	i = 0;
-	in[i] = new_r_Proj(irg, get_irg_start_block(irg), get_irg_start(irg), mode_M, pn_Start_M);
+	in[i] = new_r_Proj(get_irg_start(irg), mode_M, pn_Start_M);
 	set_irg_initial_mem(irg, in[i]);
 	++i;
 
-	for (calls = call; calls; calls = get_irn_link(calls)) {
+	for (calls = call; calls != NULL; calls = (ir_node*)get_irn_link(calls)) {
 		in[i] = get_Call_mem(calls);
 		++i;
 	}
 	assert(i == env->n_tail_calls + 1);
 
-	phis[0] = new_r_Phi(irg, block, env->n_tail_calls + 1, in, mode_M);
+	phis[0] = new_r_Phi(block, env->n_tail_calls + 1, in, mode_M);
 
 	/* build the data Phi's */
 	if (n_params > 0) {
@@ -237,7 +237,8 @@ static void do_opt_tail_rec(ir_graph *irg, tr_env *env) {
 		NEW_ARR_A(ir_node **, call_params, env->n_tail_calls);
 
 		/* collect all parameters */
-		for (i = 0, calls = call; calls; calls = get_irn_link(calls)) {
+		for (i = 0, calls = call; calls != NULL;
+		     calls = (ir_node*)get_irn_link(calls)) {
 			call_params[i] = get_Call_param_arr(calls);
 			++i;
 		}
@@ -248,11 +249,11 @@ static void do_opt_tail_rec(ir_graph *irg, tr_env *env) {
 		for (i = 0; i < n_params; ++i) {
 			ir_mode *mode = get_type_mode(get_method_param_type(method_tp, i));
 
-			in[0] = new_r_Proj(irg, args_bl, args, mode, i);
+			in[0] = new_r_Proj(args, mode, i);
 			for (j = 0; j < env->n_tail_calls; ++j)
 				in[j + 1] = call_params[j][i];
 
-			phis[i + 1] = new_r_Phi(irg, block, env->n_tail_calls + 1, in, mode);
+			phis[i + 1] = new_r_Phi(block, env->n_tail_calls + 1, in, mode);
 		}
 	}
 
@@ -265,7 +266,7 @@ static void do_opt_tail_rec(ir_graph *irg, tr_env *env) {
 		long proj = get_Proj_proj(p);
 
 		assert(0 <= proj && proj < n_params);
-		n = get_irn_link(p);
+		n = (ir_node*)get_irn_link(p);
 		exchange(p, phis[proj + 1]);
 	}
 
@@ -273,7 +274,7 @@ static void do_opt_tail_rec(ir_graph *irg, tr_env *env) {
 	set_irg_doms_inconsistent(irg);
 	set_irg_outs_inconsistent(irg);
 	set_irg_extblk_inconsistent(irg);
-	set_irg_loopinfo_state(current_ir_graph, loopinfo_cf_inconsistent);
+	set_irg_loopinfo_state(irg, loopinfo_cf_inconsistent);
 	set_trouts_inconsistent();
 	set_irg_callee_info_state(irg, irg_callee_info_inconsistent);
 
@@ -304,9 +305,9 @@ static void do_opt_tail_rec(ir_graph *irg, tr_env *env) {
 
 			modes[i] = mode;
 			if (env->variants[i] == TR_ADD) {
-				set_value(i, new_Const(get_mode_null(mode)));
+				set_value(i, new_r_Const(irg, get_mode_null(mode)));
 			} else if (env->variants[i] == TR_MUL) {
-				set_value(i, new_Const(get_mode_one(mode)));
+				set_value(i, new_r_Const(irg, get_mode_one(mode)));
 			}
 		}
 		mature_immBlock(start_block);
@@ -319,7 +320,7 @@ static void do_opt_tail_rec(ir_graph *irg, tr_env *env) {
 			ir_node *call, *mem, *jmp, *tuple;
 
 			set_cur_block(block);
-			n = get_irn_link(p);
+			n = (ir_node*)get_irn_link(p);
 
 			call = skip_Proj(get_Return_mem(p));
 			assert(is_Call(call));
@@ -328,7 +329,7 @@ static void do_opt_tail_rec(ir_graph *irg, tr_env *env) {
 
 			/* create a new jump, free of CSE */
 			set_optimize(0);
-			jmp = new_Jmp();
+			jmp = new_r_Jmp(block);
 			set_optimize(rem);
 
 			for (i = 0; i < env->n_ress; ++i) {
@@ -339,15 +340,13 @@ static void do_opt_tail_rec(ir_graph *irg, tr_env *env) {
 				}
 			}
 			/* create a new tuple for the return values */
-			tuple = new_Tuple(env->n_ress, in);
+			tuple = new_r_Tuple(block, env->n_ress, in);
 
 			turn_into_tuple(call, pn_Call_max);
 			set_Tuple_pred(call, pn_Call_M,                mem);
 			set_Tuple_pred(call, pn_Call_X_regular,        jmp);
 			set_Tuple_pred(call, pn_Call_X_except,         bad);
 			set_Tuple_pred(call, pn_Call_T_result,         tuple);
-			set_Tuple_pred(call, pn_Call_M_except,         mem);
-			set_Tuple_pred(call, pn_Call_P_value_res_base, bad);
 
 			for (i = 0; i < env->n_ress; ++i) {
 				ir_node *res = get_Return_res(p, i);
@@ -363,12 +362,14 @@ static void do_opt_tail_rec(ir_graph *irg, tr_env *env) {
 		end_block = get_irg_end_block(irg);
 		for (i = get_Block_n_cfgpreds(end_block) - 1; i >= 0; --i) {
 			ir_node *ret = get_Block_cfgpred(end_block, i);
+			ir_node *block;
 
 			/* search all Returns of a block */
 			if (! is_Return(ret))
 				continue;
 
-			set_cur_block(get_nodes_block(ret));
+			block = get_nodes_block(ret);
+			set_cur_block(block);
 			for (j = 0; j < env->n_ress; ++j) {
 				ir_node *pred = get_Return_res(ret, j);
 				ir_node *n;
@@ -379,13 +380,13 @@ static void do_opt_tail_rec(ir_graph *irg, tr_env *env) {
 
 				case TR_ADD:
 					n = get_value(j, modes[j]);
-					n = new_Add(n, pred, modes[j]);
+					n = new_r_Add(block, n, pred, modes[j]);
 					set_Return_res(ret, j, n);
 					break;
 
 				case TR_MUL:
 					n = get_value(j, modes[j]);
-					n = new_Mul(n, pred, modes[j]);
+					n = new_r_Mul(block, n, pred, modes[j]);
 					set_Return_res(ret, j, n);
 					break;
 
@@ -400,11 +401,10 @@ static void do_opt_tail_rec(ir_graph *irg, tr_env *env) {
 
 		/* no: we can kill all returns */
 		for (p = env->rets; p; p = n) {
-			n = get_irn_link(p);
+			n = (ir_node*)get_irn_link(p);
 			exchange(p, bad);
 		}
 	}
-	current_ir_graph = old;
 }
 
 /**
@@ -416,8 +416,9 @@ static void do_opt_tail_rec(ir_graph *irg, tr_env *env) {
  *
  * @return non-zero if it's ok to do tail recursion
  */
-static int check_lifetime_of_locals(ir_graph *irg) {
-	ir_node *irg_frame, *irg_val_param_base;
+static int check_lifetime_of_locals(ir_graph *irg)
+{
+	ir_node *irg_frame;
 	int i;
 	ir_type *frame_tp = get_irg_frame_type(irg);
 
@@ -425,28 +426,24 @@ static int check_lifetime_of_locals(ir_graph *irg) {
 	for (i = get_irn_n_outs(irg_frame) - 1; i >= 0; --i) {
 		ir_node *succ = get_irn_out(irg_frame, i);
 
-		/* we are only interested in entities on the frame type
-		 * (locals), not on the value type */
-		if (get_entity_owner(get_Sel_entity(succ)) != frame_tp)
-			continue;
+		if (is_Sel(succ)) {
+			/* Check if we have compound arguments.
+			   For now, we cannot handle them, */
+			if (get_entity_owner(get_Sel_entity(succ)) != frame_tp)
+				return 0;
 
-		if (is_Sel(succ) && is_address_taken(succ))
-			return 0;
+			if (is_address_taken(succ))
+				return 0;
+		}
 	}
-
-	/* Check if we have compound arguments.
-	   For now, we cannot handle them, */
-	irg_val_param_base = get_irg_value_param_base(irg);
-	if (get_irn_n_outs(irg_val_param_base) > 0)
-		return 0;
-
 	return 1;
 }
 
 /**
  * Examine irn and detect the recursion variant.
  */
-static tail_rec_variants find_variant(ir_node *irn, ir_node *call) {
+static tail_rec_variants find_variant(ir_node *irn, ir_node *call)
+{
 	ir_node           *a, *b;
 	tail_rec_variants va, vb, res;
 
@@ -458,7 +455,7 @@ static tail_rec_variants find_variant(ir_node *irn, ir_node *call) {
 	case iro_Add:
 		/* try additive */
 		a = get_Add_left(irn);
-		if (get_irn_MacroBlock(a) != get_irn_MacroBlock(call)) {
+		if (get_nodes_block(a) != get_nodes_block(call)) {
 			/* we are outside, ignore */
 			va = TR_UNKNOWN;
 		} else {
@@ -467,7 +464,7 @@ static tail_rec_variants find_variant(ir_node *irn, ir_node *call) {
 				return TR_BAD;
 		}
 		b = get_Add_right(irn);
-		if (get_irn_MacroBlock(b) != get_irn_MacroBlock(call)) {
+		if (get_nodes_block(b) != get_nodes_block(call)) {
 			/* we are outside, ignore */
 			vb = TR_UNKNOWN;
 		} else {
@@ -494,7 +491,7 @@ static tail_rec_variants find_variant(ir_node *irn, ir_node *call) {
 	case iro_Sub:
 		/* try additive, but return value must be left */
 		a = get_Sub_left(irn);
-		if (get_irn_MacroBlock(a) != get_irn_MacroBlock(call)) {
+		if (get_nodes_block(a) != get_nodes_block(call)) {
 			/* we are outside, ignore */
 			va = TR_UNKNOWN;
 		} else {
@@ -503,7 +500,7 @@ static tail_rec_variants find_variant(ir_node *irn, ir_node *call) {
 				return TR_BAD;
 		}
 		b = get_Sub_right(irn);
-		if (get_irn_MacroBlock(b) != get_irn_MacroBlock(call)) {
+		if (get_nodes_block(b) != get_nodes_block(call)) {
 			/* we are outside, ignore */
 			vb = TR_UNKNOWN;
 		} else {
@@ -520,7 +517,7 @@ static tail_rec_variants find_variant(ir_node *irn, ir_node *call) {
 	case iro_Mul:
 		/* try multiplicative */
 		a = get_Mul_left(irn);
-		if (get_irn_MacroBlock(a) != get_irn_MacroBlock(call)) {
+		if (get_nodes_block(a) != get_nodes_block(call)) {
 			/* we are outside, ignore */
 			va = TR_UNKNOWN;
 		} else {
@@ -529,7 +526,7 @@ static tail_rec_variants find_variant(ir_node *irn, ir_node *call) {
 				return TR_BAD;
 		}
 		b = get_Mul_right(irn);
-		if (get_irn_MacroBlock(b) != get_irn_MacroBlock(call)) {
+		if (get_nodes_block(b) != get_nodes_block(call)) {
 			/* we are outside, ignore */
 			vb = TR_UNKNOWN;
 		} else {
@@ -573,7 +570,8 @@ static tail_rec_variants find_variant(ir_node *irn, ir_node *call) {
 /*
  * convert simple tail-calls into loops
  */
-int opt_tail_rec_irg(ir_graph *irg) {
+int opt_tail_rec_irg(ir_graph *irg)
+{
 	tr_env            env;
 	ir_node           *end_block;
 	int               i, n_ress, n_tail_calls = 0;
@@ -581,12 +579,13 @@ int opt_tail_rec_irg(ir_graph *irg) {
 	ir_type           *mtd_type, *call_type;
 	ir_entity         *ent;
 
+	FIRM_DBG_REGISTER(dbg, "firm.opt.tailrec");
+
 	assure_irg_outs(irg);
 
 	if (! check_lifetime_of_locals(irg))
 		return 0;
 
-
 	ent      = get_irg_entity(irg);
 	mtd_type = get_entity_type(ent);
     n_ress   = get_method_n_ress(mtd_type);
@@ -707,20 +706,23 @@ int opt_tail_rec_irg(ir_graph *irg) {
 	return n_tail_calls;
 }
 
+ir_graph_pass_t *opt_tail_rec_irg_pass(const char *name)
+{
+	return def_graph_pass_ret(name ? name : "tailrec", opt_tail_rec_irg);
+}
+
 /*
  * optimize tail recursion away
  */
-void opt_tail_recursion(void) {
-	int i;
-	int n_opt_applications = 0;
-	ir_graph *irg;
+void opt_tail_recursion(void)
+{
+	size_t i, n;
+	size_t n_opt_applications = 0;
 
 	FIRM_DBG_REGISTER(dbg, "firm.opt.tailrec");
 
-	for (i = get_irp_n_irgs() - 1; i >= 0; --i) {
-		irg = get_irp_irg(i);
-
-		current_ir_graph = irg;
+	for (i = 0, n = get_irp_n_irgs(); i < n; ++i) {
+		ir_graph *irg = get_irp_irg(i);
 
 		ir_reserve_resources(irg, IR_RESOURCE_IRN_LINK);
 		if (opt_tail_rec_irg(irg))
@@ -729,6 +731,11 @@ void opt_tail_recursion(void) {
 		ir_free_resources(irg, IR_RESOURCE_IRN_LINK);
 	}
 
-	DB((dbg, LEVEL_1, "Performed tail recursion for %d of %d graphs\n",
+	DB((dbg, LEVEL_1, "Performed tail recursion for %zu of %zu graphs\n",
 	    n_opt_applications, get_irp_n_irgs()));
 }
+
+ir_prog_pass_t *opt_tail_recursion_pass(const char *name)
+{
+	return def_prog_pass(name ? name : "tailrec", opt_tail_recursion);
+}