Do not warn about wrong register for a Phi input if that register is a joker (i.e...
[libfirm] / ir / be / bessadestr.c
index b6016af..77cf75c 100644 (file)
@@ -1,10 +1,28 @@
-/**
- * Author:      Daniel Grund
- * Date:               25.05.2005
- * Copyright:   (c) Universitaet Karlsruhe
- * Licence:     This file protected by GPL -  GNU GENERAL PUBLIC LICENSE.
+/*
+ * Copyright (C) 1995-2008 University of Karlsruhe.  All right reserved.
+ *
+ * This file is part of libFirm.
+ *
+ * This file may be distributed and/or modified under the terms of the
+ * GNU General Public License version 2 as published by the Free Software
+ * Foundation and appearing in the file LICENSE.GPL included in the
+ * packaging of this file.
+ *
+ * Licensees holding valid libFirm Professional Edition licenses may use
+ * this file in accordance with the libFirm Commercial License.
+ * Agreement provided with the Software.
  *
- * Performs SSA-Destruction.
+ * This file is provided AS IS with NO WARRANTY OF ANY KIND, INCLUDING THE
+ * WARRANTY OF DESIGN, MERCHANTABILITY AND FITNESS FOR A PARTICULAR
+ * PURPOSE.
+ */
+
+/**
+ * @file
+ * @brief       Performs SSA-Destruction.
+ * @author      Daniel Grund
+ * @date        25.05.2005
+ * @version     $Id$
  */
 #ifdef HAVE_CONFIG_H
 #include "config.h"
 #include "be_t.h"
 #include "beutil.h"
 #include "bechordal_t.h"
-#include "bearch.h"
+#include "bearch_t.h"
 #include "belive_t.h"
 #include "benode_t.h"
 #include "besched_t.h"
+#include "bestatevent.h"
+#include "beirg_t.h"
+#include "beintlive_t.h"
 
 DEBUG_ONLY(static firm_dbg_module_t *dbg = NULL;)
-#define DUMP_GRAPHS
 
 #define get_chordal_arch(ce) ((ce)->birg->main_env->arch_env)
 #define get_reg(irn) arch_get_irn_register(get_chordal_arch(chordal_env), irn)
 #define set_reg(irn, reg) arch_set_irn_register(get_chordal_arch(chordal_env), irn, reg)
 
-#define is_Perm(irn)            (arch_irn_classify(arch_env, irn) == arch_irn_class_perm)
+#define is_Perm(irn)            (arch_irn_class_is(arch_env, irn, perm))
 #define get_reg_cls(irn)        (arch_get_irn_reg_class(arch_env, irn, -1))
 #define is_curr_reg_class(irn)  (get_reg_cls(p) == chordal_env->cls)
 
 static void clear_link(ir_node *irn, void *data) {
+       (void) data;
        set_irn_link(irn, NULL);
 }
 
@@ -52,7 +73,7 @@ static void clear_link(ir_node *irn, void *data) {
  */
 static void collect_phis_walker(ir_node *irn, void *data) {
        be_chordal_env_t *env = data;
-       if(is_Phi(irn) && chordal_has_class(env, irn)) {
+       if (is_Phi(irn) && chordal_has_class(env, irn)) {
                ir_node *bl = get_nodes_block(irn);
                set_irn_link(irn, get_irn_link(bl));
                set_irn_link(bl, irn);
@@ -75,16 +96,25 @@ typedef struct {
 static int cmp_perm_proj(const void *a, const void *b, size_t n) {
        const perm_proj_t *p = a;
        const perm_proj_t *q = b;
+       (void) n;
+
        return !(p->arg == q->arg);
 }
 
+typedef struct insert_all_perms_env_t {
+       be_chordal_env_t *chordal_env;
+       pmap *perm_map;
+} insert_all_perms_env_t;
+
 /**
  * Insert Perms in all predecessors of a block containing a phi
  */
 static void insert_all_perms_walker(ir_node *bl, void *data) {
-       be_chordal_env_t *chordal_env = data;
-       pmap *perm_map = chordal_env->data;
+       insert_all_perms_env_t *env = data;
+       be_chordal_env_t *chordal_env = env->chordal_env;
+       pmap *perm_map = env->perm_map;
        ir_graph *irg = chordal_env->irg;
+       be_lv_t *lv = chordal_env->birg->lv;
        int i, n;
 
        assert(is_Block(bl));
@@ -108,9 +138,12 @@ static void insert_all_perms_walker(ir_node *bl, void *data) {
                 * register class by construction.
                 */
                for(phi = get_irn_link(bl); phi; phi = get_irn_link(phi)) {
-                       perm_proj_t templ;
-                       ir_node *arg     = get_irn_n(phi, i);
-                       unsigned hash    = HASH_PTR(arg);
+                       ir_node     *arg  = get_irn_n(phi, i);
+                       unsigned     hash = hash_irn(arg);
+                       perm_proj_t  templ;
+
+                       if (arch_irn_is(chordal_env->birg->main_env->arch_env, arg, ignore))
+                               continue;
 
                        templ.arg  = arg;
                        pp         = set_find(arg_set, &templ, sizeof(templ), hash);
@@ -122,7 +155,7 @@ static void insert_all_perms_walker(ir_node *bl, void *data) {
                         * interferes with the phi and must thus not be member of a
                         * Perm. A copy will be inserted for this argument alter on.
                         */
-                       if(!pp && !is_live_in(bl, arg)) {
+                       if(!pp && !be_is_live_in(lv, bl, arg)) {
                                templ.pos = n_projs++;
                                set_insert(arg_set, &templ, sizeof(templ), hash);
                        }
@@ -139,8 +172,9 @@ static void insert_all_perms_walker(ir_node *bl, void *data) {
                                in[pp->pos] = pp->arg;
 
                        perm = be_new_Perm(chordal_env->cls, irg, pred_bl, n_projs, in);
-                       free(in);
-                       insert_after = sched_skip(sched_last(pred_bl), 0, sched_skip_cf_predicator, chordal_env->birg->main_env->arch_env);
+                       be_stat_ev("phi_perm", n_projs);
+
+                       insert_after = sched_skip(sched_last(pred_bl), 0, sched_skip_cf_predicator, &chordal_env->birg->main_env->arch_env);
                        sched_add_after(insert_after, perm);
 
                        /*
@@ -154,7 +188,6 @@ static void insert_all_perms_walker(ir_node *bl, void *data) {
                                pp->proj = proj;
                                assert(get_reg(pp->arg));
                                set_reg(proj, get_reg(pp->arg));
-                               sched_add_after(insert_after, proj);
                                insert_after = proj;
                                DBG((dbg, LEVEL_2, "Copy register assignment %s from %+F to %+F\n", get_reg(pp->arg)->name, pp->arg, pp->proj));
                        }
@@ -166,13 +199,23 @@ static void insert_all_perms_walker(ir_node *bl, void *data) {
                                perm_proj_t templ;
 
                                templ.arg = get_irn_n(phi, i);
-                               pp        = set_find(arg_set, &templ, sizeof(templ), HASH_PTR(templ.arg));
+                               pp        = set_find(arg_set, &templ, sizeof(templ), hash_irn(templ.arg));
 
                                /* If not found, it was an interfering argument */
-                               if (pp)
+                               if (pp) {
                                        set_irn_n(phi, i, pp->proj);
+                                       be_liveness_introduce(lv, pp->proj);
+                               }
                        }
 
+                       /* update the liveness of the Perm's operands. It might be changed. */
+                       {
+                               int i;
+                               for (i = 0; i < n_projs; ++i)
+                                       be_liveness_update(lv, in[i]);
+                       }
+                       free(in);
+
                        /* register in perm map */
                        pmap_insert(perm_map, pred_bl, perm);
                }
@@ -191,6 +234,7 @@ static void insert_all_perms_walker(ir_node *bl, void *data) {
  */
 static void    set_regs_or_place_dupls_walker(ir_node *bl, void *data) {
        be_chordal_env_t *chordal_env = data;
+       be_lv_t *lv = chordal_env->birg->lv;
        ir_node *phi;
 
        /* Consider all phis of this block */
@@ -212,11 +256,14 @@ static void       set_regs_or_place_dupls_walker(ir_node *bl, void *data) {
                        arg_block = get_Block_cfgpred_block(phi_block, i);
                        arg_reg   = get_reg(arg);
 
+                       if (arch_irn_is(chordal_env->birg->main_env->arch_env, arg, ignore))
+                               continue;
+
                        assert(arg_reg && "Register must be set while placing perms");
 
                        DBG((dbg, LEVEL_1, "  for %+F(%s) -- %+F(%s)\n", phi, phi_reg->name, arg, arg_reg->name));
 
-                       if(nodes_interfere(chordal_env, phi, arg)) {
+                       if (values_interfere(chordal_env->birg, phi, arg)) {
                                /*
                                        Insert a duplicate in arguments block,
                                        make it the new phi arg,
@@ -225,6 +272,9 @@ static void set_regs_or_place_dupls_walker(ir_node *bl, void *data) {
                                        pin it
                                */
                                ir_node *dupl  = be_new_Copy(cls, chordal_env->irg, arg_block, arg);
+
+                               /* this is commented out because it will fail in case of unknown float */
+#if 0
                                ir_mode *m_phi = get_irn_mode(phi), *m_dupl = get_irn_mode(dupl);
 
                                /*
@@ -234,11 +284,14 @@ static void       set_regs_or_place_dupls_walker(ir_node *bl, void *data) {
                                assert(((mode_is_int(m_phi) && mode_is_int(m_dupl)) ||
                                        (mode_is_float(m_phi) && mode_is_float(m_dupl))) &&
                                        (get_mode_size_bits(m_phi) == get_mode_size_bits(m_dupl)));
+#endif /* if 0 */
 
                                set_irn_n(phi, i, dupl);
                                set_reg(dupl, phi_reg);
-                               sched_add_after(sched_skip(sched_last(arg_block), 0, sched_skip_cf_predicator, chordal_env->birg->main_env->arch_env), dupl);
+                               sched_add_after(sched_skip(sched_last(arg_block), 0, sched_skip_cf_predicator, &chordal_env->birg->main_env->arch_env), dupl);
                                pin_irn(dupl, phi_block);
+                               be_liveness_introduce(lv, dupl);
+                               be_liveness_update(lv, arg);
                                DBG((dbg, LEVEL_1, "    they do interfere: insert %+F(%s)\n", dupl, get_reg(dupl)->name));
                                continue; /* with next argument */
                        }
@@ -288,8 +341,11 @@ static void        set_regs_or_place_dupls_walker(ir_node *bl, void *data) {
                                        pin it
                                */
                                ir_node *perm     = get_Proj_pred(arg);
-                               ir_node *orig_val = get_irn_n(perm, get_Proj_proj(arg));
-                               ir_node *dupl     = be_new_Copy(cls, chordal_env->irg, arg_block, orig_val);
+                               ir_node *dupl     = be_new_Copy(cls, chordal_env->irg, arg_block, arg);
+                               ir_node *ins;
+
+                               /* this is commented out because it will fail in case of unknown float */
+#if 0
                                ir_mode *m_phi    = get_irn_mode(phi);
                                ir_mode *m_dupl   = get_irn_mode(dupl);
 
@@ -300,11 +356,16 @@ static void       set_regs_or_place_dupls_walker(ir_node *bl, void *data) {
                                assert(((mode_is_int(m_phi) && mode_is_int(m_dupl)) ||
                                        (mode_is_float(m_phi) && mode_is_float(m_dupl))) &&
                                        (get_mode_size_bits(m_phi) == get_mode_size_bits(m_dupl)));
+#endif /* if 0 */
 
                                set_irn_n(phi, i, dupl);
                                set_reg(dupl, phi_reg);
-                               sched_add_before(perm, dupl);
+                               /* skip the Perm's Projs and insert the copies behind. */
+                               for(ins = sched_next(perm); is_Proj(ins); ins = sched_next(ins));
+                               sched_add_before(ins, dupl);
                                pin_irn(dupl, phi_block);
+                               be_liveness_introduce(lv, dupl);
+                               be_liveness_update(lv, arg);
                                DBG((dbg, LEVEL_1, "      arg is pinned: insert %+F(%s)\n", dupl, get_reg(dupl)->name));
                        } else {
                                /*
@@ -320,28 +381,40 @@ static void       set_regs_or_place_dupls_walker(ir_node *bl, void *data) {
 }
 
 void be_ssa_destruction(be_chordal_env_t *chordal_env) {
+       insert_all_perms_env_t insert_perms_env;
        pmap *perm_map = pmap_create();
        ir_graph *irg  = chordal_env->irg;
+       be_lv_t *lv    = be_assure_liveness(chordal_env->birg);
 
        FIRM_DBG_REGISTER(dbg, "ir.be.ssadestr");
 
+       be_liveness_invalidate(lv);
+
        /* create a map for fast lookup of perms: block --> perm */
-       chordal_env->data = perm_map;
        irg_walk_graph(irg, clear_link, collect_phis_walker, chordal_env);
 
        DBG((dbg, LEVEL_1, "Placing perms...\n"));
-       irg_block_walk_graph(irg, insert_all_perms_walker, NULL, chordal_env);
-#ifdef DUMP_GRAPHS
-       be_dump(irg, "-ssa_destr_perms_placed", dump_ir_block_graph_sched);
-#endif
+       insert_perms_env.chordal_env = chordal_env;
+       insert_perms_env.perm_map = perm_map;
+       irg_block_walk_graph(irg, insert_all_perms_walker, NULL, &insert_perms_env);
 
-       be_liveness(irg);
+       // Matze: really needed here?
+       // Sebastian: Yes. the walker function uses interference.
+       be_liveness_invalidate(lv);
+
+       if (chordal_env->opts->dump_flags & BE_CH_DUMP_SSADESTR)
+               be_dump(irg, "-ssa_destr_perms_placed", dump_ir_block_graph_sched);
+
+       be_liveness_assure_chk(lv);
 
        DBG((dbg, LEVEL_1, "Setting regs and placing dupls...\n"));
        irg_block_walk_graph(irg, set_regs_or_place_dupls_walker, NULL, chordal_env);
-#ifdef DUMP_GRAPHS
-       be_dump(irg, "-ssa_destr_regs_set", dump_ir_block_graph_sched);
-#endif
+
+       /* TODO: unfortunaltely updating doesn't work yet. */
+       be_liveness_invalidate(lv);
+
+       if (chordal_env->opts->dump_flags & BE_CH_DUMP_SSADESTR)
+               be_dump(irg, "-ssa_destr_regs_set", dump_ir_block_graph_sched);
 
        pmap_destroy(perm_map);
 }
@@ -359,6 +432,9 @@ static void ssa_destruction_check_walker(ir_node *bl, void *data) {
                for (i = 0, max = get_irn_arity(phi); i < max; ++i) {
                        ir_node *arg = get_irn_n(phi, i);
 
+                       if (arch_irn_is(chordal_env->birg->main_env->arch_env, arg, ignore))
+                               continue;
+
                        arg_reg = get_reg(arg);
 
                        if (phi_reg != arg_reg) {