cleanup: Remove end-of-if/for/function comments.
[libfirm] / ir / opt / opt_blocks.c
index 273470d..7cc8015 100644 (file)
@@ -1,5 +1,5 @@
 /*
- * Copyright (C) 1995-2008 University of Karlsruhe.  All right reserved.
+ * Copyright (C) 1995-2011 University of Karlsruhe.  All right reserved.
  *
  * This file is part of libFirm.
  *
  * @file
  * @brief   Combining congruent blocks
  * @author  Michael Beck
- * @version $Id$
  *
- * This phase find congruent blocks. Works currently for
- * predecessors of the end block only.
+ * This phase find congruent blocks.
  * Two block are congruent, if they contains only equal calculations.
  */
 #include "config.h"
-#include "ircons.h"
+
 #include "iroptimize.h"
+#include "ircons.h"
 #include "irgmod.h"
 #include "irgraph_t.h"
 #include "irnode_t.h"
 #include "trouts.h"
 #include "irgwalk.h"
 #include "set.h"
+#include "irpass.h"
 #include "debug.h"
+#include "util.h"
 
-/* define this for gneral block shaping (buggy yet) */
-#undef GENERAL_SHAPE
+/* define this for general block shaping: congruent blocks
+   are found not only before the end block but anywhere in the graph */
+#define GENERAL_SHAPE
 
 typedef struct partition_t     partition_t;
 typedef struct block_t         block_t;
@@ -51,15 +53,20 @@ typedef struct phi_t           phi_t;
 typedef struct opcode_key_t    opcode_key_t;
 typedef struct listmap_entry_t listmap_entry_t;
 typedef struct environment_t   environment_t;
+typedef struct pred_t          pred_t;
 
 /** An opcode map key. */
 struct opcode_key_t {
-       ir_opcode   code;   /**< The Firm opcode. */
+       unsigned    code;   /**< The Firm opcode. */
        ir_mode     *mode;  /**< The mode of all nodes in the partition. */
        int         arity;  /**< The arity of this opcode (needed for Phi etc. */
        union {
-               long      proj;   /**< For Proj nodes, its proj number */
-               ir_entity *ent;   /**< For Sel Nodes, its entity */
+               long            proj;   /**< For Proj nodes, its proj number */
+               ir_entity       *ent;   /**< For Sel nodes, its entity */
+               ir_tarval       *tv;    /**< For Const nodes, its tarval */
+               symconst_symbol sym;    /**< For SymConst nodes, its symbol .*/
+               void            *addr;  /**< Alias all addresses. */
+               int             intVal; /**< For Conv/Div nodes: strict/remainderless. */
        } u;
 };
 
@@ -84,7 +91,8 @@ struct block_t {
        node_t     *cf_root;     /**< The control flow root node of this block. */
        pair_t     *input_pairs; /**< The list of inputs to this block. */
        phi_t      *phis;        /**< The list of Phis in this block. */
-       block_t    *all_next;    /**< links all craeted blocks. */
+       block_t    *all_next;    /**< Links all created blocks. */
+       int        meet_input;   /**< Input number of this block in the meet-block. */
 };
 
 /** A node. */
@@ -104,7 +112,7 @@ struct environment_t {
        struct obstack  obst;           /** obstack for temporary data */
 };
 
-/** A node, input index pair. */
+/** A (node, input index) pair. */
 struct pair_t {
        pair_t  *next;    /**< Points to the next pair entry. */
        ir_node *irn;     /**< The IR-node. */
@@ -119,6 +127,12 @@ struct phi_t {
        ir_node **ins;    /**< A new in array once allocated. */
 };
 
+/** Describes a predecessor input. */
+struct pred_t {
+       ir_node *pred;  /**< The predecessor. */
+       int     index;  /**< Its input index. */
+};
+
 /**
  * An entry in the list_map.
  */
@@ -140,15 +154,15 @@ typedef struct listmap_t {
 DEBUG_ONLY(static firm_dbg_module_t *dbg;)
 
 /** Next partition number. */
-DEBUG_ONLY(static unsigned part_nr = 0);
+DEBUG_ONLY(static unsigned part_nr = 0;)
 
 #ifdef DEBUG_libfirm
 /**
  * Dump partition to output.
  */
-static void dump_partition(const char *msg, const partition_t *part) {
-       const block_t *block;
-       int           first = 1;
+static void dump_partition(const char *msg, const partition_t *part)
+{
+       int first = 1;
 
        DB((dbg, LEVEL_2, " %s part%u (%u blocks) {\n  ", msg, part->nr, part->n_blocks));
        list_for_each_entry(block_t, block, &part->blocks, block_list) {
@@ -156,12 +170,13 @@ static void dump_partition(const char *msg, const partition_t *part) {
                first = 0;
        }
        DB((dbg, LEVEL_2, "\n }\n"));
-}  /* dump_partition */
+}
 
 /**
  * Dumps a list.
  */
-static void dump_list(const char *msg, const block_t *block) {
+static void dump_list(const char *msg, const block_t *block)
+{
        const block_t *p;
        int           first = 1;
 
@@ -171,7 +186,7 @@ static void dump_list(const char *msg, const block_t *block) {
                first = 0;
        }
        DB((dbg, LEVEL_3, "\n  }\n"));
-}  /* do_dump_list */
+}
 #else
 #define dump_partition(msg, part)
 #define dump_list(msg, block)
@@ -180,32 +195,35 @@ static void dump_list(const char *msg, const block_t *block) {
 /**
  * Compare two pointer values of a listmap.
  */
-static int listmap_cmp_ptr(const void *elt, const void *key, size_t size) {
-       const listmap_entry_t *e1 = elt;
-       const listmap_entry_t *e2 = key;
+static int listmap_cmp_ptr(const void *elt, const void *key, size_t size)
+{
+       const listmap_entry_t *e1 = (const listmap_entry_t*)elt;
+       const listmap_entry_t *e2 = (const listmap_entry_t*)key;
 
        (void) size;
        return e1->id != e2->id;
-}  /* listmap_cmp_ptr */
+}
 
 /**
  * Initializes a listmap.
  *
  * @param map  the listmap
  */
-static void listmap_init(listmap_t *map) {
+static void listmap_init(listmap_t *map)
+{
        map->map    = new_set(listmap_cmp_ptr, 16);
        map->values = NULL;
-}  /* listmap_init */
+}
 
 /**
  * Terminates a listmap.
  *
  * @param map  the listmap
  */
-static void listmap_term(listmap_t *map) {
+static void listmap_term(listmap_t *map)
+{
        del_set(map->map);
-}  /* listmap_term */
+}
 
 /**
  * Return the associated listmap entry for a given id.
@@ -215,13 +233,14 @@ static void listmap_term(listmap_t *map) {
  *
  * @return the associated listmap entry for the given id
  */
-static listmap_entry_t *listmap_find(listmap_t *map, void *id) {
+static listmap_entry_t *listmap_find(listmap_t *map, void *id)
+{
        listmap_entry_t key, *entry;
 
        key.id   = id;
        key.list = NULL;
        key.next = NULL;
-       entry = set_insert(map->map, &key, sizeof(key), HASH_PTR(id));
+       entry    = set_insert(listmap_entry_t, map->map, &key, sizeof(key), hash_ptr(id));
 
        if (entry->list == NULL) {
                /* a new entry, put into the list */
@@ -229,7 +248,7 @@ static listmap_entry_t *listmap_find(listmap_t *map, void *id) {
                map->values = entry;
        }
        return entry;
-}  /* listmap_find */
+}
 
 /**
  * Calculate the hash value for an opcode map entry.
@@ -238,50 +257,56 @@ static listmap_entry_t *listmap_find(listmap_t *map, void *id) {
  *
  * @return a hash value for the given opcode map entry
  */
-static unsigned opcode_hash(const opcode_key_t *entry) {
-       return (entry->mode - (ir_mode *)0) * 9 + entry->code + entry->u.proj * 3 + HASH_PTR(entry->u.ent) + entry->arity;
-}  /* opcode_hash */
+static unsigned opcode_hash(const opcode_key_t *entry)
+{
+       /* assume long >= int */
+       return (unsigned)(PTR_TO_INT(entry->mode) * 9 + entry->code + entry->u.proj * 3 + hash_ptr(entry->u.addr) + entry->arity);
+}
 
 /**
  * Compare two entries in the opcode map.
  */
-static int cmp_opcode(const void *elt, const void *key, size_t size) {
-       const opcode_key_t *o1 = elt;
-       const opcode_key_t *o2 = key;
+static int cmp_opcode(const void *elt, const void *key, size_t size)
+{
+       const opcode_key_t *o1 = (opcode_key_t*)elt;
+       const opcode_key_t *o2 = (opcode_key_t*)key;
 
        (void) size;
        return o1->code != o2->code || o1->mode != o2->mode ||
               o1->arity != o2->arity ||
-              o1->u.proj != o2->u.proj || o1->u.ent != o2->u.ent;
-}  /* cmp_opcode */
+              o1->u.proj != o2->u.proj || o1->u.addr != o2->u.addr;
+}
 
 /**
  * Creates a new empty partition and put in on the
  * partitions list.
  *
- * @param meet_block  the control flow meet block of thi partition
+ * @param meet_block  the control flow meet block of this partition
  * @param env         the environment
  */
-static partition_t *create_partition(ir_node *meet_block, environment_t *env) {
-       partition_t *part = obstack_alloc(&env->obst, sizeof(*part));
+static partition_t *create_partition(ir_node *meet_block, environment_t *env)
+{
+       partition_t *part = OALLOC(&env->obst, partition_t);
 
        INIT_LIST_HEAD(&part->blocks);
        part->meet_block = meet_block;
        part->n_blocks   = 0;
-       DEBUG_ONLY(part->nr = part_nr++);
+       DEBUG_ONLY(part->nr = part_nr++;)
        list_add_tail(&part->part_list, &env->partitions);
        return part;
-}  /* create_partition */
+}
 
 /**
  * Allocate a new block in the given partition.
  *
  * @param block      the IR-node
+ * @param meet_input Input number of this block in the meet-block
  * @param partition  the partition to add to
  * @param env        the environment
  */
-static block_t *create_block(ir_node *block, partition_t *partition, environment_t *env) {
-       block_t *bl = obstack_alloc(&env->obst, sizeof(*bl));
+static block_t *create_block(ir_node *block, int meet_input, partition_t *partition, environment_t *env)
+{
+       block_t *bl = OALLOC(&env->obst, block_t);
 
        set_irn_link(block, bl);
 
@@ -292,6 +317,7 @@ static block_t *create_block(ir_node *block, partition_t *partition, environment
        bl->cf_root     = NULL;
        bl->input_pairs = NULL;
        bl->phis        = NULL;
+       bl->meet_input  = meet_input;
 
        /* put it into the list of partition blocks */
        list_add_tail(&bl->block_list, &partition->blocks);
@@ -302,7 +328,7 @@ static block_t *create_block(ir_node *block, partition_t *partition, environment
        env->all_blocks = bl;
 
        return bl;
-}  /* create_block */
+}
 
 /**
  * Allocate a new node and add it to a blocks wait queue.
@@ -311,8 +337,9 @@ static block_t *create_block(ir_node *block, partition_t *partition, environment
  * @param block  the block to add to
  * @param env    the environment
  */
-static node_t *create_node(ir_node *irn, block_t *block, environment_t *env) {
-       node_t *node = obstack_alloc(&env->obst, sizeof(*node));
+static node_t *create_node(ir_node *irn, block_t *block, environment_t *env)
+{
+       node_t *node = OALLOC(&env->obst, node_t);
 
        node->node     = irn;
        node->is_input = 0;
@@ -320,7 +347,7 @@ static node_t *create_node(ir_node *irn, block_t *block, environment_t *env) {
        list_add_tail(&node->node_list, &block->nodes);
 
        return node;
-}  /* create_node */
+}
 
 /**
  * Add an input pair to a block.
@@ -330,8 +357,9 @@ static node_t *create_node(ir_node *irn, block_t *block, environment_t *env) {
  * @param idx    the index of the block input in node's predecessors
  * @param env    the environment
  */
-static void add_pair(block_t *block, ir_node *irn, int idx, environment_t *env) {
-       pair_t *pair = obstack_alloc(&env->obst, sizeof(*pair));
+static void add_pair(block_t *block, ir_node *irn, int idx, environment_t *env)
+{
+       pair_t *pair = OALLOC(&env->obst, pair_t);
 
        pair->next  = block->input_pairs;
        pair->irn   = irn;
@@ -339,7 +367,7 @@ static void add_pair(block_t *block, ir_node *irn, int idx, environment_t *env)
        pair->ins   = NULL;
 
        block->input_pairs = pair;
-}  /* add_pair */
+}
 
 /**
  * Add a Phi to a block.
@@ -348,20 +376,22 @@ static void add_pair(block_t *block, ir_node *irn, int idx, environment_t *env)
  * @param phi    the Phi node
  * @param env    the environment
  */
-static void add_phi(block_t *block, ir_node *phi, environment_t *env) {
-       phi_t *node = obstack_alloc(&env->obst, sizeof(*node));
+static void add_phi(block_t *block, ir_node *phi, environment_t *env)
+{
+       phi_t *node = OALLOC(&env->obst, phi_t);
 
        node->next = block->phis;
        node->phi  = phi;
        node->ins  = NULL;
 
        block->phis = node;
-}  /** add_phi */
+}
 
 /**
  * Creates an opcode from a node.
  */
-static opcode_key_t *opcode(const node_t *node, environment_t *env) {
+static opcode_key_t *opcode(const node_t *node, environment_t *env)
+{
        opcode_key_t key, *entry;
        ir_node      *irn = node->node;
 
@@ -376,7 +406,7 @@ static opcode_key_t *opcode(const node_t *node, environment_t *env) {
        }
        key.mode   = get_irn_mode(node->node);
        key.u.proj = 0;
-       key.u.ent  = NULL;
+       key.u.addr = NULL;
 
        switch (key.code) {
        case iro_Proj:
@@ -385,13 +415,28 @@ static opcode_key_t *opcode(const node_t *node, environment_t *env) {
        case iro_Sel:
                key.u.ent = get_Sel_entity(irn);
                break;
+       case iro_SymConst:
+               key.u.sym = get_SymConst_symbol(irn);
+               break;
+       case iro_Const:
+               key.u.tv  = get_Const_tarval(irn);
+               break;
+       case iro_Load:
+               key.mode = get_Load_mode(irn);
+               break;
+       case iro_Div:
+               key.u.intVal = get_Div_no_remainder(irn);
+               break;
+       case iro_Builtin:
+               key.u.intVal = get_Builtin_kind(irn);
+               break;
        default:
                break;
        }
 
-       entry = set_insert(env->opcode2id_map, &key, sizeof(key), opcode_hash(&key));
+       entry = set_insert(opcode_key_t, env->opcode2id_map, &key, sizeof(key), opcode_hash(&key));
        return entry;
-}  /* opcode */
+}
 
 /**
  * Split a partition by a local list.
@@ -402,7 +447,8 @@ static opcode_key_t *opcode(const node_t *node, environment_t *env) {
  *
  * @return  a new partition containing the nodes of g
  */
-static partition_t *split(partition_t *Z, block_t *g, environment_t *env) {
+static partition_t *split(partition_t *Z, block_t *g, environment_t *env)
+{
        partition_t *Z_prime;
        block_t     *block;
        unsigned    n = 0;
@@ -430,24 +476,39 @@ static partition_t *split(partition_t *Z, block_t *g, environment_t *env) {
        dump_partition("Now ", Z);
        dump_partition("Created new ", Z_prime);
        return Z_prime;
-}  /* split */
+}
+
+/**
+ * Return non-zero if pred should be tread as a input node.
+ */
+static int is_input_node(ir_node *pred, ir_node *irn, int index)
+{
+       /* for now, do NOT turn direct calls into indirect one */
+       if (index != 1)
+               return 1;
+       if (! is_SymConst_addr_ent(pred))
+               return 1;
+       if (! is_Call(irn))
+               return 1;
+       return 0;
+}
 
 /**
  * Propagate nodes on all wait queues of the given partition.
  *
  * @param part  the partition
- * @param env    the environment
+ * @param env   the environment
  */
-void propagate_blocks(partition_t *part, environment_t *env) {
+static void propagate_blocks(partition_t *part, environment_t *env)
+{
        block_t         *ready_blocks = NULL;
        unsigned        n_ready       = 0;
-       block_t         *bl, *next;
        listmap_t       map;
        listmap_entry_t *iter;
 
        DB((dbg, LEVEL_2, " Propagate blocks on part%u\n", part->nr));
 
-       /* Let map be an empty mapping from the range of Opcodes to (local) list of Nodes. */
+       /* Let map be an empty mapping from the range of Opcodes to (local) list of blocks. */
        listmap_init(&map);
        list_for_each_entry_safe(block_t, bl, next, &part->blocks, block_list) {
                opcode_key_t    *id;
@@ -476,19 +537,21 @@ void propagate_blocks(partition_t *part, environment_t *env) {
                        for (i = get_irn_arity(irn) - 1; i >= 0; --i) {
                                ir_node *pred  = get_irn_n(irn, i);
                                ir_node *block = get_nodes_block(skip_Proj(pred));
-                               node_t *p_node;
 
                                if (block != bl->block) {
-                                       p_node = create_node(pred, bl, env);
-                                       /* do not threat Constants like live-ins */
-                                       if (! is_irn_constlike(pred)) {
+                                       node_t *p_node = create_node(pred, bl, env);
+                                       if (is_input_node(pred, irn, i)) {
+                                               /* is a block live input */
                                                p_node->is_input = 1;
                                                if (! is_Phi(irn))
                                                        add_pair(bl, irn, i, env);
+                                       } else if (is_Phi(pred)) {
+                                               /* update the Phi list */
+                                               add_phi(bl, pred, env);
                                        }
                                } else if (! irn_visited_else_mark(pred)) {
                                        /* not yet visited, ok */
-                                       p_node = create_node(pred, bl, env);
+                                       create_node(pred, bl, env);
 
                                        if (is_Phi(pred)) {
                                                /* update the Phi list */
@@ -500,7 +563,7 @@ void propagate_blocks(partition_t *part, environment_t *env) {
                        DB((dbg, LEVEL_3, "  propagate Input %+F\n", node->node));
                }
 
-               /* Add bl to map[opcode(bl)]. */
+               /* Add bl to map[opcode(n)]. */
                id          = opcode(node, env);
                entry       = listmap_find(&map, id);
                bl->next    = entry->list;
@@ -539,16 +602,15 @@ void propagate_blocks(partition_t *part, environment_t *env) {
                split(part, S, env);
        }
        listmap_term(&map);
-}  /* propagate_blocks */
+}
 
 /**
  * Propagate nodes on all wait queues.
  *
  * @param env    the environment
  */
-void propagate(environment_t *env) {
-       partition_t *part, *next;
-
+static void propagate(environment_t *env)
+{
        list_for_each_entry_safe(partition_t, part, next, &env->partitions, part_list) {
                if (part->n_blocks < 2) {
                        /* zero or one block left, kill this partition */
@@ -557,7 +619,90 @@ void propagate(environment_t *env) {
                } else
                        propagate_blocks(part, env);
        }
-}  /* propagate */
+}
+
+/**
+ * Map a block to the phi[block->input] live-trough.
+ */
+static void *live_throughs(const block_t *bl, const ir_node *phi)
+{
+       ir_node *input = get_Phi_pred(phi, bl->meet_input);
+
+       /* If this input is inside our block, this
+          is a live-out and not a live trough.
+          Live-outs are tested inside propagate, so map all of
+          them to the "general" value NULL */
+       if (get_nodes_block(input) == bl->block)
+               return NULL;
+       return input;
+}
+
+/**
+ * Split partition by live-outs and live-troughs.
+ *
+ * @param part  the partition
+ * @param env   the environment
+ */
+static void propagate_blocks_live_troughs(partition_t *part, environment_t *env)
+{
+       const ir_node   *meet_block = part->meet_block;
+       listmap_t       map;
+       listmap_entry_t *iter;
+       const ir_node   *phi;
+
+       DB((dbg, LEVEL_2, " Propagate live-troughs on part%u\n", part->nr));
+
+       for (phi = get_Block_phis(meet_block); phi != NULL; phi = get_Phi_next(phi)) {
+               /* propagate on all Phis of the meet-block */
+
+               if (part->n_blocks < 2) {
+                       /* zero or one block left, kill this partition */
+                       list_del(&part->part_list);
+                       DB((dbg, LEVEL_2, " Partition %u contains less than 2 blocks, killed\n", part->nr));
+                       return;
+               }
+
+               /* Let map be an empty mapping from the range of live-troughs to (local) list of blocks. */
+               listmap_init(&map);
+               list_for_each_entry_safe(block_t, bl, next, &part->blocks, block_list) {
+                       opcode_key_t    *id;
+                       listmap_entry_t *entry;
+
+                       /* Add bl to map[live_trough(bl)]. */
+                       id          = (opcode_key_t*)live_throughs(bl, phi);
+                       entry       = listmap_find(&map, id);
+                       bl->next    = entry->list;
+                       entry->list = bl;
+               }
+
+               /* for all sets S except one in the range of map do */
+               for (iter = map.values; iter != NULL; iter = iter->next) {
+                       block_t *S;
+
+                       if (iter->next == NULL) {
+                               /* this is the last entry, ignore */
+                               break;
+                       }
+                       S = iter->list;
+
+                       /* Add SPLIT( X, S ) to P. */
+                       split(part, S, env);
+               }
+               listmap_term(&map);
+       }
+}
+
+/**
+ * Propagate live-troughs on all partitions on the partition list.
+ *
+ * @param env    the environment
+ */
+static void propagate_live_troughs(environment_t *env)
+{
+       list_for_each_entry_safe(partition_t, part, next, &env->partitions, part_list) {
+               propagate_blocks_live_troughs(part, env);
+       }
+}
 
 /**
  * Apply analysis results by replacing all blocks of a partition
@@ -570,14 +715,14 @@ void propagate(environment_t *env) {
  *
  * @param part  the partition to process
  */
-static void apply(ir_graph *irg, partition_t *part) {
+static void apply(ir_graph *irg, partition_t *part)
+{
        block_t *repr = list_entry(part->blocks.next, block_t, block_list);
-       block_t *bl;
        ir_node *block, *end, *meet_block, *p, *next;
        ir_node **ins, **phi_ins;
        phi_t   *repr_phi, *phi;
        pair_t  *repr_pair, *pair;
-       int     i, j, k, n, block_nr, n_phis;
+       int     i, j, k, n, n_phis;
 
        list_del(&repr->block_list);
 
@@ -611,10 +756,8 @@ static void apply(ir_graph *irg, partition_t *part) {
 
        /* collect new in arrays */
        end = get_irg_end(irg);
-       block_nr = 0;
        list_for_each_entry(block_t, bl, &part->blocks, block_list) {
                block = bl->block;
-               ++block_nr;
 
                DB((dbg, LEVEL_1, "%+F, ", block));
 
@@ -693,7 +836,7 @@ static void apply(ir_graph *irg, partition_t *part) {
                if (is_op_forking(cfop)) {
                        /* a critical edge */
                        ir_node *block = new_r_Block(irg, 1, &ins[i]);
-                       ir_node *jmp   = new_r_Jmp(irg, block);
+                       ir_node *jmp   = new_r_Jmp(block);
                        ins[i] = jmp;
                }
        }
@@ -712,7 +855,7 @@ static void apply(ir_graph *irg, partition_t *part) {
        for (repr_pair = repr->input_pairs; repr_pair != NULL; repr_pair = repr_pair->next) {
                ir_node *input = get_irn_n(repr_pair->irn, repr_pair->index);
                ir_mode *mode  = get_irn_mode(input);
-               ir_node *phi   = new_r_Phi(current_ir_graph, block, n, repr_pair->ins, mode);
+               ir_node *phi   = new_r_Phi(block, n, repr_pair->ins, mode);
 
                set_irn_n(repr_pair->irn, repr_pair->index, phi);
                DEL_ARR_F(repr_pair->ins);
@@ -774,7 +917,7 @@ continue_outer:
        /* fix inputs of the meet block */
        set_irn_in(meet_block, j, ins);
        DEL_ARR_F(ins);
-}  /* apply */
+}
 
 /**
  * Create a partition for a the end block.
@@ -782,7 +925,8 @@ continue_outer:
  * @param end_block  the end block
  * @param env        the environment
  */
-static void partition_for_end_block(ir_node *end_block, environment_t *env) {
+static void partition_for_end_block(ir_node *end_block, environment_t *env)
+{
        partition_t *part = create_partition(end_block, env);
        ir_node     *end;
        int         i;
@@ -797,14 +941,14 @@ static void partition_for_end_block(ir_node *end_block, environment_t *env) {
                mark_irn_visited(pred);
 
                block = get_nodes_block(pred);
-               bl    = create_block(block, part, env);
+               bl    = create_block(block, i, part, env);
                node  = create_node(pred, bl, env);
 
                bl->cf_root = node;
        }
 
        /* collect all no-return blocks */
-       end = get_irg_end(current_ir_graph);
+       end = get_irg_end(get_irn_irg(end_block));
        for (i = get_End_n_keepalives(end) - 1; i >= 0; --i) {
                ir_node *ka    = get_End_keepalive(end, i);
                ir_node *block;
@@ -817,14 +961,14 @@ static void partition_for_end_block(ir_node *end_block, environment_t *env) {
 
                /* found one */
                block = get_nodes_block(ka);
-               bl    = create_block(block, part, env);
+               bl    = create_block(block, -1, part, env);
                node  = create_node(ka, bl, env);
 
                bl->cf_root = node;
        }
 
        dump_partition("Created", part);
-}  /* partition_for_end_block */
+}
 
 #ifdef GENERAL_SHAPE
 /**
@@ -835,12 +979,13 @@ static void partition_for_end_block(ir_node *end_block, environment_t *env) {
  * @param n_preds  number of elements in preds
  * @param env      the environment
  */
-static void partition_for_block(ir_node *block, ir_node *preds[], int n_preds, environment_t *env) {
+static void partition_for_block(ir_node *block, pred_t preds[], int n_preds, environment_t *env)
+{
        partition_t *part = create_partition(block, env);
        int         i;
 
        for (i = n_preds - 1; i >= 0; --i) {
-               ir_node *pred = preds[i];
+               ir_node *pred = preds[i].pred;
                ir_node *block;
                block_t *bl;
                node_t  *node;
@@ -848,32 +993,34 @@ static void partition_for_block(ir_node *block, ir_node *preds[], int n_preds, e
                mark_irn_visited(pred);
 
                block = get_nodes_block(pred);
-               bl    = create_block(block, part, env);
+               bl    = create_block(block, preds[i].index, part, env);
                node  = create_node(pred, bl, env);
 
                bl->cf_root = node;
        }
 
        dump_partition("Created", part);
-}  /* partition_for_block */
+}
 
 /**
  * Walker: clear the links of all block phi lists and normal
  * links.
  */
-static void clear_phi_links(ir_node *irn, void *env) {
+static void clear_phi_links(ir_node *irn, void *env)
+{
        (void) env;
        if (is_Block(irn)) {
                set_Block_phis(irn, NULL);
                set_irn_link(irn, NULL);
        }
-}  /* clear_phi_links */
+}
 
 /**
- * Walker, detect live-out only nodes.
+ * Walker, detect live-out nodes.
  */
-static void find_liveouts(ir_node *irn, void *ctx) {
-       environment_t *env        = ctx;
+static void find_liveouts(ir_node *irn, void *ctx)
+{
+       environment_t *env        = (environment_t*)ctx;
        ir_node       **live_outs = env->live_outs;
        ir_node       *this_block;
        int           i;
@@ -897,31 +1044,30 @@ static void find_liveouts(ir_node *irn, void *ctx) {
                ir_node *pred = get_irn_n(irn, i);
                int     idx   = get_irn_idx(pred);
 
-               if (live_outs[idx] == pred) {
-                       /* referenced by other nodes inside this block */
+               if (live_outs[idx] != NULL) {
+                       /* already marked as live-out */
                        return;
                }
 
                pred_block = get_nodes_block(pred);
-               if (this_block != pred_block) {
+               /* Phi nodes always refer to live-outs */
+               if (is_Phi(irn) || this_block != pred_block) {
                        /* pred is a live-out */
                        live_outs[idx] = pred_block;
-               } else {
-                       /* this node is referenced from inside this block */
-                       live_outs[idx] = pred;
                }
        }
 }
 
 /**
- * Check if the current block is the meet block of its predecessors.
+ * Check if the current block is the meet block of its predecessors.
  */
-static void check_for_cf_meet(ir_node *block, void *ctx) {
-       environment_t *env = ctx;
+static void check_for_cf_meet(ir_node *block, void *ctx)
+{
+       environment_t *env = (environment_t*)ctx;
        int           i, k, n;
-       ir_node       **preds;
+       pred_t        *preds;
 
-       if (block == get_irg_end_block(current_ir_graph)) {
+       if (block == get_irg_end_block(get_irn_irg(block))) {
                /* always create a partition for the end block */
                partition_for_end_block(block, env);
                return;
@@ -932,32 +1078,36 @@ static void check_for_cf_meet(ir_node *block, void *ctx) {
                /* Must have at least two predecessors */
                return;
        }
-       NEW_ARR_A(ir_node *, preds, n);
 
+       NEW_ARR_A(pred_t, preds, n);
        k = 0;
        for (i = n - 1; i >= 0; --i) {
                ir_node *pred = get_Block_cfgpred(block, i);
 
                /* pred must be a direct jump to us */
-               if (! is_Jmp(pred) && ! is_Raise(pred))
+               if (! is_Jmp(pred) && ! is_Raise(pred) && !is_Bad(pred))
                        continue;
-               preds[k++] = pred;
+
+               preds[k].pred  = pred;
+               preds[k].index = i;
+               ++k;
        }
 
        if (k > 1)
                partition_for_block(block, preds, k, env);
-}  /* check_for_cf_meet */
+}
 
 /**
  * Compare two nodes for root ordering.
  */
-static int cmp_nodes(const void *a, const void *b) {
-       ir_node *const *pa     = a;
-       ir_node *const *pb     = b;
+static int cmp_nodes(const void *a, const void *b)
+{
+       const ir_node *const *pa = (const ir_node*const*)a;
+       const ir_node *const *pb = (const ir_node*const*)b;
        const ir_node  *irn_a  = *pa;
        const ir_node  *irn_b  = *pb;
-       ir_opcode      code_a  = get_irn_opcode(irn_a);
-       ir_opcode      code_b  = get_irn_opcode(irn_b);
+       unsigned       code_a  = get_irn_opcode(irn_a);
+       unsigned       code_b  = get_irn_opcode(irn_b);
        ir_mode        *mode_a, *mode_b;
        unsigned       idx_a, idx_b;
 
@@ -977,12 +1127,13 @@ static int cmp_nodes(const void *a, const void *b) {
        idx_b = get_irn_idx(irn_b);
 
        return (idx_a > idx_b) - (idx_a < idx_b);
-}  /* cmp_nodes */
+}
 
 /**
  * Add the roots to all blocks.
  */
-static void add_roots(ir_graph *irg, environment_t *env) {
+static void add_roots(ir_graph *irg, environment_t *env)
+{
        unsigned idx, n      = get_irg_last_idx(irg);
        ir_node  **live_outs = env->live_outs;
        block_t  *bl;
@@ -1004,10 +1155,10 @@ static void add_roots(ir_graph *irg, environment_t *env) {
        }
        /*
         * Now sort the roots to normalize them as good as possible.
-     * Else, we will split identical blocks if we start which different roots
+     * Else, we will split identical blocks if we start which different roots.
         */
        for (bl = env->all_blocks; bl != NULL; bl = bl->all_next) {
-               int i, n = ARR_LEN(bl->roots);
+               size_t i, n = ARR_LEN(bl->roots);
 
 #if 1
                /* TODO: is this really needed? The roots are already in
@@ -1025,23 +1176,20 @@ static void add_roots(ir_graph *irg, environment_t *env) {
                DEL_ARR_F(bl->roots);
                bl->roots = NULL;
        }
-}  /* add_roots */
+}
 #endif /* GENERAL_SHAPE */
 
 /* Combines congruent end blocks into one. */
-int shape_blocks(ir_graph *irg) {
-       ir_graph      *rem;
+void shape_blocks(ir_graph *irg)
+{
        environment_t env;
-       partition_t   *part;
+       block_t       *bl;
        int           res, n;
 
-       rem = current_ir_graph;
-       current_ir_graph = irg;
-
        /* register a debug mask */
        FIRM_DBG_REGISTER(dbg, "firm.opt.blocks");
 
-       DEBUG_ONLY(part_nr = 0);
+       DEBUG_ONLY(part_nr = 0;)
        DB((dbg, LEVEL_1, "Shaping blocks for %+F\n", irg));
 
        /* works better, when returns are placed at the end of the blocks */
@@ -1053,8 +1201,7 @@ int shape_blocks(ir_graph *irg) {
        env.opcode2id_map  = new_set(cmp_opcode, iro_Last * 4);
 
        n             = get_irg_last_idx(irg);
-       env.live_outs = NEW_ARR_F(ir_node *, n);
-       memset(env.live_outs, 0, sizeof(*env.live_outs) * n);
+       env.live_outs = NEW_ARR_FZ(ir_node*, n);
 
        env.all_blocks = NULL;
 
@@ -1083,10 +1230,13 @@ int shape_blocks(ir_graph *irg) {
        partition_for_end_block(get_irg_end_block(irg), &env);
 #endif
 
+       propagate_live_troughs(&env);
        while (! list_empty(&env.partitions))
                propagate(&env);
 
        res = !list_empty(&env.ready);
+       //if (res) dump_ir_block_graph(irg, "-before");
+
 
        list_for_each_entry(partition_t, part, &env.ready, part_list) {
                dump_partition("Ready Partition", part);
@@ -1096,20 +1246,19 @@ int shape_blocks(ir_graph *irg) {
 
        if (res) {
                /* control flow changed */
-               set_irg_outs_inconsistent(irg);
-               set_irg_extblk_inconsistent(irg);
-               set_irg_doms_inconsistent(irg);
-               /* Hmm, only the root loop is inconsistent */
-               set_irg_loopinfo_inconsistent(irg);
-
-               /* Calls might be removed. */
-               set_trouts_inconsistent();
+               clear_irg_properties(irg, IR_GRAPH_PROPERTY_CONSISTENT_DOMINANCE);
+       }
+
+       for (bl = env.all_blocks; bl != NULL; bl = bl->all_next) {
+               DEL_ARR_F(bl->roots);
        }
 
        DEL_ARR_F(env.live_outs);
        del_set(env.opcode2id_map);
        obstack_free(&env.obst, NULL);
-       current_ir_graph = rem;
+}
 
-       return res;
-}  /* shape_blocks */
+ir_graph_pass_t *shape_blocks_pass(const char *name)
+{
+       return def_graph_pass(name ? name : "shape_blocks", shape_blocks);
+}