belive: Inline be_lv_remove() into its only caller.
[libfirm] / ir / be / belive.c
index b142d27..383df75 100644 (file)
@@ -1,52 +1,37 @@
+/*
+ * This file is part of libFirm.
+ * Copyright (C) 2012 University of Karlsruhe.
+ */
+
 /**
- * Interblock liveness analysis.
- * @author Sebastian Hack
- * @date 6.12.2004
+ * @file
+ * @brief       Interblock liveness analysis.
+ * @author      Sebastian Hack
+ * @date        06.12.2004
  */
-#ifdef HAVE_CONFIG_H
 #include "config.h"
-#endif
 
-#include "impl.h"
+/* statev is expensive here, only enable when needed */
+#define DISABLE_STATEV
+
 #include "iredges_t.h"
 #include "irgwalk.h"
-#include "irprintf_t.h"
-#include "irbitset.h"
+#include "irprintf.h"
 #include "irdump_t.h"
+#include "irnodeset.h"
 
+#include "absgraph.h"
+#include "statev_t.h"
+#include "be_t.h"
+#include "bearch.h"
 #include "beutil.h"
 #include "belive_t.h"
-#include "besched_t.h"
+#include "besched.h"
 #include "bemodule.h"
 
 DEBUG_ONLY(static firm_dbg_module_t *dbg = NULL;)
 
-#define LV_STD_SIZE             128
-#define LV_USE_BINARY_SEARCH
-#undef  LV_INTESIVE_CHECKS
-
-static INLINE int is_liveness_node(const ir_node *irn)
-{
-       switch(get_irn_opcode(irn)) {
-       case iro_Block:
-       case iro_Bad:
-       case iro_End:
-               return 0;
-       default:;
-       }
-
-       return 1;
-}
-
-int (be_lv_next_irn)(const struct _be_lv_t *lv, const ir_node *bl, unsigned flags, int i)
-{
-       return _be_lv_next_irn(lv, bl, flags, i);
-}
-
-const ir_node * (be_lv_get_irn)(const struct _be_lv_t *lv, const ir_node *bl, int i)
-{
-       return _be_lv_get_irn(lv, bl, i);
-}
+#define LV_STD_SIZE             64
 
 int (be_is_live_in)(const be_lv_t *lv, const ir_node *block, const ir_node *irn)
 {
@@ -63,308 +48,214 @@ int (be_is_live_end)(const be_lv_t *lv, const ir_node *block, const ir_node *irn
        return _be_is_live_xxx(lv, block, irn, be_lv_state_end);
 }
 
-
-#ifdef LV_USE_BINARY_SEARCH
-static INLINE unsigned _be_liveness_bsearch(struct _be_lv_info_t *arr, unsigned idx)
+static inline unsigned _be_liveness_bsearch(be_lv_info_t *arr, const ir_node *node)
 {
-       struct _be_lv_info_t *payload = arr + 1;
+       be_lv_info_t *payload = arr + 1;
 
-       unsigned n   = arr[0].u.head.n_members;
+       unsigned n   = arr[0].head.n_members;
        unsigned res = 0;
        int lo       = 0;
        int hi       = n;
 
-       if(n == 0)
+       if (n == 0)
                return 0;
 
-#if 0
-       if(idx < payload[0].u.node.idx)
-               return 0;
-
-       if(idx > payload[n - 1].u.node.idx)
-               return n - 1;
-#endif
-
-       /* start a binary search for the requested node. */
-       while(lo < hi) {
-               int md          = lo + ((hi - lo) >> 1);
-               unsigned md_idx = payload[md].u.node.idx;
+       do {
+               int md           = lo + ((hi - lo) >> 1);
+               ir_node *md_node = payload[md].node.node;
 
-               if(idx > md_idx)
+               if (node > md_node)
                        lo = md + 1;
-               else if(idx < md_idx)
+               else if (node < md_node)
                        hi = md;
                else {
                        res = md;
-                       assert(payload[res].u.node.idx == idx);
                        break;
                }
 
                res = lo;
-       }
-
-#ifdef LV_INTESIVE_CHECKS
-       {
-               unsigned i;
-               for(i = res; i < n; ++i)
-                       assert(payload[i].u.node.idx >= idx);
-
-               for(i = 0; i < res; ++i)
-                       assert(payload[i].u.node.idx < idx);
-       }
-#endif
+       } while (lo < hi);
 
        return res;
 }
 
-#else
-
-/**
- * This function searches linearly for the node in the array.
- */
-static INLINE unsigned _be_liveness_bsearch(struct _be_lv_info_t *arr, unsigned idx) {
-       unsigned n  = arr[0].u.head.n_members;
-       unsigned i;
-
-       for(i = 0; i < n; ++i) {
-               if(arr[i + 1].u.node.idx == idx)
-                       return i;
-       }
-
-       return i;
-}
-#endif
-
-struct _be_lv_info_node_t *be_lv_get(const struct _be_lv_t *li, const ir_node *bl, const ir_node *irn)
+be_lv_info_node_t *be_lv_get(const be_lv_t *li, const ir_node *bl,
+                             const ir_node *irn)
 {
-       struct _be_lv_info_t *irn_live = phase_get_irn_data(&li->ph, bl);
-
-       if(irn_live) {
-               unsigned idx = get_irn_idx(irn);
+       be_lv_info_t *irn_live;
+       be_lv_info_node_t *res = NULL;
 
+       stat_ev_tim_push();
+       irn_live = ir_nodehashmap_get(be_lv_info_t, &li->map, bl);
+       if (irn_live != NULL) {
                /* Get the position of the index in the array. */
-               int pos = _be_liveness_bsearch(irn_live, idx);
+               int pos = _be_liveness_bsearch(irn_live, irn);
 
                /* Get the record in question. 1 must be added, since the first record contains information about the array and must be skipped. */
-               struct _be_lv_info_node_t *res = &irn_live[pos + 1].u.node;
+               be_lv_info_node_t *rec = &irn_live[pos + 1].node;
 
                /* Check, if the irn is in deed in the array. */
-               if(res->idx == idx)
-                       return res;
+               if (rec->node == irn)
+                       res = rec;
        }
+       stat_ev_tim_pop("be_lv_get");
 
-       return NULL;
+       return res;
 }
 
-static struct _be_lv_info_node_t *be_lv_get_or_set(struct _be_lv_t *li, ir_node *bl, ir_node *irn)
+static be_lv_info_node_t *be_lv_get_or_set(be_lv_t *li, ir_node *bl,
+                                           ir_node *irn)
 {
-       struct _be_lv_info_t *irn_live = phase_get_or_set_irn_data(&li->ph, bl);
-
-       unsigned idx = get_irn_idx(irn);
+       be_lv_info_t *irn_live = ir_nodehashmap_get(be_lv_info_t, &li->map, bl);
+       if (irn_live == NULL) {
+               irn_live = OALLOCNZ(&li->obst, be_lv_info_t, LV_STD_SIZE);
+               irn_live[0].head.n_size = LV_STD_SIZE-1;
+               ir_nodehashmap_insert(&li->map, bl, irn_live);
+       }
 
        /* Get the position of the index in the array. */
-       unsigned pos = _be_liveness_bsearch(irn_live, idx);
+       unsigned pos = _be_liveness_bsearch(irn_live, irn);
 
        /* Get the record in question. 1 must be added, since the first record contains information about the array and must be skipped. */
-       struct _be_lv_info_node_t *res = &irn_live[pos + 1].u.node;
+       be_lv_info_node_t *res = &irn_live[pos + 1].node;
 
        /* Check, if the irn is in deed in the array. */
-       if(res->idx != idx) {
-               struct _be_lv_info_t *payload;
-               unsigned n_members = irn_live[0].u.head.n_members;
-               unsigned n_size    = irn_live[0].u.head.n_size;
+       if (res->node != irn) {
+               be_lv_info_t *payload;
+               unsigned n_members = irn_live[0].head.n_members;
+               unsigned n_size    = irn_live[0].head.n_size;
                unsigned i;
 
-               if(n_members == n_size - 1) {
-                       unsigned new_size = 2 * n_size * sizeof(irn_live[0]);
-                       struct _be_lv_info_t *nw = phase_alloc(&li->ph, new_size);
-                       memcpy(nw, irn_live, new_size);
-                       nw[0].u.head.n_size = new_size;
+               if (n_members + 1 >= n_size) {
+                       /* double the array size. Remember that the first entry is
+                        * metadata about the array and not a real array element */
+                       unsigned old_size_bytes  = (n_size + 1) * sizeof(irn_live[0]);
+                       unsigned new_size        = (2 * n_size) + 1;
+                       size_t   new_size_bytes  = new_size * sizeof(irn_live[0]);
+                       be_lv_info_t *nw = OALLOCN(&li->obst, be_lv_info_t, new_size);
+                       memcpy(nw, irn_live, old_size_bytes);
+                       memset(((char*) nw) + old_size_bytes, 0,
+                              new_size_bytes - old_size_bytes);
+                       nw[0].head.n_size = new_size - 1;
                        irn_live = nw;
-                       phase_set_irn_data(&li->ph, irn, nw);
+                       ir_nodehashmap_insert(&li->map, bl, nw);
                }
 
                payload = &irn_live[1];
-               for(i = n_members; i > pos; --i) {
+               for (i = n_members; i > pos; --i) {
                        payload[i] = payload[i - 1];
                }
 
-               ++irn_live[0].u.head.n_members;
-
-               res = &payload[pos].u.node;
-               res->idx    = idx;
-               res->flags  = 0;
-       }
-
-#ifdef LV_INTESIVE_CHECKS
-       {
-               unsigned i;
-               unsigned n = irn_live[0].u.head.n_members;
-               unsigned last = 0;
-               struct _be_lv_info_t *payload = &irn_live[1];
+               ++irn_live[0].head.n_members;
 
-               for(i = 0; i < n; ++i) {
-                       assert(payload[i].u.node.idx >= last);
-                       last = payload[i].u.node.idx;
-               }
+               res        = &payload[pos].node;
+               res->node  = irn;
+               res->flags = 0;
        }
-#endif
 
        return res;
 }
 
+typedef struct lv_remove_walker_t {
+       be_lv_t       *lv;
+       ir_node const *irn;
+} lv_remove_walker_t;
+
 /**
  * Removes a node from the list of live variables of a block.
- * @return 1 if the node was live at that block, 0 if not.
  */
-static int be_lv_remove(struct _be_lv_t *li, ir_node *bl, ir_node *irn)
+static void lv_remove_irn_walker(ir_node *const bl, void *const data)
 {
-       struct _be_lv_info_t *irn_live = phase_get_irn_data(&li->ph, bl);
-
-       if(irn_live) {
-               unsigned n   = irn_live[0].u.head.n_members;
-               unsigned idx = get_irn_idx(irn);
-               unsigned pos = _be_liveness_bsearch(irn_live, idx);
-               struct _be_lv_info_t *payload  = irn_live + 1;
-               struct _be_lv_info_node_t *res = &payload[pos].u.node;
+       lv_remove_walker_t *const w        = (lv_remove_walker_t*)data;
+       ir_node      const *const irn      = w->irn;
+       be_lv_info_t       *const irn_live = ir_nodehashmap_get(be_lv_info_t, &w->lv->map, bl);
+       if (irn_live != NULL) {
+               unsigned n   = irn_live[0].head.n_members;
+               unsigned pos = _be_liveness_bsearch(irn_live, irn);
+               be_lv_info_t *payload  = irn_live + 1;
+               be_lv_info_node_t *res = &payload[pos].node;
 
                /* The node is in deed in the block's array. Let's remove it. */
-               if(res->idx == idx) {
+               if (res->node == irn) {
                        unsigned i;
 
-                       for(i = pos + 1; i < n; ++i)
+                       for (i = pos + 1; i < n; ++i)
                                payload[i - 1] = payload[i];
 
-                       payload[n - 1].u.node.idx   = 0;
-                       payload[n - 1].u.node.flags = 0;
+                       payload[n - 1].node.node  = NULL;
+                       payload[n - 1].node.flags = 0;
 
-                       --irn_live[0].u.head.n_members;
+                       --irn_live[0].head.n_members;
                        DBG((dbg, LEVEL_3, "\tdeleting %+F from %+F at pos %d\n", irn, bl, pos));
-                       return 1;
                }
        }
-
-       return 0;
 }
 
-static void register_node(be_lv_t *lv, const ir_node *irn)
-{
-       unsigned idx = get_irn_idx(irn);
-       if(idx >= bitset_size(lv->nodes)) {
-               bitset_t *nw = bitset_malloc(2 * idx);
-               bitset_copy(nw, lv->nodes);
-               bitset_free(lv->nodes);
-               lv->nodes = nw;
-       }
-
-       bitset_set(lv->nodes, idx);
-}
-
-/**
- * Mark a node as live-in in a block.
- */
-static INLINE void mark_live_in(be_lv_t *lv, ir_node *block, ir_node *irn)
-{
-       struct _be_lv_info_node_t *n = be_lv_get_or_set(lv, block, irn);
-       DBG((dbg, LEVEL_2, "marking %+F live in at %+F\n", irn, block));
-       n->flags |= be_lv_state_in;
-       register_node(lv, irn);
-}
-
-/**
- * Mark a node as live-out in a block.
- */
-static INLINE void mark_live_out(be_lv_t *lv, ir_node *block, ir_node *irn)
-{
-       struct _be_lv_info_node_t *n = be_lv_get_or_set(lv, block, irn);
-       DBG((dbg, LEVEL_2, "marking %+F live out at %+F\n", irn, block));
-       n->flags |= be_lv_state_out | be_lv_state_end;
-       register_node(lv, irn);
-}
-
-/**
- * Mark a node as live-end in a block.
- */
-static INLINE void mark_live_end(be_lv_t *lv, ir_node *block, ir_node *irn)
-{
-       struct _be_lv_info_node_t *n = be_lv_get_or_set(lv, block, irn);
-       DBG((dbg, LEVEL_2, "marking %+F live end at %+F\n", irn, block));
-       n->flags |= be_lv_state_end;
-       register_node(lv, irn);
-}
+static struct {
+       be_lv_t  *lv;         /**< The liveness object. */
+       ir_node  *def;        /**< The node (value). */
+       ir_node  *def_block;  /**< The block of def. */
+} re;
 
 /**
  * Mark a node (value) live out at a certain block. Do this also
  * transitively, i.e. if the block is not the block of the value's
  * definition, all predecessors are also marked live.
- * @param def The node (value).
  * @param block The block to mark the value live out of.
- * @param visited A set were all visited blocks are recorded.
- * @param is_true_out Is the node real out there or only live at the end
- * of the block.
+ * @param state The liveness bits to set, either end or end+out.
  */
-static void live_end_at_block(be_lv_t *lv, ir_node *def, ir_node *block, bitset_t *visited, int is_true_out)
+static void live_end_at_block(ir_node *const block, be_lv_state_t const state)
 {
-       mark_live_end(lv, block, def);
-       if(is_true_out)
-               mark_live_out(lv, block, def);
+       be_lv_info_node_t *const n      = be_lv_get_or_set(re.lv, block, re.def);
+       be_lv_state_t      const before = n->flags;
 
-       if(!bitset_contains_irn(visited, block)) {
-               bitset_add_irn(visited, block);
+       assert(state == be_lv_state_end || state == (be_lv_state_end | be_lv_state_out));
+       DBG((dbg, LEVEL_2, "marking %+F live %s at %+F\n", re.def, state & be_lv_state_out ? "end+out" : "end", block));
+       n->flags |= state;
 
-               /*
-               * If this block is not the definition block, we have to go up
-               * further.
-               */
-               if(get_nodes_block(def) != block) {
-                       int i, n;
+       /* There is no need to recurse further, if we where here before (i.e., any
+        * live state bits were set before). */
+       if (before != be_lv_state_none)
+               return;
 
-                       mark_live_in(lv, block, def);
+       /* Stop going up further, if this is the block of the definition. */
+       if (re.def_block == block)
+               return;
 
-                       for(i = 0, n = get_Block_n_cfgpreds(block); i < n; ++i)
-                               live_end_at_block(lv, def, get_Block_cfgpred_block(block, i), visited, 1);
-               }
+       DBG((dbg, LEVEL_2, "marking %+F live in at %+F\n", re.def, block));
+       n->flags |= be_lv_state_in;
 
+       for (int i = get_Block_n_cfgpreds(block); i-- != 0;) {
+               ir_node *const pred_block = get_Block_cfgpred_block(block, i);
+               live_end_at_block(pred_block, be_lv_state_end | be_lv_state_out);
        }
 }
 
-struct _lv_walker_t {
-       be_lv_t *lv;
-       void *data;
-};
-
 /**
  * Liveness analysis for a value.
- * This functions is meant to be called by a firm walker, to compute the
- * set of all blocks a value is live in.
- * @param irn The node (value).
- * @param env Ignored.
+ * Compute the set of all blocks a value is live in.
+ * @param irn     The node (value).
  */
-static void liveness_for_node(ir_node *irn, void *data)
+static void liveness_for_node(ir_node *irn)
 {
-       struct _lv_walker_t *walker = data;
-       be_lv_t *lv       = walker->lv;
-       bitset_t *visited = walker->data;
-       const ir_edge_t *edge;
-       ir_node *def_block;
+       ir_node *const def_block = get_nodes_block(irn);
 
-       /* Don't compute liveness information for non-data nodes. */
-       if(!is_liveness_node(irn))
-               return;
-
-       bitset_clear_all(visited);
-       def_block = get_nodes_block(irn);
+       re.def       = irn;
+       re.def_block = def_block;
 
        /* Go over all uses of the value */
        foreach_out_edge(irn, edge) {
                ir_node *use = edge->src;
                ir_node *use_block;
 
+               DBG((dbg, LEVEL_4, "%+F: use at %+F, pos %d in %+F\n", irn, use, edge->pos, get_block(use)));
+               assert(get_irn_n(use, edge->pos) == irn);
+
                /*
                 * If the usage is no data node, skip this use, since it does not
                 * affect the liveness of the node.
                 */
-               if(!is_liveness_node(use))
+               if (!is_liveness_node(use))
                        continue;
 
                /* Get the block where the usage is in. */
@@ -375,154 +266,141 @@ static void liveness_for_node(ir_node *irn, void *data)
                 * through which the value reaches the phi function and mark the
                 * value as live out of that block.
                 */
-               if(is_Phi(use)) {
+               if (is_Phi(use)) {
                        ir_node *pred_block = get_Block_cfgpred_block(use_block, edge->pos);
-                       live_end_at_block(lv, irn, pred_block, visited, 0);
+                       live_end_at_block(pred_block, be_lv_state_end);
                }
 
                /*
                 * Else, the value is live in at this block. Mark it and call live
                 * out on the predecessors.
                 */
-               else if(def_block != use_block) {
-                       int i, n;
+               else if (def_block != use_block) {
+                       int i;
 
-                       mark_live_in(lv, use_block, irn);
+                       be_lv_info_node_t *const n = be_lv_get_or_set(re.lv, use_block, irn);
+                       DBG((dbg, LEVEL_2, "marking %+F live in at %+F\n", irn, use_block));
+                       n->flags |= be_lv_state_in;
 
-                       for(i = 0, n = get_Block_n_cfgpreds(use_block); i < n; ++i) {
+                       for (i = get_Block_n_cfgpreds(use_block) - 1; i >= 0; --i) {
                                ir_node *pred_block = get_Block_cfgpred_block(use_block, i);
-                               live_end_at_block(lv, irn, pred_block, visited, 1);
+                               live_end_at_block(pred_block, be_lv_state_end | be_lv_state_out);
                        }
                }
        }
 }
 
-static void lv_remove_irn_walker(ir_node *bl, void *data)
+/**
+ * Walker, collect all nodes for which we want calculate liveness info
+ * on an obstack.
+ */
+static void collect_liveness_nodes(ir_node *irn, void *data)
 {
-       struct _lv_walker_t *w = data;
-       be_lv_t *lv  = w->lv;
-       ir_node *irn = w->data;
-       be_lv_remove(lv, bl, irn);
+       ir_node **nodes = (ir_node**)data;
+       if (is_liveness_node(irn))
+               nodes[get_irn_idx(irn)] = irn;
 }
 
-static const char *lv_flags_to_str(unsigned flags)
+void be_liveness_compute_sets(be_lv_t *lv)
 {
-       static const char *states[] = {
-               "---",
-               "i--",
-               "-e-",
-               "ie-",
-               "--o",
-               "i-o",
-               "-eo",
-               "ieo"
-       };
-
-       return states[flags & 7];
-}
+       int       i;
+       int       n;
 
-static void lv_dump_block(void *context, FILE *f, const ir_node *bl)
-{
-       if(is_Block(bl)) {
-               be_lv_t *lv = context;
-               struct _be_lv_info_t *info = phase_get_irn_data(&lv->ph, bl);
+       if (lv->sets_valid)
+               return;
 
-               fprintf(f, "liveness:\n");
-               if(info) {
-                       unsigned n = info[0].u.head.n_members;
-                       unsigned i;
+       be_timer_push(T_LIVE);
+       ir_nodehashmap_init(&lv->map);
+       obstack_init(&lv->obst);
 
-                       for(i = 0; i < n; ++i) {
-                               struct _be_lv_info_node_t *n = &info[i+1].u.node;
-                               ir_fprintf(f, "%s %+F\n", lv_flags_to_str(n->flags), get_idx_irn(lv->irg, n->idx));
-                       }
-               }
+       n = get_irg_last_idx(lv->irg);
+       ir_node **const nodes = NEW_ARR_FZ(ir_node*, n);
+
+       /* inserting the variables sorted by their ID is probably
+        * more efficient since the binary sorted set insertion
+        * will not need to move around the data. */
+       irg_walk_graph(lv->irg, NULL, collect_liveness_nodes, nodes);
+
+       re.lv = lv;
+
+       for (i = 0; i < n; ++i) {
+               if (nodes[i] != NULL)
+                       liveness_for_node(nodes[i]);
        }
+
+       DEL_ARR_F(nodes);
+
+       be_timer_pop(T_LIVE);
+
+       lv->sets_valid = true;
 }
 
-static void *lv_phase_data_init(ir_phase *phase, ir_node *irn, void *old)
+void be_liveness_compute_chk(be_lv_t *lv)
 {
-       struct _be_lv_info_t *info = phase_alloc(phase, LV_STD_SIZE * sizeof(info[0]));
-       memset(info, 0, LV_STD_SIZE * sizeof(info[0]));
-       info[0].u.head.n_size = LV_STD_SIZE - 1;
-       return info;
+       if (lv->lvc != NULL)
+               return;
+       lv->lvc = lv_chk_new(lv->irg);
 }
 
-static void compute_liveness(be_lv_t *lv)
+void be_liveness_invalidate_sets(be_lv_t *lv)
 {
-       struct _lv_walker_t w;
-       w.lv   = lv;
-       w.data = bitset_malloc(get_irg_last_idx(lv->irg));
-       irg_walk_graph(lv->irg, liveness_for_node, NULL, &w);
-       bitset_free(w.data);
+       if (!lv->sets_valid)
+               return;
+       obstack_free(&lv->obst, NULL);
+       ir_nodehashmap_destroy(&lv->map);
+       lv->sets_valid = false;
 }
 
-/* Compute the inter block liveness for a graph. */
-be_lv_t *be_liveness(ir_graph *irg)
+void be_liveness_invalidate_chk(be_lv_t *lv)
 {
-       be_lv_t *lv = xmalloc(sizeof(lv[0]));
+       be_liveness_invalidate_sets(lv);
 
-       memset(lv, 0, sizeof(lv[0]));
-       lv->irg = irg;
-       lv->nodes = bitset_malloc(2 * get_irg_last_idx(irg));
-       lv->hook_info.context = lv;
-       lv->hook_info.hook._hook_node_info = lv_dump_block;
-       register_hook(hook_node_info, &lv->hook_info);
-       phase_init(&lv->ph, "liveness", irg, PHASE_DEFAULT_GROWTH, lv_phase_data_init, NULL);
-       compute_liveness(lv);
-
-       return lv;
+       if (lv->lvc == NULL)
+               return;
+       lv_chk_free(lv->lvc);
+       lv->lvc = NULL;
 }
 
-void be_liveness_recompute(be_lv_t *lv)
+be_lv_t *be_liveness_new(ir_graph *irg)
 {
-       unsigned last_idx = get_irg_last_idx(lv->irg);
-       if(last_idx >= bitset_size(lv->nodes)) {
-               bitset_free(lv->nodes);
-               lv->nodes = bitset_malloc(last_idx * 2);
-       }
+       be_lv_t *lv = XMALLOCZ(be_lv_t);
 
-       else
-               bitset_clear_all(lv->nodes);
+       lv->irg = irg;
 
-       phase_free(&lv->ph);
-       phase_init(&lv->ph, "liveness", lv->irg, PHASE_DEFAULT_GROWTH, lv_phase_data_init, NULL);
-       compute_liveness(lv);
+       return lv;
 }
 
-
 void be_liveness_free(be_lv_t *lv)
 {
-       unregister_hook(hook_node_info, &lv->hook_info);
-       phase_free(&lv->ph);
-       bitset_free(lv->nodes);
-       free(lv);
+       be_liveness_invalidate_sets(lv);
+       be_liveness_invalidate_chk(lv);
+
+       xfree(lv);
 }
 
-void be_liveness_remove(be_lv_t *lv, ir_node *irn)
+void be_liveness_remove(be_lv_t *lv, const ir_node *irn)
 {
-       unsigned idx = get_irn_idx(irn);
-       struct _lv_walker_t w;
-
-       /*
-        * Removes a single irn from the liveness information.
-        * Since an irn can only be live at blocks dominated by the block of its
-        * definition, we only have to process that dominance subtree.
-        */
-       w.lv   = lv;
-       w.data = irn;
-       dom_tree_walk(get_nodes_block(irn), lv_remove_irn_walker, NULL, &w);
-       if(idx <= bitset_size(lv->nodes))
-               bitset_clear(lv->nodes, idx);
+       if (lv->sets_valid) {
+               lv_remove_walker_t w;
+
+               /*
+                * Removes a single irn from the liveness information.
+                * Since an irn can only be live at blocks dominated by the block of its
+                * definition, we only have to process that dominance subtree.
+                */
+               w.lv  = lv;
+               w.irn = irn;
+               dom_tree_walk(get_nodes_block(irn), lv_remove_irn_walker, NULL, &w);
+       }
 }
 
 void be_liveness_introduce(be_lv_t *lv, ir_node *irn)
 {
-       struct _lv_walker_t w;
-       w.lv   = lv;
-       w.data = bitset_malloc(get_irg_last_idx(lv->irg));
-       liveness_for_node(irn, &w);
-       bitset_free(w.data);
+       /* Don't compute liveness information for non-data nodes. */
+       if (lv->sets_valid && is_liveness_node(irn)) {
+               re.lv = lv;
+               liveness_for_node(irn);
+       }
 }
 
 void be_liveness_update(be_lv_t *lv, ir_node *irn)
@@ -531,221 +409,106 @@ void be_liveness_update(be_lv_t *lv, ir_node *irn)
        be_liveness_introduce(lv, irn);
 }
 
-static void lv_add_missing_walker(ir_node *irn, void *data)
-{
-       struct _lv_walker_t *w = data;
-       if(!is_Block(irn) && !bitset_contains_irn(w->lv->nodes, irn)) {
-               liveness_for_node(irn, w);
-       }
-}
-
-void be_liveness_add_missing(be_lv_t *lv)
-{
-       struct _lv_walker_t w;
-       w.lv   = lv;
-       w.data = bitset_malloc(get_irg_last_idx(lv->irg));
-       irg_walk_graph(lv->irg, lv_add_missing_walker, NULL, &w);
-       bitset_free(w.data);
-}
-
-static void lv_check_walker(ir_node *bl, void *data)
-{
-       struct _lv_walker_t *w = data;
-       be_lv_t *lv    = w->lv;
-       be_lv_t *fresh = w->data;
-
-       struct _be_lv_info_t *curr = phase_get_irn_data(&lv->ph, bl);
-       struct _be_lv_info_t *fr   = phase_get_irn_data(&fresh->ph, bl);
-
-       if(!fr && curr && curr[0].u.head.n_members > 0) {
-               unsigned i;
-
-               ir_fprintf(stderr, "%+F liveness should be empty but current liveness contains:\n", bl);
-               for(i = 0; i < curr[0].u.head.n_members; ++i) {
-                       ir_fprintf(stderr, "\t%+F\n", get_idx_irn(lv->irg, curr[1 + i].u.node.idx));
-               }
-       }
-
-       else if(curr) {
-               unsigned n_curr  = curr[0].u.head.n_members;
-               unsigned n_fresh = fr[0].u.head.n_members;
-
-               unsigned i;
-
-               if(n_curr != n_fresh) {
-                       ir_fprintf(stderr, "%+F: liveness set sizes differ. curr %d, correct %d\n", bl, n_curr, n_fresh);
-
-                       ir_fprintf(stderr, "current:\n");
-                       for(i = 0; i < n_curr; ++i) {
-                               struct _be_lv_info_node_t *n = &curr[1 + i].u.node;
-                               ir_fprintf(stderr, "%+F %u %+F %s\n", bl, i, get_idx_irn(lv->irg, n->idx), lv_flags_to_str(n->flags));
-                       }
-
-                       ir_fprintf(stderr, "correct:\n");
-                       for(i = 0; i < n_fresh; ++i) {
-                               struct _be_lv_info_node_t *n = &fr[1 + i].u.node;
-                               ir_fprintf(stderr, "%+F %u %+F %s\n", bl, i, get_idx_irn(lv->irg, n->idx), lv_flags_to_str(n->flags));
-                       }
-               }
-       }
-}
-
-void be_liveness_check(be_lv_t *lv)
+void be_liveness_transfer(const arch_register_class_t *cls,
+                          ir_node *node, ir_nodeset_t *nodeset)
 {
-       struct _lv_walker_t w;
-       be_lv_t *fresh = be_liveness(lv->irg);
+       /* You should better break out of your loop when hitting the first phi
+        * function. */
+       assert(!is_Phi(node) && "liveness_transfer produces invalid results for phi nodes");
 
-       w.lv   = lv;
-       w.data = fresh;
-       irg_block_walk_graph(lv->irg, lv_check_walker, NULL, &w);
-       be_liveness_free(fresh);
-}
+       be_foreach_definition(node, cls, value, req,
+               ir_nodeset_remove(nodeset, value);
+       );
 
-
-static void lv_dump_block_walker(ir_node *irn, void *data)
-{
-       struct _lv_walker_t *w = data;
-       if(is_Block(irn))
-               lv_dump_block(w->lv, w->data, irn);
+       be_foreach_use(node, cls, in_req, op, op_req,
+               ir_nodeset_insert(nodeset, op);
+       );
 }
 
 
-/* Dump the liveness information for a graph. */
-void be_liveness_dump(const be_lv_t *lv, FILE *f)
-{
-       struct _lv_walker_t w;
-
-       w.lv   = (be_lv_t *) lv;
-       w.data = f;
-       irg_block_walk_graph(lv->irg, lv_dump_block_walker, NULL, &w);
-}
 
-/* Dump the liveness information for a graph. */
-void be_liveness_dumpto(const be_lv_t *lv, const char *cls_name)
+void be_liveness_end_of_block(const be_lv_t *lv,
+                              const arch_register_class_t *cls,
+                              const ir_node *block, ir_nodeset_t *live)
 {
-       FILE *f;
-       char buf[128];
-       ir_snprintf(buf, sizeof(buf), "%F_%s-live.txt", lv->irg, cls_name);
-       if((f = fopen(buf, "wt")) != NULL) {
-               be_liveness_dump(lv, f);
-               fclose(f);
+       assert(lv->sets_valid && "live sets must be computed");
+       be_lv_foreach_cls(lv, block, be_lv_state_end, cls, node) {
+               ir_nodeset_insert(live, node);
        }
 }
 
-/**
- * Walker: checks the every predecessors of a node dominate
- * the note.
- */
-static void dom_check(ir_node *irn, void *data)
-{
-       int *problem_found = data;
-
-       if(!is_Block(irn) && irn != get_irg_end(get_irn_irg(irn))) {
-               int i, n;
-               ir_node *bl = get_nodes_block(irn);
-
-               for(i = 0, n = get_irn_arity(irn); i < n; ++i) {
-                       ir_node *op     = get_irn_n(irn, i);
-                       ir_node *def_bl = get_nodes_block(op);
-                       ir_node *use_bl = bl;
 
-                       if(is_Phi(irn))
-                               use_bl = get_Block_cfgpred_block(bl, i);
 
-                       if(get_irn_opcode(use_bl) != iro_Bad
-                            && get_irn_opcode(def_bl) != iro_Bad
-                            && !block_dominates(def_bl, use_bl)) {
-                               ir_fprintf(stderr, "Verify warning: %+F in %+F must dominate %+F for user %+F (%s)\n", op, def_bl, use_bl, irn, get_irg_dump_name(get_irn_irg(op)));
-                               *problem_found = 1;
-                       }
-               }
+void be_liveness_nodes_live_before(be_lv_t const *const lv, arch_register_class_t const *const cls, ir_node const *const pos, ir_nodeset_t *const live)
+{
+       ir_node *const bl = get_nodes_block(pos);
+       be_liveness_end_of_block(lv, cls, bl, live);
+       sched_foreach_reverse(bl, irn) {
+               be_liveness_transfer(cls, irn, live);
+               if (irn == pos)
+                       return;
        }
 }
 
-/* Check, if the SSA dominance property is fulfilled. */
-int be_check_dominance(ir_graph *irg)
+static void collect_node(ir_node *irn, void *data)
 {
-       int problem_found = 0;
-
-       assure_doms(irg);
-       irg_walk_graph(irg, dom_check, NULL, &problem_found);
-
-       return !problem_found;
+       struct obstack *obst = (struct obstack*)data;
+       obstack_ptr_grow(obst, irn);
 }
 
-pset *be_liveness_transfer(const arch_env_t *arch_env, const arch_register_class_t *cls, ir_node *irn, pset *live)
+static void be_live_chk_compare(be_lv_t *lv, lv_chk_t *lvc)
 {
-       int i, n;
-
-       /* You should better break out of your loop when hitting the first phi function. */
-       assert(!is_Phi(irn) && "liveness_transfer produces invalid results for phi nodes");
+       ir_graph *irg    = lv->irg;
 
-       if(arch_irn_consider_in_reg_alloc(arch_env, cls, irn)) {
-               ir_node *del = pset_remove_ptr(live, irn);
-               (void) del;
-               assert(irn == del);
-       }
+       struct obstack obst;
+       ir_node **nodes;
+       ir_node **blocks;
+       int i, j;
 
-       for(i = 0, n = get_irn_arity(irn); i < n; ++i) {
-               ir_node *op = get_irn_n(irn, i);
+       obstack_init(&obst);
 
-               if(arch_irn_consider_in_reg_alloc(arch_env, cls, op))
-                       pset_insert_ptr(live, op);
-       }
+       irg_block_walk_graph(irg, collect_node, NULL, &obst);
+       obstack_ptr_grow(&obst, NULL);
+       blocks = (ir_node**)obstack_finish(&obst);
 
-       return live;
-}
+       irg_walk_graph(irg, collect_node, NULL, &obst);
+       obstack_ptr_grow(&obst, NULL);
+       nodes = (ir_node**)obstack_finish(&obst);
 
-pset *be_liveness_end_of_block(const be_lv_t *lv, const arch_env_t *arch_env, const arch_register_class_t *cls, const ir_node *bl, pset *live)
-{
-       int i;
-       be_lv_foreach(lv, bl, be_lv_state_end, i) {
-               ir_node *irn = be_lv_get_irn(lv, bl, i);
-               if(arch_irn_consider_in_reg_alloc(arch_env, cls, irn))
-                       pset_insert_ptr(live, irn);
-       }
-
-       return live;
-}
-
-pset *be_liveness_nodes_live_at(const be_lv_t *lv, const arch_env_t *arch_env, const arch_register_class_t *cls, const ir_node *pos, pset *live)
-{
-       const ir_node *bl = is_Block(pos) ? pos : get_nodes_block(pos);
-       ir_node *irn;
+       stat_ev_ctx_push("be_lv_chk_compare");
+       for (j = 0; nodes[j]; ++j) {
+               ir_node *irn = nodes[j];
+               if (is_Block(irn))
+                       continue;
 
-       be_liveness_end_of_block(lv, arch_env, cls, bl, live);
-       sched_foreach_reverse(bl, irn) {
-               /*
-                * If we encounter the node we want to insert the Perm after,
-                * exit immediately, so that this node is still live
-                */
-               if(irn == pos)
-                       return live;
+               for (i = 0; blocks[i]; ++i) {
+                       ir_node *bl = blocks[i];
+                       int lvr_in  = be_is_live_in (lv, bl, irn);
+                       int lvr_out = be_is_live_out(lv, bl, irn);
+                       int lvr_end = be_is_live_end(lv, bl, irn);
 
-               be_liveness_transfer(arch_env, cls, irn, live);
-       }
+                       int lvc_in  = lv_chk_bl_in (lvc, bl, irn);
+                       int lvc_out = lv_chk_bl_out(lvc, bl, irn);
+                       int lvc_end = lv_chk_bl_end(lvc, bl, irn);
 
-       return live;
-}
+                       if (lvr_in - lvc_in != 0)
+                               ir_fprintf(stderr, "live in  info for %+F at %+F differs: nml: %d, chk: %d\n", irn, bl, lvr_in, lvc_in);
 
-pset *be_liveness_nodes_live_at_input(const be_lv_t *lv, const arch_env_t *arch_env, const arch_register_class_t *cls, const ir_node *pos, pset *live)
-{
-       const ir_node *bl = is_Block(pos) ? pos : get_nodes_block(pos);
-       ir_node *irn;
+                       if (lvr_end - lvc_end != 0)
+                               ir_fprintf(stderr, "live end info for %+F at %+F differs: nml: %d, chk: %d\n", irn, bl, lvr_end, lvc_end);
 
-       be_liveness_end_of_block(lv, arch_env, cls, bl, live);
-       sched_foreach_reverse(bl, irn) {
-               be_liveness_transfer(arch_env, cls, irn, live);
-               if(irn == pos)
-                       return live;
+                       if (lvr_out - lvc_out != 0)
+                               ir_fprintf(stderr, "live out info for %+F at %+F differs: nml: %d, chk: %d\n", irn, bl, lvr_out, lvc_out);
+               }
        }
+       stat_ev_ctx_pop("be_lv_chk_compare");
 
-       return live;
+       obstack_free(&obst, NULL);
 }
 
+BE_REGISTER_MODULE_CONSTRUCTOR(be_init_live)
 void be_init_live(void)
 {
+       (void)be_live_chk_compare;
        FIRM_DBG_REGISTER(dbg, "firm.be.liveness");
 }
-
-BE_REGISTER_MODULE_CONSTRUCTOR(be_init_live);