- C99 feature removed and simplified code
[libfirm] / ir / be / bechordal.c
index d9d5506..6d32c1c 100644 (file)
@@ -24,9 +24,7 @@
  * @date        08.12.2004
  * @version     $Id$
  */
-#ifdef HAVE_CONFIG_H
 #include "config.h"
-#endif
 
 #include <ctype.h>
 
@@ -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)
 {
@@ -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;
 
@@ -310,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)
@@ -332,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) ||
@@ -377,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
@@ -433,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);
@@ -467,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);
        }
 
        /*
@@ -500,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;
@@ -546,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();
 
        /*
@@ -645,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);
@@ -676,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));
                }
@@ -696,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);
@@ -705,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 : "<none>"));
 
@@ -713,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));
                        }
@@ -885,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;
 
@@ -912,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");
@@ -937,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
@@ -948,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));
 
@@ -969,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");
@@ -998,7 +982,6 @@ 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;
 
-       be_assure_dom_front(birg);
        lv = be_assure_liveness(birg);
        be_liveness_assure_sets(lv);
        be_liveness_assure_chk(lv);