X-Git-Url: http://nsz.repo.hu/git/?a=blobdiff_plain;f=ir%2Fopt%2Fscalar_replace.c;h=9fcb8d55959bd0804a6d39a540a11e8d19c141ba;hb=188a17803798f8d78b1c02c3b68976056bce33d9;hp=eaa8ee64ad8fa3526098cc74ff9d6c75f11cb642;hpb=4b1138a9eee25ce11adbb7d7162eaa49421e8b51;p=libfirm diff --git a/ir/opt/scalar_replace.c b/ir/opt/scalar_replace.c index eaa8ee64a..9fcb8d559 100644 --- a/ir/opt/scalar_replace.c +++ b/ir/opt/scalar_replace.c @@ -23,9 +23,7 @@ * @author Beyhan Veliev, Michael Beck * @version $Id$ */ -#ifdef HAVE_CONFIG_H #include "config.h" -#endif #include @@ -172,9 +170,9 @@ static int check_load_store_mode(ir_mode *mode, ir_mode *ent_mode) { */ int is_address_taken(ir_node *sel) { - int i; - ir_mode *emode, *mode; - ir_node *value; + int i, input_nr, k; + ir_mode *emode, *mode; + ir_node *value; ir_entity *ent; if (! is_const_sel(sel)) @@ -185,15 +183,15 @@ int is_address_taken(ir_node *sel) switch (get_irn_opcode(succ)) { case iro_Load: + /* do not remove volatile variables */ + if (get_Load_volatility(succ) == volatility_is_volatile) + return 1; /* check if this load is not a hidden conversion */ mode = get_Load_mode(succ); ent = get_Sel_entity(sel); emode = get_type_mode(get_entity_type(ent)); if (! check_load_store_mode(mode, emode)) return 1; - /* do not remove volatile variables */ - if (get_Load_volatility(succ) == volatility_is_volatile) - return 1; break; case iro_Store: @@ -201,15 +199,15 @@ int is_address_taken(ir_node *sel) value = get_Store_value(succ); if (value == sel) return 1; + /* do not remove volatile variables */ + if (get_Store_volatility(succ) == volatility_is_volatile) + return 1; /* check if this Store is not a hidden conversion */ mode = get_irn_mode(value); ent = get_Sel_entity(sel); emode = get_type_mode(get_entity_type(ent)); if (! check_load_store_mode(mode, emode)) return 1; - /* do not remove volatile variables */ - if (get_Store_volatility(succ) == volatility_is_volatile) - return 1; break; case iro_Sel: { @@ -230,6 +228,33 @@ int is_address_taken(ir_node *sel) */ return 1; + case iro_Id: { + int res = is_address_taken(succ); + if (res) + return 1; + break; + } + + case iro_Tuple: + /* Non-optimized Tuple, happens in inlining */ + for (input_nr = get_Tuple_n_preds(succ) - 1; input_nr >= 0; --input_nr) { + ir_node *pred = get_Tuple_pred(succ, input_nr); + + if (pred == sel) { + /* we found one input */ + for (k = get_irn_n_outs(succ) - 1; k >= 0; --k) { + ir_node *proj = get_irn_out(succ, k); + + if (is_Proj(proj) && get_Proj_proj(proj) == input_nr) { + int res = is_address_taken(proj); + if (res) + return 1; + } + } + } + } + break; + default: /* another op, the address is taken */ return 1; @@ -243,38 +268,34 @@ int is_address_taken(ir_node *sel) * * @param ent the entity that will be scalar replaced * @param sel a Sel node that selects some fields of this entity - * - * Uses the visited flag to mark already linked Sel nodes. */ -static void link_all_leave_sels(ir_entity *ent, ir_node *sel) { - int i, flag = 1; +static int link_all_leave_sels(ir_entity *ent, ir_node *sel) { + int i, is_leave = 1; for (i = get_irn_n_outs(sel) - 1; i >= 0; --i) { ir_node *succ = get_irn_out(sel, i); if (is_Sel(succ)) { + /* the current leave has further Sel's, no leave */ + is_leave = 0; link_all_leave_sels(ent, succ); - flag = 0; + } else if (is_Id(succ)) { + is_leave &= link_all_leave_sels(ent, succ); } } - if (flag) { - /* if Sel nodes with memory inputs are used, a entity can be - * visited more than once causing a ring here, so we use the - * node flag to mark linked nodes - */ - if (irn_visited(sel)) - return; + if (is_leave) { + /* beware of Id's */ + sel = skip_Id(sel); /* we know we are at a leave, because this function is only - * called if the address is NOT taken, so succ must be a Load - * or a Store node + * called if the address is NOT taken, so sel's successor(s) + * must be Loads or Stores */ set_irn_link(sel, get_entity_link(ent)); set_entity_link(ent, sel); - - mark_irn_visited(sel); } + return is_leave; } /* we need a special address that serves as an address taken marker */ @@ -301,9 +322,6 @@ static int find_possible_replacements(ir_graph *irg) { int i; int res = 0; - ir_reserve_resources(irg, IR_RESOURCE_IRN_VISITED); - inc_irg_visited(irg); - /* * First, clear the link field of all interesting entities. */ @@ -345,7 +363,8 @@ static int find_possible_replacements(ir_graph *irg) { /* we can handle arrays, structs and atomic types yet */ if (is_Array_type(ent_type) || is_Struct_type(ent_type) || is_atomic_type(ent_type)) { if (is_address_taken(succ)) { - if (get_entity_link(ent)) /* killing one */ + /* killing one */ + if (get_entity_link(ent)) --res; set_entity_link(ent, ADDRESS_TAKEN); } else { @@ -358,7 +377,6 @@ static int find_possible_replacements(ir_graph *irg) { } } - ir_free_resources(irg, IR_RESOURCE_IRN_VISITED); return res; } @@ -379,8 +397,7 @@ static path_t *find_path(ir_node *sel, unsigned len) { if (! is_Sel(pred)) { /* we found the root */ - - res = xmalloc(sizeof(*res) + (len - 1) * sizeof(res->path)); + res = XMALLOCF(path_t, path, len); res->path_len = len; } else res = find_path(pred, len); @@ -557,13 +574,14 @@ static void topologic_walker(ir_node *node, void *ctx) { DB((dbg, SET_LEVEL_3, "replacing by value %u\n", vnum)); + block = get_nodes_block(node); + set_cur_block(block); + /* Beware: A Store can contain a hidden conversion in Firm. */ val = get_Store_value(node); if (get_irn_mode(val) != env->modes[vnum]) val = new_d_Conv(get_irn_dbg_info(node), val, env->modes[vnum]); - block = get_nodes_block(node); - set_cur_block(block); set_value(vnum, val); mem = get_Store_mem(node); @@ -627,6 +645,10 @@ int scalar_replacement_opt(ir_graph *irg) { /* Call algorithm that computes the out edges */ assure_irg_outs(irg); + /* we use the link firld to store the VNUM */ + ir_reserve_resources(irg, IR_RESOURCE_IRN_LINK); + irp_reserve_resources(irp, IR_RESOURCE_ENTITY_LINK); + /* Find possible scalar replacements */ if (find_possible_replacements(irg)) { DB((dbg, SET_LEVEL_1, "Scalar Replacement: %s\n", get_entity_name(get_irg_entity(irg)))); @@ -694,6 +716,9 @@ int scalar_replacement_opt(ir_graph *irg) { DEL_ARR_F(modes); } + ir_free_resources(irg, IR_RESOURCE_IRN_LINK); + irp_free_resources(irp, IR_RESOURCE_ENTITY_LINK); + current_ir_graph = rem; return res; }