X-Git-Url: http://nsz.repo.hu/git/?a=blobdiff_plain;f=ir%2Fopt%2Fldstopt.c;h=2e9aeeab0e5e56476b30a94263b25ee65fdeb69d;hb=b84d550997d07bfcc4d828e8fbb5742098e050e4;hp=3517964c3f487e1a2f6913d706c2fae6079bddef;hpb=f1856f4362da72ba576f379aec28a174cfb74097;p=libfirm diff --git a/ir/opt/ldstopt.c b/ir/opt/ldstopt.c index 3517964c3..2e9aeeab0 100644 --- a/ir/opt/ldstopt.c +++ b/ir/opt/ldstopt.c @@ -286,10 +286,10 @@ static entity *find_constant_entity(ir_node *ptr) return get_SymConst_entity(ptr); } else if (op == op_Sel) { - entity *ent = get_Sel_entity(ptr); - type *tp = get_entity_owner(ent); + entity *ent = get_Sel_entity(ptr); + ir_type *tp = get_entity_owner(ent); - /* Do not fiddle about polymorphy. */ + /* Do not fiddle with polymorphism. */ if (is_Class_type(get_entity_owner(ent)) && ((get_entity_n_overwrites(ent) != 0) || (get_entity_n_overwrittenby(ent) != 0) ) ) @@ -421,16 +421,16 @@ static unsigned optimize_load(ir_node *load) * like x = new O; x->t; */ if (info->projs[pn_Load_X_except]) { - if (get_irn_op(ptr) == op_Sel) { + if (is_Sel(ptr)) { ir_node *mem = get_Sel_mem(ptr); if (get_irn_op(skip_Proj(mem)) == op_Alloc) { /* ok, check the types */ - entity *ent = get_Sel_entity(ptr); - type *s_type = get_entity_type(ent); - type *a_type = get_Alloc_type(mem); + entity *ent = get_Sel_entity(ptr); + ir_type *s_type = get_entity_type(ent); + ir_type *a_type = get_Alloc_type(mem); - if (is_subclass_of(s_type, a_type)) { + if (is_SubClass_of(s_type, a_type)) { /* ok, condition met: there can't be an exception because * Alloc guarantees that enough memory was allocated */ @@ -464,7 +464,7 @@ static unsigned optimize_load(ir_node *load) } /* Load from a constant polymorphic field, where we can resolve - polymorphy. */ + polymorphism. */ new_node = transform_node_Load(load); if (new_node != load) { if (info->projs[pn_Load_M]) { @@ -590,7 +590,7 @@ static unsigned optimize_load(ir_node *load) * BEWARE: one might think that checking the modes is useless, because * if the pointers are identical, they refer to the same object. * This is only true in strong typed languages, not in C were the following - * is possible a = *(type1 *)p; b = *(type2 *)p ... + * is possible a = *(ir_type1 *)p; b = *(ir_type2 *)p ... */ if (get_irn_op(pred) == op_Store && get_Store_ptr(pred) == ptr && @@ -701,7 +701,7 @@ static unsigned optimize_store(ir_node *store) * BEWARE: one might think that checking the modes is useless, because * if the pointers are identical, they refer to the same object. * This is only true in strong typed languages, not is C were the following - * is possible *(type1 *)p = a; *(type2 *)p = b ... + * is possible *(ir_type1 *)p = a; *(ir_type2 *)p = b ... */ ptr = get_Store_ptr(store); @@ -920,7 +920,7 @@ static unsigned optimize_phi(ir_node *phi, void *env) } /** - * walker, do the optiimizations + * walker, do the optimizations */ static void do_load_store_optimize(ir_node *n, void *env) { @@ -955,7 +955,7 @@ void optimize_load_store(ir_graph *irg) assert(get_irg_pinned(irg) != op_pin_state_floats && "LoadStore optimization needs pinned graph"); - if (!get_opt_redundant_LoadStore()) + if (! get_opt_redundant_loadstore()) return; obstack_init(&env.obst); @@ -978,7 +978,6 @@ void optimize_load_store(ir_graph *irg) if (env.changes & CF_CHANGED) { /* is this really needed: Yes, control flow changed, block might get Bad. */ - if (get_irg_dom_state(current_ir_graph) == dom_consistent) - set_irg_dom_inconsistent(current_ir_graph); + set_irg_doms_inconsistent(current_ir_graph); } }