becopyheur2: Remove unnecessary indirection.
[libfirm] / ir / ir / irnode.c
index aaf0036..2f92773 100644 (file)
@@ -1,20 +1,6 @@
 /*
- * 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.
- *
- * This file is provided AS IS with NO WARRANTY OF ANY KIND, INCLUDING THE
- * WARRANTY OF DESIGN, MERCHANTABILITY AND FITNESS FOR A PARTICULAR
- * PURPOSE.
+ * Copyright (C) 2012 University of Karlsruhe.
  */
 
 /**
 
 /* some constants fixing the positions of nodes predecessors
    in the in array */
-#define CALL_PARAM_OFFSET     (n_Call_max+1)
-#define BUILTIN_PARAM_OFFSET  (n_Builtin_max+1)
-#define SEL_INDEX_OFFSET      (n_Sel_max+1)
-#define RETURN_RESULT_OFFSET  (n_Return_max+1)
 #define END_KEEPALIVE_OFFSET  0
 
 static const char *relation_names [] = {
@@ -89,72 +71,22 @@ ir_relation get_inversed_relation(ir_relation relation)
        ir_relation code    = relation & ~(ir_relation_less|ir_relation_greater);
        bool        less    = relation & ir_relation_less;
        bool        greater = relation & ir_relation_greater;
-       code |= (less ? ir_relation_greater : 0) | (greater ? ir_relation_less : 0);
+       code |= (less ? ir_relation_greater : ir_relation_false)
+             | (greater ? ir_relation_less : ir_relation_false);
        return code;
 }
 
-/**
- * Indicates, whether additional data can be registered to ir nodes.
- * If set to 1, this is not possible anymore.
- */
-static int forbid_new_data = 0;
-
-/**
- * The amount of additional space for custom data to be allocated upon
- * creating a new node.
- */
-unsigned firm_add_node_size = 0;
-
-
-/* register new space for every node */
-unsigned firm_register_additional_node_data(unsigned size)
-{
-       assert(!forbid_new_data && "Too late to register additional node data");
-
-       if (forbid_new_data)
-               return 0;
-
-       return firm_add_node_size += size;
-}
-
-
-void init_irnode(void)
-{
-       /* Forbid the addition of new data to an ir node. */
-       forbid_new_data = 1;
-}
-
-struct struct_align {
-       char c;
-       struct s {
-               int i;
-               float f;
-               double d;
-       } s;
-};
-
-/*
- * irnode constructor.
- * Create a new irnode in irg, with an op, mode, arity and
- * some incoming irnodes.
- * If arity is negative, a node with a dynamic array is created.
- */
 ir_node *new_ir_node(dbg_info *db, ir_graph *irg, ir_node *block, ir_op *op,
                      ir_mode *mode, int arity, ir_node *const *in)
 {
-       ir_node *res;
-       unsigned align = offsetof(struct struct_align, s) - 1;
-       unsigned add_node_size = (firm_add_node_size + align) & ~align;
-       size_t node_size = offsetof(ir_node, attr) + op->attr_size + add_node_size;
-       char *p;
        int i;
 
        assert(irg);
        assert(op);
        assert(mode);
-       p = (char*)obstack_alloc(irg->obst, node_size);
-       memset(p, 0, node_size);
-       res = (ir_node *)(p + add_node_size);
+
+       size_t   const node_size = offsetof(ir_node, attr) + op->attr_size;
+       ir_node *const res       = (ir_node*)OALLOCNZ(get_irg_obstack(irg), char, node_size);
 
        res->kind     = k_ir_node;
        res->op       = op;
@@ -171,13 +103,12 @@ ir_node *new_ir_node(dbg_info *db, ir_graph *irg, ir_node *block, ir_op *op,
                if (op == op_End || op == op_Sync)
                        res->in = NEW_ARR_F(ir_node *, (arity+1));
                else
-                       res->in = NEW_ARR_D(ir_node *, irg->obst, (arity+1));
+                       res->in = NEW_ARR_D(ir_node*, get_irg_obstack(irg), arity + 1);
                memcpy(&res->in[1], in, sizeof(ir_node *) * arity);
        }
 
        res->in[0]   = block;
        set_irn_dbg_info(res, db);
-       res->out     = NULL;
        res->node_nr = get_irp_new_node_nr();
 
        for (i = 0; i < EDGE_KIND_LAST; ++i) {
@@ -188,20 +119,19 @@ ir_node *new_ir_node(dbg_info *db, ir_graph *irg, ir_node *block, ir_op *op,
        }
 
        /* don't put this into the for loop, arity is -1 for some nodes! */
-       edges_notify_edge(res, -1, res->in[0], NULL, irg);
+       if (block != NULL)
+               edges_notify_edge(res, -1, block, NULL, irg);
        for (i = 1; i <= arity; ++i)
                edges_notify_edge(res, i - 1, res->in[i], NULL, irg);
 
        hook_new_node(irg, res);
-       if (get_irg_phase_state(irg) == phase_backend) {
-               be_info_new_node(res);
+       if (irg_is_constrained(irg, IR_GRAPH_CONSTRAINT_BACKEND)) {
+               be_info_new_node(irg, res);
        }
 
        return res;
 }
 
-/*-- getting some parameters from ir_nodes --*/
-
 int (is_ir_node)(const void *thing)
 {
        return is_ir_node_(thing);
@@ -212,18 +142,12 @@ int (get_irn_arity)(const ir_node *node)
        return get_irn_arity_(node);
 }
 
-/* Returns the array with ins. This array is shifted with respect to the
-   array accessed by get_irn_n: The block operand is at position 0 not -1.
-   (@@@ This should be changed.)
-   The order of the predecessors in this array is not guaranteed, except that
-   lists of operands as predecessors of Block or arguments of a Call are
-   consecutive. */
 ir_node **get_irn_in(const ir_node *node)
 {
        return node->in;
 }
 
-void set_irn_in(ir_node *node, int arity, ir_node **in)
+void set_irn_in(ir_node *const node, int const arity, ir_node *const *const in)
 {
        int i;
        ir_node *** pOld_in;
@@ -231,6 +155,13 @@ void set_irn_in(ir_node *node, int arity, ir_node **in)
 
        pOld_in = &node->in;
 
+#ifndef NDEBUG
+       assert(node != NULL && node->kind == k_ir_node);
+       assert(arity >= 0);
+       for (i = 0; i < arity; ++i) {
+               assert(in[i] != NULL && in[0]->kind == k_ir_node);
+       }
+#endif
 
        for (i = 0; i < arity; i++) {
                if (i < (int)ARR_LEN(*pOld_in)-1)
@@ -244,15 +175,15 @@ void set_irn_in(ir_node *node, int arity, ir_node **in)
 
        if (arity != (int)ARR_LEN(*pOld_in) - 1) {
                ir_node * block = (*pOld_in)[0];
-               *pOld_in = NEW_ARR_D(ir_node *, irg->obst, arity + 1);
+               *pOld_in = NEW_ARR_D(ir_node*, get_irg_obstack(irg), arity + 1);
                (*pOld_in)[0] = block;
        }
-       fix_backedges(irg->obst, node);
+       fix_backedges(get_irg_obstack(irg), node);
 
        memcpy((*pOld_in) + 1, in, sizeof(ir_node *) * arity);
 
        /* update irg flags */
-       clear_irg_state(irg, IR_GRAPH_STATE_CONSISTENT_OUTS | IR_GRAPH_STATE_CONSISTENT_LOOPINFO);
+       clear_irg_properties(irg, IR_GRAPH_PROPERTY_CONSISTENT_OUTS | IR_GRAPH_PROPERTY_CONSISTENT_LOOPINFO);
 }
 
 ir_node *(get_irn_n)(const ir_node *node, int n)
@@ -277,7 +208,7 @@ void set_irn_n(ir_node *node, int n, ir_node *in)
        node->in[n + 1] = in;
 
        /* update irg flags */
-       clear_irg_state(irg, IR_GRAPH_STATE_CONSISTENT_OUTS | IR_GRAPH_STATE_CONSISTENT_LOOPINFO);
+       clear_irg_properties(irg, IR_GRAPH_PROPERTY_CONSISTENT_OUTS | IR_GRAPH_PROPERTY_CONSISTENT_LOOPINFO);
 }
 
 int add_irn_n(ir_node *node, ir_node *in)
@@ -293,16 +224,37 @@ int add_irn_n(ir_node *node, ir_node *in)
        /* Call the hook */
        hook_set_irn_n(node, pos, node->in[pos + 1], NULL);
 
+       /* update irg flags */
+       clear_irg_properties(irg, IR_GRAPH_PROPERTY_CONSISTENT_OUTS);
+
        return pos;
 }
 
+static void del_irn_n(ir_node *node, int n)
+{
+       ir_graph *irg = get_irn_irg(node);
+
+       /* remove the edge */
+       ir_node *pred = node->in[n+1];
+       edges_notify_edge(node, n, NULL, pred, irg);
+
+       int arity = get_irn_arity(node);
+       if (n != arity-1) {
+               /* exchange with the last one */
+               ir_node *old = node->in[arity];
+               edges_notify_edge(node, arity-1, NULL, old, irg);
+               node->in[n+1] = old;
+               edges_notify_edge(node, n, old, NULL, irg);
+       }
+       ARR_SHRINKLEN(node->in, arity);
+
+       /* update irg flags */
+       clear_irg_properties(irg, IR_GRAPH_PROPERTY_CONSISTENT_OUTS);
+}
+
 void del_Sync_n(ir_node *n, int i)
 {
-       int      arity     = get_Sync_n_preds(n);
-       ir_node *last_pred = get_Sync_pred(n, arity - 1);
-       set_Sync_pred(n, i, last_pred);
-       edges_notify_edge(n, arity - 1, NULL, last_pred, get_irn_irg(n));
-       ARR_SHRINKLEN(get_irn_in(n), arity);
+       del_irn_n(n, i);
 }
 
 int (get_irn_deps)(const ir_node *node)
@@ -315,44 +267,50 @@ ir_node *(get_irn_dep)(const ir_node *node, int pos)
        return get_irn_dep_(node, pos);
 }
 
-void (set_irn_dep)(ir_node *node, int pos, ir_node *dep)
+void set_irn_dep(ir_node *node, int pos, ir_node *dep)
 {
-       set_irn_dep_(node, pos, dep);
+       ir_node *old;
+       ir_graph *irg;
+
+       assert(node->deps && "dependency array node yet allocated. use add_irn_dep()");
+       assert(pos >= 0 && pos < (int)ARR_LEN(node->deps) && "dependency index out of range");
+       assert(dep != NULL);
+       old = node->deps[pos];
+       node->deps[pos] = dep;
+       irg = get_irn_irg(node);
+       if (edges_activated_kind(irg, EDGE_KIND_DEP))
+               edges_notify_edge_kind(node, pos, dep, old, EDGE_KIND_DEP, irg);
 }
 
-int add_irn_dep(ir_node *node, ir_node *dep)
+void add_irn_dep(ir_node *node, ir_node *dep)
 {
-       int res = 0;
-
-       /* DEP edges are only allowed in backend phase */
-       assert(get_irg_phase_state(get_irn_irg(node)) == phase_backend);
+       ir_graph *irg;
+       assert(dep != NULL);
        if (node->deps == NULL) {
-               node->deps = NEW_ARR_F(ir_node *, 1);
-               node->deps[0] = dep;
-       } else {
-               int i, n;
-               int first_zero = -1;
-
-               for (i = 0, n = ARR_LEN(node->deps); i < n; ++i) {
-                       if (node->deps[i] == NULL)
-                               first_zero = i;
+               node->deps = NEW_ARR_F(ir_node *, 0);
+       }
+       ARR_APP1(ir_node*, node->deps, dep);
+       irg = get_irn_irg(node);
+       if (edges_activated_kind(irg, EDGE_KIND_DEP))
+               edges_notify_edge_kind(node, ARR_LEN(node->deps)-1, dep, NULL, EDGE_KIND_DEP, irg);
+}
 
-                       if (node->deps[i] == dep)
-                               return i;
-               }
+void delete_irn_dep(ir_node *node, ir_node *dep)
+{
+       size_t i;
+       size_t n_deps;
+       if (node->deps == NULL)
+               return;
 
-               if (first_zero >= 0) {
-                       node->deps[first_zero] = dep;
-                       res = first_zero;
-               } else {
-                       ARR_APP1(ir_node *, node->deps, dep);
-                       res = n;
+       n_deps = ARR_LEN(node->deps);
+       for (i = 0; i < n_deps; ++i) {
+               if (node->deps[i] == dep) {
+                       set_irn_dep(node, i, node->deps[n_deps-1]);
+                       edges_notify_edge(node, i, NULL, dep, get_irn_irg(node));
+                       ARR_SHRINKLEN(node->deps, n_deps-1);
+                       break;
                }
        }
-
-       edges_notify_edge_kind(node, res, dep, NULL, EDGE_KIND_DEP, get_irn_irg(node));
-
-       return res;
 }
 
 void add_irn_deps(ir_node *tgt, ir_node *src)
@@ -379,7 +337,6 @@ ir_op *(get_irn_op)(const ir_node *node)
        return get_irn_op_(node);
 }
 
-/* should be private to the library: */
 void (set_irn_op)(ir_node *node, ir_op *op)
 {
        set_irn_op_(node, op);
@@ -392,8 +349,6 @@ unsigned (get_irn_opcode)(const ir_node *node)
 
 const char *get_irn_opname(const ir_node *node)
 {
-       assert(node);
-       if (is_Phi0(node)) return "Phi0";
        return get_id_str(node->op->name);
 }
 
@@ -460,7 +415,6 @@ void set_irn_pinned(ir_node *node, op_pin_state state)
        node->attr.except.pin_state = state;
 }
 
-/* Outputs a unique number for this node */
 long get_irn_node_nr(const ir_node *node)
 {
        assert(node);
@@ -485,18 +439,6 @@ unsigned (get_irn_idx)(const ir_node *node)
        return get_irn_idx_(node);
 }
 
-int get_irn_pred_pos(ir_node *node, ir_node *arg)
-{
-       int i;
-       for (i = get_irn_arity(node) - 1; i >= 0; i--) {
-               if (get_irn_n(node, i) == arg)
-                       return i;
-       }
-       return -1;
-}
-
-/** manipulate fields of individual nodes **/
-
 ir_node *(get_nodes_block)(const ir_node *node)
 {
        return get_nodes_block_(node);
@@ -504,12 +446,10 @@ ir_node *(get_nodes_block)(const ir_node *node)
 
 void set_nodes_block(ir_node *node, ir_node *block)
 {
-       assert(node->op != op_Block);
+       assert(!is_Block(node));
        set_irn_n(node, -1, block);
 }
 
-/* Test whether arbitrary node is frame pointer, i.e. Proj(pn_Start_P_frame_base)
- * from Start.  If so returns frame type, else Null. */
 ir_type *is_frame_pointer(const ir_node *n)
 {
        if (is_Proj(n) && (get_Proj_proj(n) == pn_Start_P_frame_base)) {
@@ -521,28 +461,6 @@ ir_type *is_frame_pointer(const ir_node *n)
        return NULL;
 }
 
-ir_node **get_Block_cfgpred_arr(ir_node *node)
-{
-       assert(is_Block(node));
-       return (ir_node **)&(get_irn_in(node)[1]);
-}
-
-int (get_Block_n_cfgpreds)(const ir_node *node)
-{
-       return get_Block_n_cfgpreds_(node);
-}
-
-ir_node *(get_Block_cfgpred)(const ir_node *node, int pos)
-{
-       return get_Block_cfgpred_(node, pos);
-}
-
-void set_Block_cfgpred(ir_node *node, int pos, ir_node *pred)
-{
-       assert(is_Block(node));
-       set_irn_n(node, pos, pred);
-}
-
 int get_Block_cfgpred_pos(const ir_node *block, const ir_node *pred)
 {
        int i;
@@ -591,23 +509,6 @@ int (Block_block_visited)(const ir_node *node)
        return Block_block_visited_(node);
 }
 
-ir_extblk *get_Block_extbb(const ir_node *block)
-{
-       ir_extblk *res;
-       assert(is_Block(block));
-       res = block->attr.block.extblk;
-       assert(res == NULL || is_ir_extbb(res));
-       return res;
-}
-
-void set_Block_extbb(ir_node *block, ir_extblk *extblk)
-{
-       assert(is_Block(block));
-       assert(extblk == NULL || is_ir_extbb(extblk));
-       block->attr.block.extblk = extblk;
-}
-
-/* returns the graph of a Block. */
 ir_graph *(get_Block_irg)(const ir_node *block)
 {
        return get_Block_irg_(block);
@@ -620,15 +521,10 @@ ir_entity *create_Block_entity(ir_node *block)
 
        entity = block->attr.block.entity;
        if (entity == NULL) {
-               ir_label_t  nr;
-               ir_type   *glob;
-
-               glob = get_glob_type();
-               entity = new_entity(glob, id_unique("block_%u"), get_code_type());
+               ir_label_t nr = get_irp_next_label_nr();
+               entity = new_label_entity(nr);
                set_entity_visibility(entity, ir_visibility_local);
                set_entity_linkage(entity, IR_LINKAGE_CONSTANT);
-               nr = get_irp_next_label_nr();
-               set_entity_label(entity, nr);
                set_entity_compiler_generated(entity, 1);
 
                block->attr.block.entity = entity;
@@ -636,24 +532,6 @@ ir_entity *create_Block_entity(ir_node *block)
        return entity;
 }
 
-ir_entity *get_Block_entity(const ir_node *block)
-{
-       assert(is_Block(block));
-       return block->attr.block.entity;
-}
-
-void set_Block_entity(ir_node *block, ir_entity *entity)
-{
-       assert(is_Block(block));
-       assert(get_entity_type(entity) == get_code_type());
-       block->attr.block.entity = entity;
-}
-
-int has_Block_entity(const ir_node *block)
-{
-       return block->attr.block.entity != NULL;
-}
-
 ir_node *(get_Block_phis)(const ir_node *block)
 {
        return get_Block_phis_(block);
@@ -669,43 +547,22 @@ void (add_Block_phi)(ir_node *block, ir_node *phi)
        add_Block_phi_(block, phi);
 }
 
-/* Get the Block mark (single bit). */
 unsigned (get_Block_mark)(const ir_node *block)
 {
        return get_Block_mark_(block);
 }
 
-/* Set the Block mark (single bit). */
 void (set_Block_mark)(ir_node *block, unsigned mark)
 {
        set_Block_mark_(block, mark);
 }
 
-int get_End_n_keepalives(const ir_node *end)
-{
-       assert(is_End(end));
-       return (get_irn_arity(end) - END_KEEPALIVE_OFFSET);
-}
-
-ir_node *get_End_keepalive(const ir_node *end, int pos)
-{
-       assert(is_End(end));
-       return get_irn_n(end, pos + END_KEEPALIVE_OFFSET);
-}
-
 void add_End_keepalive(ir_node *end, ir_node *ka)
 {
        assert(is_End(end));
        add_irn_n(end, ka);
 }
 
-void set_End_keepalive(ir_node *end, int pos, ir_node *ka)
-{
-       assert(is_End(end));
-       set_irn_n(end, pos + END_KEEPALIVE_OFFSET, ka);
-}
-
-/* Set new keep-alives */
 void set_End_keepalives(ir_node *end, int n, ir_node *in[])
 {
        size_t e;
@@ -724,48 +581,29 @@ void set_End_keepalives(ir_node *end, int n, ir_node *in[])
        }
 
        /* update irg flags */
-       clear_irg_state(irg, IR_GRAPH_STATE_CONSISTENT_OUTS);
+       clear_irg_properties(irg, IR_GRAPH_PROPERTY_CONSISTENT_OUTS);
 }
 
-/* Set new keep-alives from old keep-alives, skipping irn */
 void remove_End_keepalive(ir_node *end, ir_node *irn)
 {
-       int      n = get_End_n_keepalives(end);
-       int      i, idx;
-       ir_graph *irg;
+       int n = get_End_n_keepalives(end);
+       int idx = -1;
+       for (int i = n;;) {
+               if (i-- == 0)
+                       return;
 
-       idx = -1;
-       for (i = n -1; i >= 0; --i) {
                ir_node *old_ka = end->in[1 + END_KEEPALIVE_OFFSET + i];
 
                /* find irn */
                if (old_ka == irn) {
-                       idx = i;
-                       goto found;
+                       idx = END_KEEPALIVE_OFFSET + i;
+                       break;
                }
        }
-       return;
-found:
-       irg = get_irn_irg(end);
-
-       /* remove the edge */
-       edges_notify_edge(end, idx, NULL, irn, irg);
-
-       if (idx != n - 1) {
-               /* exchange with the last one */
-               ir_node *old = end->in[1 + END_KEEPALIVE_OFFSET + n - 1];
-               edges_notify_edge(end, n - 1, NULL, old, irg);
-               end->in[1 + END_KEEPALIVE_OFFSET + idx] = old;
-               edges_notify_edge(end, idx, old, NULL, irg);
-       }
-       /* now n - 1 keeps, 1 block input */
-       ARR_RESIZE(ir_node *, end->in, (n - 1) + 1 + END_KEEPALIVE_OFFSET);
-
-       /* update irg flags */
-       clear_irg_state(irg, IR_GRAPH_STATE_CONSISTENT_OUTS);
+       assert(idx != -1);
+       del_irn_n(end, idx);
 }
 
-/* remove Bads, NoMems and doublets from the keep-alive set */
 void remove_End_Bads_and_doublets(ir_node *end)
 {
        pset_new_t keeps;
@@ -784,28 +622,16 @@ void remove_End_Bads_and_doublets(ir_node *end)
 
                if (is_Bad(ka) || is_NoMem(ka) || pset_new_contains(&keeps, ka)) {
                        changed = true;
-                       /* remove the edge */
-                       edges_notify_edge(end, idx, NULL, ka, irg);
-
-                       if (idx != n - 1) {
-                               /* exchange with the last one */
-                               ir_node *old = end->in[1 + END_KEEPALIVE_OFFSET + n - 1];
-                               edges_notify_edge(end, n - 1, NULL, old, irg);
-                               end->in[1 + END_KEEPALIVE_OFFSET + idx] = old;
-                               edges_notify_edge(end, idx, old, NULL, irg);
-                       }
+                       del_irn_n(end, idx - END_KEEPALIVE_OFFSET);
                        --n;
                } else {
                        pset_new_insert(&keeps, ka);
                }
        }
-       /* n keeps, 1 block input */
-       ARR_RESIZE(ir_node *, end->in, n + 1 + END_KEEPALIVE_OFFSET);
-
        pset_new_destroy(&keeps);
 
        if (changed) {
-               clear_irg_state(irg, IR_GRAPH_STATE_CONSISTENT_OUTS);
+               clear_irg_properties(irg, IR_GRAPH_PROPERTY_CONSISTENT_OUTS);
        }
 }
 
@@ -818,35 +644,6 @@ void free_End(ir_node *end)
                             in array afterwards ... */
 }
 
-size_t get_Return_n_ress(const ir_node *node)
-{
-       assert(is_Return(node));
-       return (size_t)(get_irn_arity(node) - RETURN_RESULT_OFFSET);
-}
-
-ir_node **get_Return_res_arr(ir_node *node)
-{
-       assert(is_Return(node));
-       if (get_Return_n_ress(node) > 0)
-               return (ir_node **)&(get_irn_in(node)[1 + RETURN_RESULT_OFFSET]);
-       else
-               return NULL;
-}
-
-ir_node *get_Return_res(const ir_node *node, int pos)
-{
-       assert(is_Return(node));
-       assert(pos >= 0);
-       assert(get_Return_n_ress(node) > (size_t)pos);
-       return get_irn_n(node, pos + RETURN_RESULT_OFFSET);
-}
-
-void set_Return_res(ir_node *node, int pos, ir_node *res)
-{
-       assert(is_Return(node));
-       set_irn_n(node, pos + RETURN_RESULT_OFFSET, res);
-}
-
 int (is_Const_null)(const ir_node *node)
 {
        return is_Const_null_(node);
@@ -893,8 +690,6 @@ void set_SymConst_type(ir_node *node, ir_type *tp)
        node->attr.symc.sym.type_p = tp;
 }
 
-
-/* Only to access SymConst of kind symconst_addr_ent.  Else assertion: */
 ir_entity *get_SymConst_entity(const ir_node *node)
 {
        assert(is_SymConst(node) && SYMCONST_HAS_ENT(get_SymConst_kind(node)));
@@ -932,82 +727,6 @@ void set_SymConst_symbol(ir_node *node, union symconst_symbol sym)
        node->attr.symc.sym = sym;
 }
 
-int get_Sel_n_indexs(const ir_node *node)
-{
-       assert(is_Sel(node));
-       return (get_irn_arity(node) - SEL_INDEX_OFFSET);
-}
-
-ir_node **get_Sel_index_arr(ir_node *node)
-{
-       assert(is_Sel(node));
-       if (get_Sel_n_indexs(node) > 0)
-               return (ir_node **)& get_irn_in(node)[SEL_INDEX_OFFSET + 1];
-       else
-               return NULL;
-}
-
-ir_node *get_Sel_index(const ir_node *node, int pos)
-{
-       assert(is_Sel(node));
-       return get_irn_n(node, pos + SEL_INDEX_OFFSET);
-}
-
-void set_Sel_index(ir_node *node, int pos, ir_node *index)
-{
-       assert(is_Sel(node));
-       set_irn_n(node, pos + SEL_INDEX_OFFSET, index);
-}
-
-ir_node **get_Call_param_arr(ir_node *node)
-{
-       assert(is_Call(node));
-       return &get_irn_in(node)[CALL_PARAM_OFFSET + 1];
-}
-
-size_t get_Call_n_params(const ir_node *node)
-{
-       assert(is_Call(node));
-       return (size_t) (get_irn_arity(node) - CALL_PARAM_OFFSET);
-}
-
-ir_node *get_Call_param(const ir_node *node, int pos)
-{
-       assert(is_Call(node));
-       return get_irn_n(node, pos + CALL_PARAM_OFFSET);
-}
-
-void set_Call_param(ir_node *node, int pos, ir_node *param)
-{
-       assert(is_Call(node));
-       set_irn_n(node, pos + CALL_PARAM_OFFSET, param);
-}
-
-ir_node **get_Builtin_param_arr(ir_node *node)
-{
-       assert(is_Builtin(node));
-       return &get_irn_in(node)[BUILTIN_PARAM_OFFSET + 1];
-}
-
-int get_Builtin_n_params(const ir_node *node)
-{
-       assert(is_Builtin(node));
-       return (get_irn_arity(node) - BUILTIN_PARAM_OFFSET);
-}
-
-ir_node *get_Builtin_param(const ir_node *node, int pos)
-{
-       assert(is_Builtin(node));
-       return get_irn_n(node, pos + BUILTIN_PARAM_OFFSET);
-}
-
-void set_Builtin_param(ir_node *node, int pos, ir_node *param)
-{
-       assert(is_Builtin(node));
-       set_irn_n(node, pos + BUILTIN_PARAM_OFFSET, param);
-}
-
-/* Returns a human readable string for the ir_builtin_kind. */
 const char *get_builtin_kind_name(ir_builtin_kind kind)
 {
 #define X(a)    case a: return #a
@@ -1053,11 +772,10 @@ ir_entity *get_Call_callee(const ir_node *node, size_t pos)
 
 void set_Call_callee_arr(ir_node *node, size_t n, ir_entity ** arr)
 {
-       ir_graph *irg = get_irn_irg(node);
-
        assert(is_Call(node));
        if (node->attr.call.callee_arr == NULL || get_Call_n_callees(node) != n) {
-               node->attr.call.callee_arr = NEW_ARR_D(ir_entity *, irg->obst, n);
+               ir_graph *const irg = get_irn_irg(node);
+               node->attr.call.callee_arr = NEW_ARR_D(ir_entity*, get_irg_obstack(irg), n);
        }
        memcpy(node->attr.call.callee_arr, arr, n * sizeof(ir_entity *));
 }
@@ -1068,52 +786,6 @@ void remove_Call_callee_arr(ir_node *node)
        node->attr.call.callee_arr = NULL;
 }
 
-/* Checks for upcast.
- *
- * Returns true if the Cast node casts a class type to a super type.
- */
-int is_Cast_upcast(ir_node *node)
-{
-       ir_type *totype   = get_Cast_type(node);
-       ir_type *fromtype = get_irn_typeinfo_type(get_Cast_op(node));
-
-       assert(get_irg_typeinfo_state(get_irn_irg(node)) == ir_typeinfo_consistent);
-       assert(fromtype);
-
-       while (is_Pointer_type(totype) && is_Pointer_type(fromtype)) {
-               totype   = get_pointer_points_to_type(totype);
-               fromtype = get_pointer_points_to_type(fromtype);
-       }
-
-       assert(fromtype);
-
-       if (!is_Class_type(totype)) return 0;
-       return is_SubClass_of(fromtype, totype);
-}
-
-/* Checks for downcast.
- *
- * Returns true if the Cast node casts a class type to a sub type.
- */
-int is_Cast_downcast(ir_node *node)
-{
-       ir_type *totype   = get_Cast_type(node);
-       ir_type *fromtype = get_irn_typeinfo_type(get_Cast_op(node));
-
-       assert(get_irg_typeinfo_state(get_irn_irg(node)) == ir_typeinfo_consistent);
-       assert(fromtype);
-
-       while (is_Pointer_type(totype) && is_Pointer_type(fromtype)) {
-               totype   = get_pointer_points_to_type(totype);
-               fromtype = get_pointer_points_to_type(fromtype);
-       }
-
-       assert(fromtype);
-
-       if (!is_Class_type(totype)) return 0;
-       return is_SubClass_of(totype, fromtype);
-}
-
 int (is_unop)(const ir_node *node)
 {
        return is_unop_(node);
@@ -1165,39 +837,6 @@ void set_binop_right(ir_node *node, ir_node *right)
        set_irn_n(node, node->op->op_index + 1, right);
 }
 
-int is_Phi0(const ir_node *n)
-{
-       assert(n);
-
-       return ((get_irn_op(n) == op_Phi) &&
-               (get_irn_arity(n) == 0) &&
-               (get_irg_phase_state(get_irn_irg(n)) ==  phase_building));
-}
-
-ir_node **get_Phi_preds_arr(ir_node *node)
-{
-  assert(is_Phi(node));
-  return (ir_node **)&(get_irn_in(node)[1]);
-}
-
-int get_Phi_n_preds(const ir_node *node)
-{
-       assert(is_Phi(node) || is_Phi0(node));
-       return (get_irn_arity(node));
-}
-
-ir_node *get_Phi_pred(const ir_node *node, int pos)
-{
-       assert(is_Phi(node) || is_Phi0(node));
-       return get_irn_n(node, pos);
-}
-
-void set_Phi_pred(ir_node *node, int pos, ir_node *pred)
-{
-       assert(is_Phi(node) || is_Phi0(node));
-       set_irn_n(node, pos, pred);
-}
-
 ir_node *(get_Phi_next)(const ir_node *phi)
 {
        return get_Phi_next_(phi);
@@ -1210,71 +849,23 @@ void (set_Phi_next)(ir_node *phi, ir_node *next)
 
 int is_memop(const ir_node *node)
 {
-       unsigned code = get_irn_opcode(node);
-       return (code == iro_Load || code == iro_Store);
+       return is_op_uses_memory(get_irn_op(node));
 }
 
 ir_node *get_memop_mem(const ir_node *node)
 {
+       const ir_op *op = get_irn_op(node);
        assert(is_memop(node));
-       assert(n_Load_mem == 0 && n_Store_mem == 0);
-       return get_irn_n(node, 0);
+       return get_irn_n(node, op->memory_index);
 }
 
 void set_memop_mem(ir_node *node, ir_node *mem)
 {
+       const ir_op *op = get_irn_op(node);
        assert(is_memop(node));
-       assert(n_Load_mem == 0 && n_Store_mem == 0);
-       set_irn_n(node, 0, mem);
-}
-
-ir_node *get_memop_ptr(const ir_node *node)
-{
-       assert(is_memop(node));
-       assert(n_Load_mem == 1 && n_Store_mem == 1);
-       return get_irn_n(node, 1);
-}
-
-void set_memop_ptr(ir_node *node, ir_node *ptr)
-{
-       assert(is_memop(node));
-       assert(n_Load_mem == 1 && n_Store_mem == 1);
-       set_irn_n(node, 1, ptr);
-}
-
-
-ir_node **get_Sync_preds_arr(ir_node *node)
-{
-       assert(is_Sync(node));
-       return (ir_node **)&(get_irn_in(node)[1]);
-}
-
-int get_Sync_n_preds(const ir_node *node)
-{
-       assert(is_Sync(node));
-       return (get_irn_arity(node));
+       set_irn_n(node, op->memory_index, mem);
 }
 
-/*
-void set_Sync_n_preds(ir_node *node, int n_preds)
-{
-       assert(is_Sync(node));
-}
-*/
-
-ir_node *get_Sync_pred(const ir_node *node, int pos)
-{
-       assert(is_Sync(node));
-       return get_irn_n(node, pos);
-}
-
-void set_Sync_pred(ir_node *node, int pos, ir_node *pred)
-{
-       assert(is_Sync(node));
-       set_irn_n(node, pos, pred);
-}
-
-/* Add a new Sync predecessor */
 void add_Sync_pred(ir_node *node, ir_node *pred)
 {
        assert(is_Sync(node));
@@ -1322,59 +913,23 @@ int ir_throws_exception(const ir_node *node)
        return attr->throws_exception;
 }
 
-ir_node **get_Tuple_preds_arr(ir_node *node)
-{
-       assert(is_Tuple(node));
-       return (ir_node **)&(get_irn_in(node)[1]);
-}
-
-int get_Tuple_n_preds(const ir_node *node)
-{
-       assert(is_Tuple(node));
-       return get_irn_arity(node);
-}
-
-ir_node *get_Tuple_pred(const ir_node *node, int pos)
-{
-  assert(is_Tuple(node));
-  return get_irn_n(node, pos);
-}
-
-void set_Tuple_pred(ir_node *node, int pos, ir_node *pred)
-{
-       assert(is_Tuple(node));
-       set_irn_n(node, pos, pred);
-}
-
-int get_ASM_n_input_constraints(const ir_node *node)
-{
-       assert(is_ASM(node));
-       return ARR_LEN(node->attr.assem.input_constraints);
-}
-
-int get_ASM_n_output_constraints(const ir_node *node)
+size_t get_ASM_n_output_constraints(const ir_node *node)
 {
        assert(is_ASM(node));
        return ARR_LEN(node->attr.assem.output_constraints);
 }
 
-int get_ASM_n_clobbers(const ir_node *node)
+size_t get_ASM_n_clobbers(const ir_node *node)
 {
        assert(is_ASM(node));
        return ARR_LEN(node->attr.assem.clobbers);
 }
 
-/* returns the graph of a node */
 ir_graph *(get_irn_irg)(const ir_node *node)
 {
        return get_irn_irg_(node);
 }
 
-
-/*----------------------------------------------------------------*/
-/*  Auxiliary routines                                            */
-/*----------------------------------------------------------------*/
-
 ir_node *skip_Proj(ir_node *node)
 {
        /* don't assert node !!! */
@@ -1423,23 +978,6 @@ restart:
        return node;
 }
 
-/* returns operand of node if node is a Cast */
-ir_node *skip_Cast(ir_node *node)
-{
-       if (is_Cast(node))
-               return get_Cast_op(node);
-       return node;
-}
-
-/* returns operand of node if node is a Cast */
-const ir_node *skip_Cast_const(const ir_node *node)
-{
-       if (is_Cast(node))
-               return get_Cast_op(node);
-       return node;
-}
-
-/* returns operand of node if node is a Pin */
 ir_node *skip_Pin(ir_node *node)
 {
        if (is_Pin(node))
@@ -1447,7 +985,6 @@ ir_node *skip_Pin(ir_node *node)
        return node;
 }
 
-/* returns operand of node if node is a Confirm */
 ir_node *skip_Confirm(ir_node *node)
 {
        if (is_Confirm(node))
@@ -1455,7 +992,6 @@ ir_node *skip_Confirm(ir_node *node)
        return node;
 }
 
-/* skip all high-level ops */
 ir_node *skip_HighLevel_ops(ir_node *node)
 {
        while (is_op_highlevel(get_irn_op(node))) {
@@ -1465,15 +1001,15 @@ ir_node *skip_HighLevel_ops(ir_node *node)
 }
 
 
-/* This should compact Id-cycles to self-cycles. It has the same (or less?) complexity
- * than any other approach, as Id chains are resolved and all point to the real node, or
- * all id's are self loops.
- *
- * Note: This function takes 10% of mostly ANY the compiler run, so it's
- * a little bit "hand optimized".
- */
 ir_node *skip_Id(ir_node *node)
 {
+       /* This should compact Id-cycles to self-cycles. It has the same (or less?) complexity
+        * than any other approach, as Id chains are resolved and all point to the real node, or
+        * all id's are self loops.
+        *
+        * Note: This function takes 10% of mostly ANY the compiler run, so it's
+        * a little bit "hand optimized".
+        */
        ir_node *pred;
        /* don't assert node !!! */
 
@@ -1504,18 +1040,11 @@ ir_node *skip_Id(ir_node *node)
        }
 }
 
-int (is_strictConv)(const ir_node *node)
-{
-       return is_strictConv_(node);
-}
-
-/* Returns true if node is a SymConst node with kind symconst_addr_ent. */
 int (is_SymConst_addr_ent)(const ir_node *node)
 {
        return is_SymConst_addr_ent_(node);
 }
 
-/* Returns true if the operation manipulates control flow. */
 int is_cfop(const ir_node *node)
 {
        if (is_fragile_op(node) && ir_throws_exception(node))
@@ -1529,21 +1058,11 @@ int is_unknown_jump(const ir_node *node)
        return is_op_unknown_jump(get_irn_op(node));
 }
 
-/* Returns true if the operation can change the control flow because
-   of an exception. */
 int is_fragile_op(const ir_node *node)
 {
        return is_op_fragile(get_irn_op(node));
 }
 
-/* Returns the memory operand of fragile operations. */
-ir_node *get_fragile_op_mem(ir_node *node)
-{
-       assert(node && is_fragile_op(node));
-       return get_irn_n(node, node->op->fragile_mem_index);
-}
-
-/* Returns true if the operation is a forking control flow operation. */
 int (is_irn_forking)(const ir_node *node)
 {
        return is_irn_forking_(node);
@@ -1554,49 +1073,36 @@ void (copy_node_attr)(ir_graph *irg, const ir_node *old_node, ir_node *new_node)
        copy_node_attr_(irg, old_node, new_node);
 }
 
-/* Return the type attribute of a node n (SymConst, Call, Alloc, Free,
-   Cast) or NULL.*/
 ir_type *(get_irn_type_attr)(ir_node *node)
 {
        return get_irn_type_attr_(node);
 }
 
-/* Return the entity attribute of a node n (SymConst, Sel) or NULL. */
 ir_entity *(get_irn_entity_attr)(ir_node *node)
 {
        return get_irn_entity_attr_(node);
 }
 
-/* Returns non-zero for constant-like nodes. */
 int (is_irn_constlike)(const ir_node *node)
 {
        return is_irn_constlike_(node);
 }
 
-/*
- * Returns non-zero for nodes that are allowed to have keep-alives and
- * are neither Block nor PhiM.
- */
 int (is_irn_keep)(const ir_node *node)
 {
        return is_irn_keep_(node);
 }
 
-/*
- * Returns non-zero for nodes that are always placed in the start block.
- */
 int (is_irn_start_block_placed)(const ir_node *node)
 {
        return is_irn_start_block_placed_(node);
 }
 
-/* Returns non-zero for nodes that are CSE neutral to its users. */
 int (is_irn_cse_neutral)(const ir_node *node)
 {
        return is_irn_cse_neutral_(node);
 }
 
-/* Gets the string representation of the jump prediction .*/
 const char *get_cond_jmp_predicate_name(cond_jmp_predicate pred)
 {
 #define X(a)    case a: return #a
@@ -1627,64 +1133,36 @@ static ir_entity *get_SymConst_attr_entity(const ir_node *self)
        return NULL;
 }
 
-/** the get_type_attr operation must be always implemented */
-static ir_type *get_Null_type(const ir_node *n)
+static void register_get_type_func(ir_op *op, get_type_attr_func func)
 {
-       (void) n;
-       return firm_unknown_type;
+       op->ops.get_type_attr = func;
 }
 
-/* Sets the get_type operation for an ir_op_ops. */
-ir_op_ops *firm_set_default_get_type_attr(unsigned code, ir_op_ops *ops)
+static void register_get_entity_func(ir_op *op, get_entity_attr_func func)
 {
-       switch (code) {
-       case iro_SymConst: ops->get_type_attr = get_SymConst_attr_type; break;
-       case iro_Call:     ops->get_type_attr = get_Call_type; break;
-       case iro_Alloc:    ops->get_type_attr = get_Alloc_type; break;
-       case iro_Free:     ops->get_type_attr = get_Free_type; break;
-       case iro_Cast:     ops->get_type_attr = get_Cast_type; break;
-       default:
-               /* not allowed to be NULL */
-               if (! ops->get_type_attr)
-                       ops->get_type_attr = get_Null_type;
-               break;
-       }
-       return ops;
+       op->ops.get_entity_attr = func;
 }
 
-/** the get_entity_attr operation must be always implemented */
-static ir_entity *get_Null_ent(const ir_node *n)
+void ir_register_getter_ops(void)
 {
-       (void) n;
-       return NULL;
-}
+       register_get_type_func(op_Alloc,    get_Alloc_type);
+       register_get_type_func(op_Builtin,  get_Builtin_type);
+       register_get_type_func(op_Call,     get_Call_type);
+       register_get_type_func(op_CopyB,    get_CopyB_type);
+       register_get_type_func(op_Free,     get_Free_type);
+       register_get_type_func(op_InstOf,   get_InstOf_type);
+       register_get_type_func(op_SymConst, get_SymConst_attr_type);
 
-/* Sets the get_type operation for an ir_op_ops. */
-ir_op_ops *firm_set_default_get_entity_attr(unsigned code, ir_op_ops *ops)
-{
-       switch (code) {
-       case iro_SymConst: ops->get_entity_attr = get_SymConst_attr_entity; break;
-       case iro_Sel:      ops->get_entity_attr = get_Sel_entity; break;
-       default:
-               /* not allowed to be NULL */
-               if (! ops->get_entity_attr)
-                       ops->get_entity_attr = get_Null_ent;
-               break;
-       }
-       return ops;
+       register_get_entity_func(op_SymConst, get_SymConst_attr_entity);
+       register_get_entity_func(op_Sel,      get_Sel_entity);
+       register_get_entity_func(op_Block,    get_Block_entity);
 }
 
-/* Sets the debug information of a node. */
 void (set_irn_dbg_info)(ir_node *n, dbg_info *db)
 {
        set_irn_dbg_info_(n, db);
 }
 
-/**
- * Returns the debug information of an node.
- *
- * @param n   The node.
- */
 dbg_info *(get_irn_dbg_info)(const ir_node *n)
 {
        return get_irn_dbg_info_(n);
@@ -1742,32 +1220,18 @@ ir_switch_table *ir_switch_table_duplicate(ir_graph *irg,
        return res;
 }
 
-/*
- * Calculate a hash value of a node.
- */
-unsigned firm_default_hash(const ir_node *node)
+bool only_used_by_keepalive(const ir_node *node)
 {
-       unsigned h;
-       int i, irn_arity;
-
-       /* hash table value = 9*(9*(9*(9*(9*arity+in[0])+in[1])+ ...)+mode)+code */
-       h = irn_arity = get_irn_arity(node);
-
-       /* consider all in nodes... except the block if not a control flow. */
-       for (i = is_cfop(node) ? -1 : 0;  i < irn_arity;  ++i) {
-               ir_node *pred = get_irn_n(node, i);
-               if (is_irn_cse_neutral(pred))
-                       h *= 9;
-               else
-                       h = 9*h + HASH_PTR(pred);
+       foreach_out_edge(node, edge) {
+               ir_node *succ = get_edge_src_irn(edge);
+               if (is_End(succ))
+                       continue;
+               if (is_Proj(succ) && only_used_by_keepalive(succ))
+                       return true;
+               /* found a real user */
+               return false;
        }
-
-       /* ...mode,... */
-       h = 9*h + HASH_PTR(get_irn_mode(node));
-       /* ...and code */
-       h = 9*h + HASH_PTR(get_irn_op(node));
-
-       return h;
+       return true;
 }
 
 /* include generated code */