X-Git-Url: http://nsz.repo.hu/git/?a=blobdiff_plain;f=ir%2Fopt%2Fopt_osr.c;h=7144e18fed36bf2cc8c772549e11fdaccd1c5392;hb=c7dc950ac0cdd7d24acffb798b5867d0db5dd7c8;hp=43734a102300f765d583e8f856774d189aeddd45;hpb=c0fd951c4803e9ee0450f427f42f5354ca340f5b;p=libfirm

diff --git a/ir/opt/opt_osr.c b/ir/opt/opt_osr.c
index 43734a102..7144e18fe 100644
--- a/ir/opt/opt_osr.c
+++ b/ir/opt/opt_osr.c
@@ -322,24 +322,25 @@ static ir_node *do_apply(ir_opcode code, dbg_info *db, ir_node *op1, ir_node *op
  *
  * @return the newly created node
  */
-static ir_node *apply(ir_node *orig, ir_node *op1, ir_node *op2, iv_env *env) {
+static ir_node *apply(ir_node *header, ir_node *orig, ir_node *op1, ir_node *op2, iv_env *env) {
 	ir_opcode code = get_irn_opcode(orig);
 	ir_node *result = search(code, op1, op2, env);
 
-	if (! result) {
+	if (result == NULL) {
 		dbg_info *db = get_irn_dbg_info(orig);
 		ir_node *op1_header = get_irn_ne(op1, env)->header;
 		ir_node *op2_header = get_irn_ne(op2, env)->header;
 
-		if (op1_header != NULL && is_rc(op2, op1_header)) {
+		if (op1_header == header && is_rc(op2, op1_header)) {
 			result = reduce(orig, op1, op2, env);
 		}
-		else if (op2_header != NULL && is_rc(op1, op2_header)) {
+		else if (op2_header == header && is_rc(op1, op2_header)) {
 			result = reduce(orig, op2, op1, env);
 		}
 		else {
 			result = do_apply(code, db, op1, op2, get_irn_mode(orig));
-			get_irn_ne(result, env)->header = NULL;		}
+			get_irn_ne(result, env)->header = NULL;
+		}
 	}
 	return result;
 }
@@ -359,7 +360,7 @@ static ir_node *reduce(ir_node *orig, ir_node *iv, ir_node *rc, iv_env *env) {
 	ir_opcode code = get_irn_opcode(orig);
 	ir_node *result = search(code, iv, rc, env);
 
-	if (! result) {
+	if (result == NULL) {
 		node_entry *e, *iv_e;
 		int i, n;
 		ir_mode *mode = get_irn_mode(orig);
@@ -390,7 +391,7 @@ static ir_node *reduce(ir_node *orig, ir_node *iv, ir_node *rc, iv_env *env) {
 			if (e->header == iv_e->header)
 				o = reduce(orig, o, rc, env);
 			else if (is_Phi(result) || code == iro_Mul)
-				o = apply(orig, o, rc, env);
+				o = apply(iv_e->header, orig, o, rc, env);
 			set_irn_n(result, i, o);
 		}
 	}
@@ -409,6 +410,7 @@ static void update_scc(ir_node *iv, node_entry *e, iv_env *env) {
 	ir_node *header = e->header;
 	waitq    *wq = new_waitq();
 
+	DB((dbg, LEVEL_2, "  Creating SCC for new an induction variable:\n  "));
 	pscc->head = NULL;
 	waitq_put(wq, iv);
 	do {
@@ -419,16 +421,21 @@ static void update_scc(ir_node *iv, node_entry *e, iv_env *env) {
 		ne->pscc   = pscc;
 		ne->next   = pscc->head;
 		pscc->head = irn;
+		DB((dbg, LEVEL_2, " %+F,", irn));
 
 		for (i = get_irn_arity(irn) - 1; i >= 0; --i) {
 			ir_node    *pred = get_irn_n(irn, i);
 			node_entry *pe   = get_irn_ne(pred, env);
 
-			if (pe->header == header && pe->pscc == NULL)
+			if (pe->header == header && pe->pscc == NULL) {
+				/* set the pscc here to ensure that the node is NOT enqueued another time */
+				pe->pscc = pscc;
 				waitq_put(wq, pred);
+			}
 		}
 	} while (! waitq_empty(wq));
 	del_waitq(wq);
+	DB((dbg, LEVEL_2, "\n"));
 }
 
 /**
@@ -462,6 +469,7 @@ static int replace(ir_node *irn, ir_node *iv, ir_node *rc, iv_env *env) {
 	return 0;
 }
 
+#if 0
 /**
  * check if a given node is a mul with 2, 4, 8
  */
@@ -487,6 +495,7 @@ static int is_x86_shift_const(ir_node *mul) {
 	}
 	return 0;
 }
+#endif
 
 /**
  * Check if an IV represents a counter with constant limits.
@@ -655,11 +664,6 @@ static int check_replace(ir_node *irn, iv_env *env) {
 				if (! check_users_for_reg_pressure(iv, env))
 					return 0;
 			}
-			/* check for x86 constants */
-			if (env->flags & osr_flag_ignore_x86_shift)
-				if (is_x86_shift_const(irn))
-					return 0;
-
 			return replace(irn, iv, rc, env);
 		}
 		break;
@@ -1259,7 +1263,6 @@ void opt_osr(ir_graph *irg, unsigned flags) {
 	current_ir_graph = irg;
 
 	FIRM_DBG_REGISTER(dbg, "firm.opt.osr");
-	firm_dbg_set_mask(dbg, -1);
 
 	DB((dbg, LEVEL_1, "Doing Operator Strength Reduction for %+F\n", irg));
 
@@ -1294,7 +1297,8 @@ void opt_osr(ir_graph *irg, unsigned flags) {
 
 	if (env.replaced) {
 		/* try linear function test replacements */
-		lftr(irg, &env);
+		//lftr(irg, &env); // currently buggy :-(
+		(void) lftr;
 
 		set_irg_outs_inconsistent(irg);
 		DB((dbg, LEVEL_1, "Replacements: %u + %u (lftr)\n\n", env.replaced, env.lftr_replaced));