X-Git-Url: http://nsz.repo.hu/git/?a=blobdiff_plain;f=ir%2Fopt%2Ffunccall.c;h=8b0554f1efb5b0e8c5d5ace1c0ed38bdbfd1d40b;hb=bf9be89cbbf5d6ca1b80fd5736b80f47c1a14e90;hp=1640ddf529d05d1ed577c78d519143822795f47c;hpb=46e024b6e0be49950fba1f24834b7198faac02ec;p=libfirm

diff --git a/ir/opt/funccall.c b/ir/opt/funccall.c
index 1640ddf52..8b0554f1e 100644
--- a/ir/opt/funccall.c
+++ b/ir/opt/funccall.c
@@ -23,13 +23,11 @@
  * @author  Michael Beck
  * @version $Id$
  */
-#ifdef HAVE_CONFIG_H
 #include "config.h"
-#endif
 
 #include <adt/raw_bitset.h>
 
-#include "funccall_t.h"
+#include "opt_init.h"
 
 #include "irnode_t.h"
 #include "irgraph_t.h"
@@ -38,8 +36,11 @@
 #include "irvrfy.h"
 #include "dbginfo_t.h"
 #include "irflag_t.h"
+#include "irloop_t.h"
 #include "ircons.h"
 #include "iredges_t.h"
+#include "irpass_t.h"
+#include "iroptimize.h"
 #include "analyze_irg_args.h"
 #include "irhooks.h"
 #include "debug.h"
@@ -52,10 +53,11 @@ DEBUG_ONLY(static firm_dbg_module_t *dbg;)
 typedef struct _env_t {
 	unsigned n_calls_SymConst;
 	unsigned n_calls_Sel;
-	ir_node  *const_call_list;       /**< The list of all const function calls that will be changed. */
-	ir_node  *pure_call_list;        /**< The list of all pure function calls that will be changed. */
-	ir_node  *nothrow_call_list;     /**< The list of all nothrow function calls that will be changed. */
-	ir_node  *proj_list;             /**< The list of all potential Proj nodes that must be fixed. */
+	ir_node  *float_const_call_list;    /**< The list of all floating const function calls that will be changed. */
+	ir_node  *nonfloat_const_call_list; /**< The list of all non-floating const function calls that will be changed. */
+	ir_node  *pure_call_list;           /**< The list of all pure function calls that will be changed. */
+	ir_node  *nothrow_call_list;        /**< The list of all nothrow function calls that will be changed. */
+	ir_node  *proj_list;                /**< The list of all potential Proj nodes that must be fixed. */
 } env_t;
 
 /** If non-null, evaluates entities for being a heap alloc. */
@@ -78,11 +80,12 @@ static unsigned *busy_set;
  * Walker: Collect all calls to const and pure functions
  * to lists. Collect all Proj(Call) nodes into a Proj list.
  */
-static void collect_const_and_pure_calls(ir_node *node, void *env) {
-	env_t *ctx = env;
-	ir_node *call, *ptr;
+static void collect_const_and_pure_calls(ir_node *node, void *env)
+{
+	env_t     *ctx = env;
+	ir_node   *call, *ptr;
 	ir_entity *ent;
-	unsigned prop;
+	unsigned  and_prop, or_prop, prop;
 
 	if (is_Call(node)) {
 		call = node;
@@ -90,8 +93,8 @@ static void collect_const_and_pure_calls(ir_node *node, void *env) {
 		/* set the link to NULL for all non-const/pure calls */
 		set_irn_link(call, NULL);
 		ptr = get_Call_ptr(call);
-		if (is_SymConst_addr_ent(ptr)) {
-			ent = get_SymConst_entity(ptr);
+		if (is_Global(ptr)) {
+			ent = get_Global_entity(ptr);
 
 			prop = get_entity_additional_properties(ent);
 			if ((prop & (mtp_property_const|mtp_property_pure)) == 0)
@@ -110,17 +113,21 @@ static void collect_const_and_pure_calls(ir_node *node, void *env) {
 			}
 
 			/* note that const function are a subset of pure ones */
-			prop = mtp_property_const | mtp_property_pure;
+			and_prop = mtp_property_const | mtp_property_pure;
+			or_prop  = 0;
 			for (i = 0; i < n_callees; ++i) {
 				ent = get_Call_callee(call, i);
 				if (ent == unknown_entity) {
 					/* we don't know which entity is called here */
 					return;
 				}
-				prop &= get_entity_additional_properties(ent);
-				if (prop == mtp_no_property)
+				prop      = get_entity_additional_properties(ent);
+				and_prop &= prop;
+				or_prop  &= prop;
+				if (and_prop == mtp_no_property)
 					return;
 			}
+			prop = and_prop | (or_prop & mtp_property_has_loop);
 			++ctx->n_calls_Sel;
 		} else
 			return;
@@ -130,8 +137,13 @@ static void collect_const_and_pure_calls(ir_node *node, void *env) {
 			set_irn_link(call, ctx->pure_call_list);
 			ctx->pure_call_list = call;
 		} else {
-			set_irn_link(call, ctx->const_call_list);
-			ctx->const_call_list = call;
+			if (prop & mtp_property_has_loop) {
+				set_irn_link(call, ctx->nonfloat_const_call_list);
+				ctx->nonfloat_const_call_list = call;
+			} else {
+				set_irn_link(call, ctx->float_const_call_list);
+				ctx->float_const_call_list = call;
+			}
 		}
 	} else if (is_Proj(node)) {
 		/*
@@ -144,10 +156,9 @@ static void collect_const_and_pure_calls(ir_node *node, void *env) {
 
 		/* collect the Proj's in the Proj list */
 		switch (get_Proj_proj(node)) {
-		case pn_Call_M_regular:
+		case pn_Call_M:
 		case pn_Call_X_except:
 		case pn_Call_X_regular:
-		case pn_Call_M_except:
 			set_irn_link(node, ctx->proj_list);
 			ctx->proj_list = node;
 			break;
@@ -160,20 +171,21 @@ static void collect_const_and_pure_calls(ir_node *node, void *env) {
 /**
  * Fix the list of collected Calls.
  *
- * @param irg        the graph that contained calls to pure functions
- * @param call_list  the list of all call sites of const functions
- * @param proj_list  the list of all memory/exception Proj's of this call sites
+ * @param irg  the graph that contained calls to pure functions
+ * @param ctx  context
  */
-static void fix_const_call_list(ir_graph *irg, ir_node *call_list, ir_node *proj_list) {
+static void fix_const_call_lists(ir_graph *irg, env_t *ctx)
+{
 	ir_node *call, *next, *mem, *proj;
 	int exc_changed = 0;
 	ir_graph *rem = current_ir_graph;
 
 	current_ir_graph = irg;
 
-	/* First step: fix all calls by removing it's memory input.
-	   It's original memory input is preserved in their link fields. */
-	for (call = call_list; call; call = next) {
+	/* First step: fix all calls by removing their memory input and let
+	 * them floating.
+	 * The original memory input is preserved in their link fields. */
+	for (call = ctx->float_const_call_list; call != NULL; call = next) {
 		next = get_irn_link(call);
 		mem  = get_Call_mem(call);
 
@@ -181,7 +193,7 @@ static void fix_const_call_list(ir_graph *irg, ir_node *call_list, ir_node *proj
 		set_Call_mem(call, get_irg_no_mem(irg));
 
 		/*
-		 * Sorrily we cannot simply set the node to 'float'.
+		 * Unfortunately we cannot simply set the node to 'float'.
 		 * There is a reason for that:
 		 *
 		 * - The call might be inside a loop/if that is NOT entered
@@ -195,38 +207,37 @@ static void fix_const_call_list(ir_graph *irg, ir_node *call_list, ir_node *proj
 		 * observable states...
 		 */
 
-		/* finally, this call can float
-		set_irn_pinned(call, op_pin_state_floats); */
+		/* finally, this call can float */
+		set_irn_pinned(call, op_pin_state_floats);
 		hook_func_call(irg, call);
 	}
 
-	/* Second step: fix all Proj's */
-	for (proj = proj_list; proj; proj = next) {
+	/* Last step: fix all Proj's */
+	for (proj = ctx->proj_list; proj != NULL; proj = next) {
 		next = get_irn_link(proj);
 		call = get_Proj_pred(proj);
 		mem  = get_irn_link(call);
 
 		/* beware of calls in the pure call list */
-		if (! mem || get_irn_op(mem) == op_Call)
+		if (!mem || is_Call(mem))
 			continue;
 		assert(get_irn_mode(mem) == mode_M);
 
 		switch (get_Proj_proj(proj)) {
-		case pn_Call_M_regular: {
+		case pn_Call_M: {
 			/* in dead code there might be cycles where proj == mem */
 			if (proj != mem)
 				exchange(proj, mem);
 			 break;
 		}
 		case pn_Call_X_except:
-		case pn_Call_M_except:
 			exc_changed = 1;
 			exchange(proj, get_irg_bad(irg));
 			break;
 		case pn_Call_X_regular: {
 			ir_node *block = get_nodes_block(call);
 			exc_changed = 1;
-			exchange(proj, new_r_Jmp(irg, block));
+			exchange(proj, new_r_Jmp(block));
 			break;
 		}
 		default:
@@ -249,7 +260,8 @@ static void fix_const_call_list(ir_graph *irg, ir_node *call_list, ir_node *proj
  * Walker: Collect all calls to nothrow functions
  * to lists. Collect all Proj(Call) nodes into a Proj list.
  */
-static void collect_nothrow_calls(ir_node *node, void *env) {
+static void collect_nothrow_calls(ir_node *node, void *env)
+{
 	env_t *ctx = env;
 	ir_node *call, *ptr;
 	ir_entity *ent;
@@ -261,8 +273,8 @@ static void collect_nothrow_calls(ir_node *node, void *env) {
 		/* set the link to NULL for all non-const/pure calls */
 		set_irn_link(call, NULL);
 		ptr = get_Call_ptr(call);
-		if (is_SymConst_addr_ent(ptr)) {
-			ent = get_SymConst_entity(ptr);
+		if (is_Global(ptr)) {
+			ent = get_Global_entity(ptr);
 
 			prop = get_entity_additional_properties(ent);
 			if ((prop & mtp_property_nothrow) == 0)
@@ -310,10 +322,9 @@ static void collect_nothrow_calls(ir_node *node, void *env) {
 
 		/* collect the Proj's in the Proj list */
 		switch (get_Proj_proj(node)) {
-		case pn_Call_M_regular:
+		case pn_Call_M:
 		case pn_Call_X_except:
 		case pn_Call_X_regular:
-		case pn_Call_M_except:
 			set_irn_link(node, ctx->proj_list);
 			ctx->proj_list = node;
 			break;
@@ -330,7 +341,8 @@ static void collect_nothrow_calls(ir_node *node, void *env) {
  * @param call_list  the list of all call sites of const functions
  * @param proj_list  the list of all memory/exception Proj's of this call sites
  */
-static void fix_nothrow_call_list(ir_graph *irg, ir_node *call_list, ir_node *proj_list) {
+static void fix_nothrow_call_list(ir_graph *irg, ir_node *call_list, ir_node *proj_list)
+{
 	ir_node *call, *next, *proj;
 	int exc_changed = 0;
 	ir_graph *rem = current_ir_graph;
@@ -358,14 +370,13 @@ static void fix_nothrow_call_list(ir_graph *irg, ir_node *call_list, ir_node *pr
 		/* kill any exception flow */
 		switch (get_Proj_proj(proj)) {
 		case pn_Call_X_except:
-		case pn_Call_M_except:
 			exc_changed = 1;
 			exchange(proj, get_irg_bad(irg));
 			break;
 		case pn_Call_X_regular: {
 			ir_node *block = get_nodes_block(call);
 			exc_changed = 1;
-			exchange(proj, new_r_Jmp(irg, block));
+			exchange(proj, new_r_Jmp(block));
 			break;
 		}
 		default:
@@ -398,7 +409,8 @@ static unsigned check_nothrow_or_malloc(ir_graph *irg, int top);
 /**
  * Calculate the bigger property of two. Handle the temporary flag right.
  */
-static unsigned max_property(unsigned a, unsigned b) {
+static unsigned max_property(unsigned a, unsigned b)
+{
 	unsigned r, t = (a | b) & mtp_temporary;
 	a &= ~mtp_temporary;
 	b &= ~mtp_temporary;
@@ -414,11 +426,11 @@ static unsigned max_property(unsigned a, unsigned b) {
  * the mtp_property.
  *
  * @return mtp_property_const if only calls of const functions are detected
- *         mtp_property_pure if only Loads and const/pure
- *         calls detected
- *         mtp_no_property else
+ *         mtp_property_pure  if only Loads and const/pure calls detected
+ *         mtp_no_property    else
  */
-static unsigned _follow_mem(ir_node *node) {
+static unsigned _follow_mem(ir_node *node)
+{
 	unsigned m, mode = mtp_property_const;
 	ir_node  *ptr;
 	int i;
@@ -427,11 +439,9 @@ static unsigned _follow_mem(ir_node *node) {
 		if (mode == mtp_no_property)
 			return mtp_no_property;
 
-		if (irn_visited(node))
+		if (irn_visited_else_mark(node))
 			return mode;
 
-		mark_irn_visited(node);
-
 		switch (get_irn_opcode(node)) {
 		case iro_Proj:
 			node = get_Proj_pred(node);
@@ -463,8 +473,7 @@ static unsigned _follow_mem(ir_node *node) {
 		case iro_Call:
 			/* A call is only tolerable if its either constant or pure. */
 			ptr = get_Call_ptr(node);
-			if (get_irn_op(ptr) == op_SymConst &&
-				get_SymConst_kind(ptr) == symconst_addr_ent) {
+			if (is_SymConst_addr_ent(ptr)) {
 				ir_entity *ent = get_SymConst_entity(ptr);
 				ir_graph  *irg = get_entity_irg(ent);
 
@@ -497,7 +506,8 @@ static unsigned _follow_mem(ir_node *node) {
  *         mtp_property_pure  if only Loads and const/pure calls detected
  *         mtp_no_property else
  */
-static unsigned follow_mem(ir_node *node, unsigned mode) {
+static unsigned follow_mem(ir_node *node, unsigned mode)
+{
 	unsigned m;
 
 	m = _follow_mem(node);
@@ -510,7 +520,8 @@ static unsigned follow_mem(ir_node *node, unsigned mode) {
  * @param irg  the graph to check
  * @param top  if set, this is the top call
  */
-static unsigned check_const_or_pure_function(ir_graph *irg, int top) {
+static unsigned check_const_or_pure_function(ir_graph *irg, int top)
+{
 	ir_node *end, *endbl;
 	int j;
 	unsigned prop = get_irg_additional_properties(irg);
@@ -542,8 +553,9 @@ static unsigned check_const_or_pure_function(ir_graph *irg, int top) {
 
 	current_ir_graph = irg;
 
+	ir_reserve_resources(irg, IR_RESOURCE_IRN_VISITED);
 	inc_irg_visited(irg);
-	/* mark the initial mem: recursion of follow_mem stops here */
+	/* mark the initial mem: recursion of follow_mem() stops here */
 	mark_irn_visited(get_irg_initial_mem(irg));
 
 	/* visit every Return */
@@ -577,12 +589,17 @@ static unsigned check_const_or_pure_function(ir_graph *irg, int top) {
 	if (prop != mtp_no_property) {
 		/* check, if a keep-alive exists */
 		for (j = get_End_n_keepalives(end) - 1; j >= 0; --j) {
-			ir_node *mem = get_End_keepalive(end, j);
+			ir_node *kept = get_End_keepalive(end, j);
+
+			if (is_Block(kept)) {
+				prop = mtp_no_property;
+				break;
+			}
 
-			if (mode_M != get_irn_mode(mem))
+			if (mode_M != get_irn_mode(kept))
 				continue;
 
-			prop = max_property(prop, follow_mem(mem, prop));
+			prop = max_property(prop, follow_mem(kept, prop));
 			if (prop == mtp_no_property)
 				break;
 		}
@@ -600,6 +617,7 @@ static unsigned check_const_or_pure_function(ir_graph *irg, int top) {
 	if (top)
 		SET_IRG_READY(irg);
 	CLEAR_IRG_BUSY(irg);
+	ir_free_resources(irg, IR_RESOURCE_IRN_VISITED);
 	current_ir_graph = rem;
 	return prop;
 }  /* check_const_or_pure_function */
@@ -609,7 +627,8 @@ static unsigned check_const_or_pure_function(ir_graph *irg, int top) {
  *
  * @param ctx  context
  */
-static void handle_const_Calls(env_t *ctx) {
+static void handle_const_Calls(env_t *ctx)
+{
 	int i;
 
 	ctx->n_calls_SymConst = 0;
@@ -619,18 +638,17 @@ static void handle_const_Calls(env_t *ctx) {
 	for (i = get_irp_n_irgs() - 1; i >= 0; --i) {
 		ir_graph *irg  = get_irp_irg(i);
 
-		ctx->const_call_list = NULL;
-		ctx->pure_call_list  = NULL;
-		ctx->proj_list = NULL;
-		irg_walk_graph(irg, NULL, collect_const_and_pure_calls, ctx);
+		ctx->float_const_call_list    = NULL;
+		ctx->nonfloat_const_call_list = NULL;
+		ctx->pure_call_list           = NULL;
+		ctx->proj_list                = NULL;
 
-		if (ctx->const_call_list) {
-			fix_const_call_list(irg, ctx->const_call_list, ctx->proj_list);
+		ir_reserve_resources(irg, IR_RESOURCE_IRN_LINK);
+		irg_walk_graph(irg, NULL, collect_const_and_pure_calls, ctx);
 
-			/* this graph was changed, invalidate analysis info */
-			set_irg_outs_inconsistent(irg);
-			set_irg_doms_inconsistent(irg);
-		}
+		if (ctx->float_const_call_list != NULL)
+			fix_const_call_lists(irg, ctx);
+		ir_free_resources(irg, IR_RESOURCE_IRN_LINK);
 	}
 }  /* handle_const_Calls */
 
@@ -639,7 +657,8 @@ static void handle_const_Calls(env_t *ctx) {
  *
  * @param ctx  context
  */
-static void handle_nothrow_Calls(env_t *ctx) {
+static void handle_nothrow_Calls(env_t *ctx)
+{
 	int i;
 
 	ctx->n_calls_SymConst = 0;
@@ -651,15 +670,13 @@ static void handle_nothrow_Calls(env_t *ctx) {
 
 		ctx->nothrow_call_list = NULL;
 		ctx->proj_list         = NULL;
+
+		ir_reserve_resources(irg, IR_RESOURCE_IRN_LINK);
 		irg_walk_graph(irg, NULL, collect_nothrow_calls, ctx);
 
-		if (ctx->nothrow_call_list) {
+		if (ctx->nothrow_call_list)
 			fix_nothrow_call_list(irg, ctx->nothrow_call_list, ctx->proj_list);
-
-			/* this graph was changed, invalidate analysis info */
-			set_irg_outs_inconsistent(irg);
-			set_irg_doms_inconsistent(irg);
-		}
+		ir_free_resources(irg, IR_RESOURCE_IRN_LINK);
 	}
 }
 
@@ -669,7 +686,8 @@ static void handle_nothrow_Calls(env_t *ctx) {
  *
  * @param node  the node to check
  */
-static int is_malloc_call_result(const ir_node *node) {
+static int is_malloc_call_result(const ir_node *node)
+{
 	if (is_Alloc(node) && get_Alloc_where(node) == heap_alloc) {
 		/* Firm style high-level allocation */
 		return 1;
@@ -677,8 +695,8 @@ static int is_malloc_call_result(const ir_node *node) {
 	if (is_alloc_entity != NULL && is_Call(node)) {
 		ir_node *ptr = get_Call_ptr(node);
 
-		if (is_SymConst_addr_ent(ptr)) {
-			ir_entity *ent = get_SymConst_entity(ptr);
+		if (is_Global(ptr)) {
+			ir_entity *ent = get_Global_entity(ptr);
 			return is_alloc_entity(ent);
 		}
 	}
@@ -688,7 +706,8 @@ static int is_malloc_call_result(const ir_node *node) {
 /**
  * Update a property depending on a call property.
  */
-static unsigned update_property(unsigned orig_prop, unsigned call_prop) {
+static unsigned update_property(unsigned orig_prop, unsigned call_prop)
+{
 	unsigned t = (orig_prop | call_prop) & mtp_temporary;
 	unsigned r = orig_prop & call_prop;
 	return r | t;
@@ -697,7 +716,8 @@ static unsigned update_property(unsigned orig_prop, unsigned call_prop) {
 /**
  * Check if a node is stored.
  */
-static int is_stored(const ir_node *n) {
+static int is_stored(const ir_node *n)
+{
 	const ir_edge_t *edge;
 	const ir_node   *ptr;
 
@@ -712,19 +732,19 @@ static int is_stored(const ir_node *n) {
 			break;
 		case iro_Store:
 			if (get_Store_value(succ) == n)
-				return 0;
+				return 1;
 			/* ok if its only the address input */
 			break;
 		case iro_Sel:
 		case iro_Cast:
 		case iro_Confirm:
 			if (is_stored(succ))
-				return 0;
+				return 1;
 			break;
 		case iro_Call:
 			ptr = get_Call_ptr(succ);
-			if (is_SymConst_addr_ent(ptr)) {
-				ir_entity *ent = get_SymConst_entity(ptr);
+			if (is_Global(ptr)) {
+				ir_entity *ent = get_Global_entity(ptr);
 				int       i;
 
 				/* we know the called entity */
@@ -733,19 +753,21 @@ static int is_stored(const ir_node *n) {
 						/* n is the i'th param of the call */
 						if (get_method_param_access(ent, i) & ptr_access_store) {
 							/* n is store in ent */
-							return 0;
+							return 1;
 						}
 					}
 				}
 			} else {
-				return 0;
+				/* unknown call address */
+				return 1;
 			}
 			break;
 		default:
 			/* bad, potential alias */
-			return 0;
+			return 1;
 		}
 	}
+	return 0;
 }  /* is_stored */
 
 /**
@@ -753,7 +775,8 @@ static int is_stored(const ir_node *n) {
  *
  * return ~mtp_property_malloc if return values are stored, ~0 else
  */
-static unsigned check_stored_result(ir_graph *irg) {
+static unsigned check_stored_result(ir_graph *irg)
+{
 	ir_node  *end_blk = get_irg_end_block(irg);
 	int      i, j;
 	unsigned res = ~0;
@@ -765,9 +788,9 @@ static unsigned check_stored_result(ir_graph *irg) {
 		if (! is_Return(pred))
 			continue;
 		for (j = get_Return_n_ress(pred) - 1; j >= 0; --j) {
-			const ir_node *res = get_Return_res(pred, j);
+			const ir_node *irn = get_Return_res(pred, j);
 
-			if (is_stored(res)) {
+			if (is_stored(irn)) {
 				/* bad, might create an alias */
 				res = ~mtp_property_malloc;
 				goto finish;
@@ -786,7 +809,8 @@ finish:
  * @param irg  the graph to check
  * @param top  if set, this is the top call
  */
-static unsigned check_nothrow_or_malloc(ir_graph *irg, int top) {
+static unsigned check_nothrow_or_malloc(ir_graph *irg, int top)
+{
 	ir_node   *end_blk = get_irg_end_block(irg);
 	ir_entity *ent;
 	ir_type   *mtp;
@@ -829,9 +853,9 @@ static unsigned check_nothrow_or_malloc(ir_graph *irg, int top) {
 					} else if (is_Call(res)) {
 						ir_node *ptr = get_Call_ptr(res);
 
-						if (is_SymConst_addr_ent(ptr)) {
+						if (is_Global(ptr)) {
 							/* a direct call */
-							ir_entity *ent    = get_SymConst_entity(ptr);
+							ir_entity *ent    = get_Global_entity(ptr);
 							ir_graph  *callee = get_entity_irg(ent);
 
 							if (callee == irg) {
@@ -885,9 +909,9 @@ static unsigned check_nothrow_or_malloc(ir_graph *irg, int top) {
 			if (is_Call(pred)) {
 				ir_node *ptr = get_Call_ptr(pred);
 
-				if (is_SymConst_addr_ent(ptr)) {
+				if (is_Global(ptr)) {
 					/* a direct call */
-					ir_entity *ent    = get_SymConst_entity(ptr);
+					ir_entity *ent    = get_Global_entity(ptr);
 					ir_graph  *callee = get_entity_irg(ent);
 
 					if (callee == irg) {
@@ -966,6 +990,20 @@ static unsigned check_nothrow_or_malloc(ir_graph *irg, int top) {
 	return curr_prop;
 }  /* check_nothrow_or_malloc */
 
+/**
+ * When a function was detected as "const", it might be moved out of loops.
+ * This might be dangerous if the graph can contain endless loops.
+ */
+static void check_for_possible_endless_loops(ir_graph *irg)
+{
+	ir_loop *root_loop;
+	assure_cf_loop(irg);
+
+	root_loop = get_irg_loop(irg);
+	if (root_loop->flags & loop_outer_loop)
+		set_irg_additional_property(irg, mtp_property_has_loop);
+}
+
 /*
  * optimize function calls by handling const functions
  */
@@ -980,7 +1018,7 @@ void optimize_funccalls(int force_run, check_alloc_entity_func callback)
 	is_alloc_entity = callback;
 
 	/* prepare: mark all graphs as not analyzed */
-	last_idx = get_irp_last_idx();
+	last_idx  = get_irp_last_idx();
 	ready_set = rbitset_malloc(last_idx);
 	busy_set  = rbitset_malloc(last_idx);
 
@@ -1024,6 +1062,7 @@ void optimize_funccalls(int force_run, check_alloc_entity_func callback)
 		if (prop & mtp_property_const) {
 			++num_const;
 			DB((dbg, LEVEL_2, "%+F has the const property\n", irg));
+			check_for_possible_endless_loops(irg);
 		} else if (prop & mtp_property_pure) {
 			++num_pure;
 			DB((dbg, LEVEL_2, "%+F has the pure property\n", irg));
@@ -1045,7 +1084,39 @@ void optimize_funccalls(int force_run, check_alloc_entity_func callback)
 }  /* optimize_funccalls */
 
 /* initialize the funccall optimization */
-void firm_init_funccalls(void) {
+void firm_init_funccalls(void)
+{
 	FIRM_DBG_REGISTER(dbg, "firm.opt.funccalls");
-//	firm_dbg_set_mask(dbg, -1);
 }  /* firm_init_funccalls */
+
+struct pass_t {
+	ir_prog_pass_t          pass;
+	int                     force_run;
+	check_alloc_entity_func callback;
+};
+
+/**
+ * Wrapper for running optimize_funccalls() as an ir_prog pass.
+ */
+static int pass_wrapper(ir_prog *irp, void *context)
+{
+	struct pass_t *pass = context;
+
+	(void)irp;
+	optimize_funccalls(pass->force_run, pass->callback);
+	return 0;
+}  /* pass_wrapper */
+
+/* Creates an ir_prog pass for optimize_funccalls. */
+ir_prog_pass_t *optimize_funccalls_pass(
+	const char *name,
+	int force_run, check_alloc_entity_func callback)
+{
+	struct pass_t *pass = XMALLOCZ(struct pass_t);
+
+	pass->force_run = force_run;
+	pass->callback  = callback;
+
+	return def_prog_pass_constructor(
+		&pass->pass, name ? name : "funccall", pass_wrapper);
+}  /* optimize_funccalls_pass */