X-Git-Url: http://nsz.repo.hu/git/?a=blobdiff_plain;f=ir%2Fbe%2Fbechordal.c;h=6d32c1c0635f4a9b3a938f44eaf65ff0a1d80f7a;hb=20aba3a7411997af46617a102d91cf05b2ebd8cb;hp=00e3a7f0ad7463fb83ba7e87bdbc54f5184221db;hpb=0b72f9906b67b60180be7dacf5b720621560399c;p=libfirm diff --git a/ir/be/bechordal.c b/ir/be/bechordal.c index 00e3a7f0a..6d32c1c06 100644 --- a/ir/be/bechordal.c +++ b/ir/be/bechordal.c @@ -1,5 +1,5 @@ /* - * Copyright (C) 1995-2007 University of Karlsruhe. All right reserved. + * Copyright (C) 1995-2008 University of Karlsruhe. All right reserved. * * This file is part of libFirm. * @@ -24,9 +24,7 @@ * @date 08.12.2004 * @version $Id$ */ -#ifdef HAVE_CONFIG_H #include "config.h" -#endif #include @@ -48,7 +46,6 @@ #include "irtools.h" #include "irbitset.h" #include "debug.h" -#include "xmalloc.h" #include "iredges.h" #include "beutil.h" @@ -77,12 +74,12 @@ DEBUG_ONLY(static firm_dbg_module_t *dbg = NULL;) typedef struct _be_chordal_alloc_env_t { be_chordal_env_t *chordal_env; - pset *pre_colored; /**< Set of precolored nodes. */ - bitset_t *live; /**< A liveness bitset. */ - bitset_t *tmp_colors; /**< An auxiliary bitset which is as long as the number of colors in the class. */ - bitset_t *colors; /**< The color mask. */ - bitset_t *in_colors; /**< Colors used by live in values. */ - int colors_n; /**< The number of colors. */ + pset *pre_colored; /**< Set of precolored nodes. */ + bitset_t *live; /**< A liveness bitset. */ + bitset_t *tmp_colors; /**< An auxiliary bitset which is as long as the number of colors in the class. */ + bitset_t *colors; /**< The color mask. */ + bitset_t *in_colors; /**< Colors used by live in values. */ + int colors_n; /**< The number of colors. */ } be_chordal_alloc_env_t; #include "fourcc.h" @@ -121,7 +118,7 @@ static void check_heads(be_chordal_env_t *env) * @param is_def Is the border a use or a def. * @return The created border. */ -static INLINE border_t *border_add(be_chordal_env_t *env, struct list_head *head, +static inline border_t *border_add(be_chordal_env_t *env, struct list_head *head, ir_node *irn, unsigned step, unsigned pressure, unsigned is_def, unsigned is_real) { @@ -157,7 +154,7 @@ static INLINE border_t *border_add(be_chordal_env_t *env, struct list_head *head else { b = get_irn_link(irn); - assert(b && b->magic == BORDER_FOURCC && "Illegal border encountered"); + DEBUG_ONLY(assert(b && b->magic == BORDER_FOURCC && "Illegal border encountered")); } b->pressure = pressure; @@ -178,14 +175,11 @@ static INLINE border_t *border_add(be_chordal_env_t *env, struct list_head *head * @param irn The node. * @return 1, if the node is of that register class, 0 if not. */ -static INLINE int has_reg_class(const be_chordal_env_t *env, const ir_node *irn) +static inline int has_reg_class(const be_chordal_env_t *env, const ir_node *irn) { - return arch_irn_consider_in_reg_alloc(env->birg->main_env->arch_env, env->cls, irn); + return arch_irn_consider_in_reg_alloc(env->cls, irn); } -#define has_limited_constr(req, irn) \ - (arch_get_register_req(arch_env, (req), irn, -1) && (req)->type == arch_register_req_type_limited) - static int get_next_free_reg(const be_chordal_alloc_env_t *alloc_env, bitset_t *colors) { bitset_t *tmp = alloc_env->tmp_colors; @@ -225,7 +219,6 @@ static be_insn_t *chordal_scan_insn(be_chordal_env_t *env, ir_node *irn) be_insn_env_t ie; ie.ignore_colors = env->ignore_colors; - ie.aenv = env->birg->main_env->arch_env; ie.obst = env->obst; ie.cls = env->cls; return be_scan_insn(&ie, irn); @@ -233,12 +226,11 @@ static be_insn_t *chordal_scan_insn(be_chordal_env_t *env, ir_node *irn) static ir_node *prepare_constr_insn(be_chordal_env_t *env, ir_node *irn) { - const be_irg_t *birg = env->birg; - const arch_env_t *aenv = birg->main_env->arch_env; bitset_t *tmp = bitset_alloca(env->cls->n_regs); bitset_t *def_constr = bitset_alloca(env->cls->n_regs); ir_node *bl = get_nodes_block(irn); - be_lv_t *lv = env->birg->lv; + const be_irg_t *birg = env->birg; + be_lv_t *lv = birg->lv; be_insn_t *insn; int i, j; @@ -249,17 +241,17 @@ static ir_node *prepare_constr_insn(be_chordal_env_t *env, ir_node *irn) const arch_register_t *reg; const arch_register_req_t *req; - if (arch_get_irn_reg_class(aenv, irn, i) != env->cls) + req = arch_get_register_req(irn, i); + if (req->cls != env->cls) continue; - reg = arch_get_irn_register(aenv, op); + reg = arch_get_irn_register(op); if (reg == NULL || !arch_register_type_is(reg, ignore)) continue; if(arch_register_type_is(reg, joker)) continue; - req = arch_get_register_req(aenv, irn, i); if (!arch_register_req_is(req, limited)) continue; @@ -293,6 +285,11 @@ static ir_node *prepare_constr_insn(be_chordal_env_t *env, ir_node *irn) if(a_op->carrier != op->carrier || !a_op->has_constraints) continue; + /* if the constraint is the same, no copy is necessary + * TODO generalise unequal but overlapping constraints */ + if (a_op->req == op->req) + continue; + if (be_is_Copy(get_irn_n(insn->irn, a_op->pos))) continue; @@ -305,7 +302,7 @@ static ir_node *prepare_constr_insn(be_chordal_env_t *env, ir_node *irn) } } - /* collect all registers occuring in out constraints. */ + /* collect all registers occurring in out constraints. */ for(i = 0; i < insn->use_start; ++i) { be_operand_t *op = &insn->ops[i]; if(op->has_constraints) @@ -327,7 +324,7 @@ static ir_node *prepare_constr_insn(be_chordal_env_t *env, ir_node *irn) Check, if 1) the operand is constrained. 2) lives through the node. - 3) is constrained to a register occuring in out constraints. + 3) is constrained to a register occurring in out constraints. */ if(!op->has_constraints || !values_interfere(birg, insn->irn, op->carrier) || @@ -372,13 +369,9 @@ void be_pre_spill_prepare_constr(be_chordal_env_t *cenv) { static void pair_up_operands(const be_chordal_alloc_env_t *alloc_env, be_insn_t *insn) { const be_chordal_env_t *env = alloc_env->chordal_env; - - int n_uses = be_insn_n_uses(insn); - int n_defs = be_insn_n_defs(insn); - bitset_t *bs = bitset_alloca(env->cls->n_regs); - int *pairing = alloca(MAX(n_defs, n_uses) * sizeof(pairing[0])); - - int i, j; + bitset_t *bs = bitset_alloca(env->cls->n_regs); + int i; + int j; /* For each out operand, try to find an in operand which can be assigned the @@ -428,7 +421,6 @@ static ir_node *pre_process_constraints(be_chordal_alloc_env_t *alloc_env, be_insn_t **the_insn) { be_chordal_env_t *env = alloc_env->chordal_env; - const arch_env_t *aenv = env->birg->main_env->arch_env; be_insn_t *insn = *the_insn; ir_node *perm = NULL; bitset_t *out_constr = bitset_alloca(env->cls->n_regs); @@ -462,7 +454,7 @@ static ir_node *pre_process_constraints(be_chordal_alloc_env_t *alloc_env, be_stat_ev("constr_perm", get_irn_arity(perm)); foreach_out_edge(perm, edge) { ir_node *proj = get_edge_src_irn(edge); - arch_set_irn_register(aenv, proj, NULL); + arch_set_irn_register(proj, NULL); } /* @@ -495,7 +487,6 @@ static ir_node *pre_process_constraints(be_chordal_alloc_env_t *alloc_env, static ir_node *handle_constraints(be_chordal_alloc_env_t *alloc_env, ir_node *irn, int *silent) { - const arch_env_t *aenv; int n_regs; bitset_t *bs; ir_node **alloc_nodes; @@ -506,7 +497,7 @@ static ir_node *handle_constraints(be_chordal_alloc_env_t *alloc_env, bitset_pos_t col; const ir_edge_t *edge; ir_node *perm = NULL; - int match_res, cost; + //int match_res, cost; be_chordal_env_t *env = alloc_env->chordal_env; void *base = obstack_base(env->obst); be_insn_t *insn = chordal_scan_insn(env, irn); @@ -541,13 +532,12 @@ static ir_node *handle_constraints(be_chordal_alloc_env_t *alloc_env, if(!insn->has_constraints) goto end; - aenv = env->birg->main_env->arch_env; n_regs = env->cls->n_regs; bs = bitset_alloca(n_regs); - alloc_nodes = alloca(n_regs * sizeof(alloc_nodes[0])); + alloc_nodes = ALLOCAN(ir_node*, n_regs); //bp = hungarian_new(n_regs, n_regs, 2, HUNGARIAN_MATCH_PERFECT); bp = bipartite_new(n_regs, n_regs); - assignment = alloca(n_regs * sizeof(assignment[0])); + assignment = ALLOCAN(int, n_regs); partners = pmap_create(); /* @@ -640,7 +630,7 @@ static ir_node *handle_constraints(be_chordal_alloc_env_t *alloc_env, pmap_insert(partners, proj, NULL); bitset_clear_all(bs); - arch_put_non_ignore_regs(aenv, env->cls, bs); + arch_put_non_ignore_regs(env->cls, bs); bitset_andnot(bs, env->ignore_colors); bitset_foreach(bs, col) { //hungarian_add(bp, n_alloc, col, 1); @@ -671,14 +661,14 @@ static ir_node *handle_constraints(be_chordal_alloc_env_t *alloc_env, irn = alloc_nodes[i]; if (irn != NULL) { - arch_set_irn_register(aenv, irn, reg); + arch_set_irn_register(irn, reg); (void) pset_hinsert_ptr(alloc_env->pre_colored, irn); DBG((dbg, LEVEL_2, "\tsetting %+F to register %s\n", irn, reg->name)); } irn = pmap_get(partners, alloc_nodes[i]); if (irn != NULL) { - arch_set_irn_register(aenv, irn, reg); + arch_set_irn_register(irn, reg); (void) pset_hinsert_ptr(alloc_env->pre_colored, irn); DBG((dbg, LEVEL_2, "\tsetting %+F to register %s\n", irn, reg->name)); } @@ -691,7 +681,7 @@ static ir_node *handle_constraints(be_chordal_alloc_env_t *alloc_env, /* Put the colors of all Projs in a bitset. */ foreach_out_edge(perm, edge) { ir_node *proj = get_edge_src_irn(edge); - const arch_register_t *reg = arch_get_irn_register(aenv, proj); + const arch_register_t *reg = arch_get_irn_register(proj); if(reg != NULL) bitset_set(bs, reg->index); @@ -700,7 +690,7 @@ static ir_node *handle_constraints(be_chordal_alloc_env_t *alloc_env, /* Assign the not yet assigned Projs of the Perm a suitable color. */ foreach_out_edge(perm, edge) { ir_node *proj = get_edge_src_irn(edge); - const arch_register_t *reg = arch_get_irn_register(aenv, proj); + const arch_register_t *reg = arch_get_irn_register(proj); DBG((dbg, LEVEL_2, "\tchecking reg of %+F: %s\n", proj, reg ? reg->name : "")); @@ -708,7 +698,7 @@ static ir_node *handle_constraints(be_chordal_alloc_env_t *alloc_env, col = get_next_free_reg(alloc_env, bs); reg = arch_register_for_index(env->cls, col); bitset_set(bs, reg->index); - arch_set_irn_register(aenv, proj, reg); + arch_set_irn_register(proj, reg); pset_insert_ptr(alloc_env->pre_colored, proj); DBG((dbg, LEVEL_2, "\tsetting %+F to register %s\n", proj, reg->name)); } @@ -880,7 +870,6 @@ static void assign(ir_node *block, void *env_ptr) bitset_t *live = alloc_env->live; bitset_t *colors = alloc_env->colors; bitset_t *in_colors = alloc_env->in_colors; - const arch_env_t *arch_env = env->birg->main_env->arch_env; struct list_head *head = get_block_border_head(env, block); be_lv_t *lv = env->birg->lv; @@ -907,7 +896,7 @@ static void assign(ir_node *block, void *env_ptr) be_lv_foreach(lv, block, be_lv_state_in, idx) { irn = be_lv_get_irn(lv, block, idx); if(has_reg_class(env, irn)) { - const arch_register_t *reg = arch_get_irn_register(arch_env, irn); + const arch_register_t *reg = arch_get_irn_register(irn); int col; assert(reg && "Node must have been assigned a register"); @@ -932,7 +921,7 @@ static void assign(ir_node *block, void *env_ptr) list_for_each_entry_reverse(border_t, b, head, list) { ir_node *irn = b->irn; int nr = get_irn_idx(irn); - int ignore = arch_irn_is(arch_env, irn, ignore); + int ignore = arch_irn_is_ignore(irn); /* * Assign a color, if it is a local def. Global defs already have a @@ -943,18 +932,18 @@ static void assign(ir_node *block, void *env_ptr) int col = NO_COLOR; if(ignore || pset_find_ptr(alloc_env->pre_colored, irn)) { - reg = arch_get_irn_register(arch_env, irn); + reg = arch_get_irn_register(irn); col = reg->index; assert(!bitset_is_set(colors, col) && "pre-colored register must be free"); } else { col = get_next_free_reg(alloc_env, colors); reg = arch_register_for_index(env->cls, col); - assert(arch_get_irn_register(arch_env, irn) == NULL && "This node must not have been assigned a register yet"); + assert(arch_get_irn_register(irn) == NULL && "This node must not have been assigned a register yet"); assert(!arch_register_type_is(reg, ignore) && "Must not assign ignore register"); } bitset_set(colors, col); - arch_set_irn_register(arch_env, irn, reg); + arch_set_irn_register(irn, reg); DBG((dbg, LEVEL_1, "\tassigning register %s(%d) to %+F\n", arch_register_get_name(reg), col, irn)); @@ -964,7 +953,7 @@ static void assign(ir_node *block, void *env_ptr) /* Clear the color upon a use. */ else if(!b->is_def) { - const arch_register_t *reg = arch_get_irn_register(arch_env, irn); + const arch_register_t *reg = arch_get_irn_register(irn); int col; assert(reg && "Register must have been assigned"); @@ -992,13 +981,7 @@ void be_ra_chordal_color(be_chordal_env_t *chordal_env) int colors_n = arch_register_class_n_regs(cls); ir_graph *irg = chordal_env->irg; - int allocatable_regs = colors_n - be_put_ignore_regs(birg, cls, NULL); - /* some special classes contain only ignore regs, no work to be done */ - if(allocatable_regs == 0) - return; - - be_assure_dom_front(birg); lv = be_assure_liveness(birg); be_liveness_assure_sets(lv); be_liveness_assure_chk(lv); @@ -1012,6 +995,8 @@ void be_ra_chordal_color(be_chordal_env_t *chordal_env) env.in_colors = bitset_alloca(colors_n); env.pre_colored = pset_new_ptr_default(); + BE_TIMER_PUSH(t_constr); + /* Handle register targeting constraints */ dom_tree_walk_irg(irg, constraints, NULL, &env); @@ -1020,6 +1005,8 @@ void be_ra_chordal_color(be_chordal_env_t *chordal_env) be_dump(chordal_env->irg, buf, dump_ir_block_graph_sched); } + BE_TIMER_POP(t_constr); + env.live = bitset_malloc(get_irg_last_idx(chordal_env->irg)); /* First, determine the pressure */