* The walker environment for updating function calls.
*/
typedef struct env_t {
- size_t n_calls_SymConst;
- size_t n_calls_Sel;
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. */
*/
static void collect_const_and_pure_calls(ir_node *node, void *env)
{
- env_t *ctx = (env_t*)env;
- ir_node *call;
- ir_node *ptr;
- ir_entity *ent;
- unsigned prop;
+ env_t *ctx = (env_t*)env;
if (is_Call(node)) {
- call = node;
+ ir_node *call = node;
/* 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);
-
- prop = get_entity_additional_properties(ent);
- if ((prop & (mtp_property_const|mtp_property_pure)) == 0)
- return;
- ++ctx->n_calls_SymConst;
- } else {
+ ir_node *ptr = get_Call_ptr(call);
+ if (!is_SymConst_addr_ent(ptr))
+ return;
+
+ ir_entity *ent = get_SymConst_entity(ptr);
+
+ unsigned prop = get_entity_additional_properties(ent);
+ if ((prop & (mtp_property_const|mtp_property_pure)) == 0)
return;
- }
/* ok, if we get here we found a call to a const or a pure function */
if (prop & mtp_property_pure) {
* Collect all memory and exception Proj's from
* calls.
*/
- call = get_Proj_pred(node);
- if (! is_Call(call))
+ ir_node *call = get_Proj_pred(node);
+ if (!is_Call(call))
return;
/* collect the Proj's in the Proj list */
break;
}
}
-} /* collect_const_and_pure_calls */
+}
/**
* Fix the list of collected Calls.
*/
static void fix_const_call_lists(ir_graph *irg, env_t *ctx)
{
- ir_node *call, *next, *mem, *proj;
- int exc_changed = 0;
+ bool exc_changed = false;
/* 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) {
+ ir_node *next;
+ for (ir_node *call = ctx->float_const_call_list; call != NULL; call = next) {
next = (ir_node*)get_irn_link(call);
- mem = get_Call_mem(call);
+ ir_node *mem = get_Call_mem(call);
set_irn_link(call, mem);
set_Call_mem(call, get_irg_no_mem(irg));
}
/* Last step: fix all Proj's */
- for (proj = ctx->proj_list; proj != NULL; proj = next) {
+ for (ir_node *proj = ctx->proj_list; proj != NULL; proj = next) {
next = (ir_node*)get_irn_link(proj);
- call = get_Proj_pred(proj);
- mem = (ir_node*)get_irn_link(call);
+ ir_node *call = get_Proj_pred(proj);
+ ir_node *mem = (ir_node*)get_irn_link(call);
/* beware of calls in the pure call list */
if (!mem || is_Call(mem))
break;
}
case pn_Call_X_except:
- exc_changed = 1;
+ exc_changed = true;
exchange(proj, new_r_Bad(irg, mode_X));
break;
case pn_Call_X_regular: {
ir_node *block = get_nodes_block(call);
- exc_changed = 1;
+ exc_changed = true;
exchange(proj, new_r_Jmp(block));
break;
}
clear_irg_properties(irg, IR_GRAPH_PROPERTY_CONSISTENT_DOMINANCE
| IR_GRAPH_PROPERTY_CONSISTENT_LOOPINFO);
}
-} /* fix_const_call_list */
+}
/**
* Walker: Collect all calls to nothrow functions
static void collect_nothrow_calls(ir_node *node, void *env)
{
env_t *ctx = (env_t*)env;
- ir_node *call, *ptr;
- ir_entity *ent;
- unsigned prop;
if (is_Call(node)) {
- call = node;
+ ir_node *call = node;
/* 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);
-
- prop = get_entity_additional_properties(ent);
- if ((prop & mtp_property_nothrow) == 0)
- return;
- ++ctx->n_calls_SymConst;
- } else {
+ ir_node *ptr = get_Call_ptr(call);
+ if (!is_SymConst_addr_ent(ptr))
+ return;
+
+ ir_entity *ent = get_SymConst_entity(ptr);
+
+ unsigned prop = get_entity_additional_properties(ent);
+ if ((prop & mtp_property_nothrow) == 0)
return;
- }
/* ok, if we get here we found a call to a nothrow function */
set_irn_link(call, ctx->nothrow_call_list);
* Collect all memory and exception Proj's from
* calls.
*/
- call = get_Proj_pred(node);
+ ir_node *call = get_Proj_pred(node);
if (! is_Call(call))
return;
break;
}
}
-} /* collect_nothrow_calls */
+}
/**
* Fix the list of collected nothrow Calls.
* @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;
+ bool exc_changed = false;
/* First step: go through the list of calls and mark them. */
- for (call = call_list; call; call = next) {
+ ir_node *next;
+ for (ir_node *call = call_list; call; call = next) {
next = (ir_node*)get_irn_link(call);
/* current_ir_graph is in memory anyway, so it's a good marker */
}
/* Second step: Remove all exception Proj's */
- for (proj = proj_list; proj; proj = next) {
+ for (ir_node *proj = proj_list; proj; proj = next) {
next = (ir_node*)get_irn_link(proj);
- call = get_Proj_pred(proj);
+ ir_node *call = get_Proj_pred(proj);
/* handle only marked calls */
if (get_irn_link(call) != ¤t_ir_graph)
/* kill any exception flow */
switch (get_Proj_proj(proj)) {
case pn_Call_X_except:
- exc_changed = 1;
+ exc_changed = true;
exchange(proj, new_r_Bad(irg, mode_X));
break;
case pn_Call_X_regular: {
ir_node *block = get_nodes_block(call);
- exc_changed = 1;
+ exc_changed = true;
exchange(proj, new_r_Jmp(block));
break;
}
clear_irg_properties(irg, IR_GRAPH_PROPERTY_CONSISTENT_DOMINANCE
| IR_GRAPH_PROPERTY_CONSISTENT_LOOPINFO);
}
-} /* fix_nothrow_call_list */
+}
/* marking */
#define SET_IRG_READY(irg) rbitset_set(ready_set, get_irg_idx(irg))
#define IS_IRG_BUSY(irg) rbitset_is_set(busy_set, get_irg_idx(irg))
/* forward */
-static mtp_additional_properties check_const_or_pure_function(ir_graph *irg, int top);
+static mtp_additional_properties check_const_or_pure_function(ir_graph *irg, bool top);
/**
* Calculate the bigger property of two. Handle the temporary flag right.
static mtp_additional_properties max_property(mtp_additional_properties a,
mtp_additional_properties b)
{
- mtp_additional_properties r;
mtp_additional_properties t = (a | b) & mtp_temporary;
a &= ~mtp_temporary;
b &= ~mtp_temporary;
if (a == mtp_no_property || b == mtp_no_property)
return mtp_no_property;
- r = a > b ? a : b;
+ mtp_additional_properties r = a > b ? a : b;
return r | t;
-} /* max_property */
+}
/**
* Follow the memory chain starting at node and determine
static mtp_additional_properties follow_mem_(ir_node *node)
{
mtp_additional_properties mode = mtp_property_const;
- mtp_additional_properties m;
- ir_node *ptr;
- int i;
for (;;) {
if (mode == mtp_no_property)
case iro_Phi:
case iro_Sync:
/* do a dfs search */
- for (i = get_irn_arity(node) - 1; i >= 0; --i) {
- m = follow_mem_(get_irn_n(node, i));
+ for (int i = get_irn_arity(node) - 1; i >= 0; --i) {
+ mtp_additional_properties m = follow_mem_(get_irn_n(node, i));
mode = max_property(mode, m);
if (mode == mtp_no_property)
return mtp_no_property;
node = get_Load_mem(node);
break;
- case iro_Call:
+ case iro_Call: {
/* A call is only tolerable if its either constant or pure. */
- ptr = get_Call_ptr(node);
- if (is_SymConst_addr_ent(ptr)) {
- ir_entity *ent = get_SymConst_entity(ptr);
- ir_graph *irg = get_entity_irg(ent);
-
- if (irg == NULL) {
- m = get_entity_additional_properties(ent) & (mtp_property_const|mtp_property_pure);
- mode = max_property(mode, m);
- } else {
- /* we have a graph, analyze it. */
- m = check_const_or_pure_function(irg, /*top=*/0);
- mode = max_property(mode, m);
- }
- } else
+ ir_node *ptr = get_Call_ptr(node);
+ if (!is_SymConst_addr_ent(ptr))
return mtp_no_property;
+
+ ir_entity *ent = get_SymConst_entity(ptr);
+ ir_graph *irg = get_entity_irg(ent);
+
+ mtp_additional_properties m;
+ if (irg == NULL) {
+ m = get_entity_additional_properties(ent) & (mtp_property_const|mtp_property_pure);
+ mode = max_property(mode, m);
+ } else {
+ /* we have a graph, analyze it. */
+ m = check_const_or_pure_function(irg, false);
+ mode = max_property(mode, m);
+ }
node = get_Call_mem(node);
break;
+ }
default:
return mtp_no_property;
* @param irg the graph to check
* @param top if set, this is the top call
*/
-static mtp_additional_properties check_const_or_pure_function(ir_graph *irg, int top)
+static mtp_additional_properties check_const_or_pure_function(ir_graph *irg, bool top)
{
- ir_node *end, *endbl;
- int j;
ir_entity *entity = get_irg_entity(irg);
ir_type *type = get_entity_type(entity);
size_t n_params = get_method_n_params(type);
- size_t i;
mtp_additional_properties may_be_const = mtp_property_const;
mtp_additional_properties prop = get_irg_additional_properties(irg);
/* libfirm handles aggregate parameters by passing around pointers to
* stuff in memory, so if we have compound parameters we are never const */
- for (i = 0; i < n_params; ++i) {
+ for (size_t i = 0; i < n_params; ++i) {
ir_type *param = get_method_param_type(type, i);
if (is_compound_type(param)) {
prop &= ~mtp_property_const;
}
if (IS_IRG_BUSY(irg)) {
/* We are still evaluate this method.
- * The function (indirectly) calls itself and thus may not terminate.
- */
+ * The function (indirectly) calls itself and thus may not terminate. */
return mtp_no_property;
}
SET_IRG_BUSY(irg);
- end = get_irg_end(irg);
- endbl = get_nodes_block(end);
- prop = may_be_const;
+ ir_node *end = get_irg_end(irg);
+ ir_node *endbl = get_nodes_block(end);
+ prop = may_be_const;
ir_reserve_resources(irg, IR_RESOURCE_IRN_VISITED);
inc_irg_visited(irg);
mark_irn_visited(get_irg_initial_mem(irg));
/* visit every Return */
- for (j = get_Block_n_cfgpreds(endbl) - 1; j >= 0; --j) {
- ir_node *node = get_Block_cfgpred(endbl, j);
- unsigned code = get_irn_opcode(node);
- ir_node *mem;
+ for (int j = get_Block_n_cfgpreds(endbl) - 1; j >= 0; --j) {
+ ir_node *node = get_Block_cfgpred(endbl, j);
+ unsigned code = get_irn_opcode(node);
/* Bad nodes usually do NOT produce anything, so it's ok */
if (code == iro_Bad)
continue;
if (code == iro_Return) {
- mem = get_Return_mem(node);
+ ir_node *mem = get_Return_mem(node);
/* Bad nodes usually do NOT produce anything, so it's ok */
if (is_Bad(mem))
if (prop != mtp_no_property) {
/* check, if a keep-alive exists */
- for (j = get_End_n_keepalives(end) - 1; j >= 0; --j) {
+ for (int j = get_End_n_keepalives(end) - 1; j >= 0; --j) {
ir_node *kept = get_End_keepalive(end, j);
if (is_Block(kept)) {
CLEAR_IRG_BUSY(irg);
ir_free_resources(irg, IR_RESOURCE_IRN_VISITED);
return prop;
-} /* check_const_or_pure_function */
+}
/**
* Handle calls to const functions.
*/
static void handle_const_Calls(env_t *ctx)
{
- size_t i, n;
-
- ctx->n_calls_SymConst = 0;
- ctx->n_calls_Sel = 0;
-
/* all calls of const functions can be transformed */
- for (i = 0, n = get_irp_n_irgs(); i < n; ++i) {
- ir_graph *irg = get_irp_irg(i);
+ size_t n = get_irp_n_irgs();
+ for (size_t i = 0; i < n; ++i) {
+ ir_graph *irg = get_irp_irg(i);
ctx->float_const_call_list = NULL;
ctx->nonfloat_const_call_list = NULL;
ir_reserve_resources(irg, IR_RESOURCE_IRN_LINK);
irg_walk_graph(irg, NULL, collect_const_and_pure_calls, ctx);
-
- if (ctx->float_const_call_list != NULL)
- fix_const_call_lists(irg, ctx);
+ fix_const_call_lists(irg, ctx);
ir_free_resources(irg, IR_RESOURCE_IRN_LINK);
confirm_irg_properties(irg,
*/
static void handle_nothrow_Calls(env_t *ctx)
{
- size_t i, n;
-
- ctx->n_calls_SymConst = 0;
- ctx->n_calls_Sel = 0;
-
/* all calls of const functions can be transformed */
- for (i = 0, n = get_irp_n_irgs(); i < n; ++i) {
+ size_t n = get_irp_n_irgs();
+ for (size_t i = 0; i < n; ++i) {
ir_graph *irg = get_irp_irg(i);
ctx->nothrow_call_list = NULL;
*
* @param node the node to check
*/
-static int is_malloc_call_result(const ir_node *node)
+static bool 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;
+ return true;
}
/* TODO: check mtp_malloc */
- return 0;
+ return false;
}
/**
*/
static mtp_additional_properties check_stored_result(ir_graph *irg)
{
- ir_node *end_blk = get_irg_end_block(irg);
- int i;
+ ir_node *end_blk = get_irg_end_block(irg);
mtp_additional_properties res = ~mtp_no_property;
assure_irg_properties(irg, IR_GRAPH_PROPERTY_CONSISTENT_OUT_EDGES);
- for (i = get_Block_n_cfgpreds(end_blk) - 1; i >= 0; --i) {
+ for (int i = get_Block_n_cfgpreds(end_blk) - 1; i >= 0; --i) {
ir_node *pred = get_Block_cfgpred(end_blk, i);
- size_t j;
if (! is_Return(pred))
continue;
- for (j = get_Return_n_ress(pred); j > 0;) {
+ for (size_t j = get_Return_n_ress(pred); j > 0;) {
const ir_node *irn = get_Return_res(pred, --j);
if (is_stored(irn)) {
* @param irg the graph to check
* @param top if set, this is the top call
*/
-static mtp_additional_properties check_nothrow_or_malloc(ir_graph *irg, int top)
+static mtp_additional_properties check_nothrow_or_malloc(ir_graph *irg, bool top)
{
- mtp_additional_properties curr_prop = mtp_property_malloc | mtp_property_nothrow;
- ir_node *end_blk = get_irg_end_block(irg);
- ir_entity *ent;
- ir_type *mtp;
- int i;
+ mtp_additional_properties curr_prop
+ = mtp_property_malloc | mtp_property_nothrow;
if (IS_IRG_READY(irg)) {
/* already checked */
}
SET_IRG_BUSY(irg);
- ent = get_irg_entity(irg);
- mtp = get_entity_type(ent);
+ ir_entity *ent = get_irg_entity(irg);
+ ir_type *mtp = get_entity_type(ent);
if (get_method_n_ress(mtp) <= 0)
curr_prop &= ~mtp_property_malloc;
- for (i = get_Block_n_cfgpreds(end_blk) - 1; i >= 0; --i) {
+ ir_node *end_blk = get_irg_end_block(irg);
+ for (int i = get_Block_n_cfgpreds(end_blk) - 1; i >= 0; --i) {
ir_node *pred = get_Block_cfgpred(end_blk, i);
if (is_Return(pred)) {
if (curr_prop & mtp_property_malloc) {
- size_t j;
-
/* check, if malloc is called here */
- for (j = get_Return_n_ress(pred); j > 0;) {
+ for (size_t j = get_Return_n_ress(pred); j > 0;) {
ir_node *res = get_Return_res(pred, --j);
/* skip Confirms and Casts */
if (callee == irg) {
/* A self-recursive call. The property did not depend on this call. */
} else if (callee != NULL) {
- mtp_additional_properties prop = check_nothrow_or_malloc(callee, /*top=*/0);
+ mtp_additional_properties prop = check_nothrow_or_malloc(callee, false);
curr_prop = update_property(curr_prop, prop);
} else {
curr_prop = update_property(curr_prop, get_entity_additional_properties(ent));
/* A self-recursive call. The property did not depend on this call. */
} else if (callee != NULL) {
/* Note: we check here for nothrow only, so do NOT reset the malloc property */
- mtp_additional_properties prop = check_nothrow_or_malloc(callee, /*top=*/0) | mtp_property_malloc;
+ mtp_additional_properties prop = check_nothrow_or_malloc(callee, false) | mtp_property_malloc;
curr_prop = update_property(curr_prop, prop);
} else {
if ((get_entity_additional_properties(ent) & mtp_property_nothrow) == 0)
}
if (curr_prop & mtp_property_malloc) {
- /*
- * Note that the malloc property means not only return newly allocated
+ /* Note that the malloc property means not only return newly allocated
* memory, but also that this memory is ALIAS FREE.
* To ensure that, we do NOT allow that the returned memory is somewhere
- * stored.
- */
+ * stored. */
curr_prop &= check_stored_result(irg);
}
- if (curr_prop != mtp_no_property) {
- if (top || (curr_prop & mtp_temporary) == 0) {
- /* We use the temporary flag here to mark an optimistic result.
- Set the property only if we are sure that it does NOT base on
- temporary results OR if we are at top-level. */
- add_irg_additional_properties(irg, curr_prop & ~mtp_temporary);
- SET_IRG_READY(irg);
- }
+ if (curr_prop != mtp_no_property
+ && (top || (curr_prop & mtp_temporary) == 0)) {
+ /* We use the temporary flag here to mark an optimistic result.
+ * Set the property only if we are sure that it does NOT base on
+ * temporary results OR if we are at top-level. */
+ add_irg_additional_properties(irg, curr_prop & ~mtp_temporary);
+ SET_IRG_READY(irg);
}
if (top)
SET_IRG_READY(irg);
CLEAR_IRG_BUSY(irg);
return curr_prop;
-} /* check_nothrow_or_malloc */
+}
/**
* When a function was detected as "const", it might be moved out of loops.
*/
static void check_for_possible_endless_loops(ir_graph *irg)
{
- ir_loop *root_loop;
assure_irg_properties(irg, IR_GRAPH_PROPERTY_CONSISTENT_LOOPINFO);
- root_loop = get_irg_loop(irg);
+ ir_loop *root_loop = get_irg_loop(irg);
if (root_loop->flags & loop_outer_loop)
add_irg_additional_properties(irg, mtp_property_has_loop);
confirm_irg_properties(irg, IR_GRAPH_PROPERTIES_ALL);
}
-/*
- * optimize function calls by handling const functions
- */
void optimize_funccalls(void)
{
- size_t i, n;
- size_t last_idx;
- env_t ctx;
- size_t num_const = 0;
- size_t num_pure = 0;
- size_t num_nothrow = 0;
- size_t num_malloc = 0;
-
/* prepare: mark all graphs as not analyzed */
- last_idx = get_irp_last_idx();
+ size_t last_idx = get_irp_last_idx();
ready_set = rbitset_malloc(last_idx);
busy_set = rbitset_malloc(last_idx);
/* first step: detect, which functions are nothrow or malloc */
DB((dbg, LEVEL_2, "Detecting nothrow and malloc properties ...\n"));
- for (i = 0, n = get_irp_n_irgs(); i < n; ++i) {
- ir_graph *irg = get_irp_irg(i);
- unsigned prop = check_nothrow_or_malloc(irg, /*top=*/1);
+ for (size_t i = 0, n = get_irp_n_irgs(); i < n; ++i) {
+ ir_graph *irg = get_irp_irg(i);
+ unsigned prop = check_nothrow_or_malloc(irg, true);
if (prop & mtp_property_nothrow) {
- ++num_nothrow;
DB((dbg, LEVEL_2, "%+F has the nothrow property\n", irg));
} else if (prop & mtp_property_malloc) {
- ++num_malloc;
DB((dbg, LEVEL_2, "%+F has the malloc property\n", irg));
}
}
/* second step: remove exception edges: this must be done before the
detection of const and pure functions take place. */
+ env_t ctx;
handle_nothrow_Calls(&ctx);
- DB((dbg, LEVEL_1, "Detected %zu nothrow graphs, %zu malloc graphs.\n", num_nothrow, num_malloc));
- DB((dbg, LEVEL_1, "Optimizes %zu(SymConst) + %zu(Sel) calls to nothrow functions.\n",
- ctx.n_calls_SymConst, ctx.n_calls_Sel));
rbitset_clear_all(ready_set, last_idx);
rbitset_clear_all(busy_set, last_idx);
/* third step: detect, which functions are const or pure */
DB((dbg, LEVEL_2, "Detecting const and pure properties ...\n"));
- for (i = 0, n = get_irp_n_irgs(); i < n; ++i) {
- ir_graph *irg = get_irp_irg(i);
- unsigned prop = check_const_or_pure_function(irg, /*top=*/1);
+ for (size_t i = 0, n = get_irp_n_irgs(); i < n; ++i) {
+ ir_graph *irg = get_irp_irg(i);
+ unsigned prop = check_const_or_pure_function(irg, true);
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));
}
}
handle_const_Calls(&ctx);
- DB((dbg, LEVEL_1, "Detected %zu const graphs, %zu pure graphs.\n",
- num_const, num_pure));
- DB((dbg, LEVEL_1, "Optimizes %u(SymConst) + %u(Sel) calls to const functions.\n",
- ctx.n_calls_SymConst, ctx.n_calls_Sel));
xfree(busy_set);
xfree(ready_set);
}
-/* initialize the funccall optimization */
void firm_init_funccalls(void)
{
FIRM_DBG_REGISTER(dbg, "firm.opt.funccalls");
}
-/* Creates an ir_prog pass for optimize_funccalls. */
ir_prog_pass_t *optimize_funccalls_pass(const char *name)
{
return def_prog_pass(name ? name : "funccall", optimize_funccalls);