Several bug fixes
[libfirm] / ir / be / beabi.c
index a763407..cd006b5 100644 (file)
@@ -2,7 +2,8 @@
  * ABI lowering.
  *
  * @author Sebastian Hack
- * @date 7.3.2005
+ * @date   7.3.2005
+ * @cvsid  $Id$
  */
 
 #ifdef HAVE_CONFIG_H
 #include "irgwalk.h"
 #include "irprintf_t.h"
 #include "irgopt.h"
+#include "irbitset.h"
+#include "height.h"
+#include "pdeq.h"
+#include "irtools.h"
 
 #include "be.h"
 #include "beabi.h"
 #include "belive_t.h"
 #include "besched_t.h"
 
-#define MAX(x, y) ((x) > (y) ? (x) : (y))
-#define MIN(x, y) ((x) < (y) ? (x) : (y))
-
 typedef struct _be_abi_call_arg_t {
-       unsigned is_res   : 1;
-       unsigned in_reg   : 1;
-       unsigned on_stack : 1;
+       unsigned is_res   : 1;  /**< 1: the call argument is a return value. 0: it's a call parameter. */
+       unsigned in_reg   : 1;  /**< 1: this argument is transmitted in registers. */
+       unsigned on_stack : 1;  /**< 1: this argument is transmitted on the stack. */
 
        int pos;
        const arch_register_t *reg;
@@ -50,37 +52,40 @@ typedef struct _be_abi_call_arg_t {
 struct _be_abi_call_t {
        be_abi_call_flags_t flags;
        const be_abi_callbacks_t *cb;
-       type *between_type;
+       ir_type *between_type;
        set *params;
 };
 
 #define N_FRAME_TYPES 3
 
-typedef struct _be_stack_frame_t {
-       type *arg_type;
-       type *between_type;
-       type *frame_type;
+/**
+ * This type describes the stack layout.
+ * The stack is divided into 3 parts:
+ * - arg_type:     A struct type describing the stack arguments and it's order.
+ * - between_type: A struct type describing the stack layout between arguments
+ *                 and frame type
+ * - frame_type:   A class type descibing the frame layout
+ */
+typedef struct _be_stack_layout_t {
+       ir_type *arg_type;                 /**< A type describing the stack argument layout. */
+       ir_type *between_type;             /**< A type describing the "between" layout. */
+       ir_type *frame_type;               /**< The frame type. */
 
-       type *order[N_FRAME_TYPES];        /**< arg, between and frame types ordered. */
+       ir_type *order[N_FRAME_TYPES];     /**< arg, between and frame types ordered. */
 
        int initial_offset;
-       int stack_dir;
-} be_stack_frame_t;
-
-struct _be_stack_slot_t {
-       struct _be_stack_frame_t *frame;
-       entity *ent;
-};
+       int stack_dir;                     /**< -1 for decreasing, 1 for increasing. */
+} be_stack_layout_t;
 
 struct _be_abi_irg_t {
        struct obstack       obst;
-       be_stack_frame_t     *frame;        /**< The stack frame model. */
+       be_stack_layout_t    *frame;        /**< The stack frame model. */
        const be_irg_t       *birg;         /**< The back end IRG. */
        const arch_isa_t     *isa;          /**< The isa. */
        survive_dce_t        *dce_survivor;
 
        be_abi_call_t        *call;         /**< The ABI call information. */
-       type                 *method_type;  /**< The type of the method of the IRG. */
+       ir_type              *method_type;  /**< The type of the method of the IRG. */
 
        ir_node              *init_sp;      /**< The node representing the stack pointer
                                                                             at the start of the function. */
@@ -110,6 +115,7 @@ struct _be_abi_irg_t {
 /* Forward, since be need it in be_abi_introduce(). */
 static const arch_irn_ops_if_t abi_irn_ops;
 static const arch_irn_handler_t abi_irn_handler;
+static heights_t *ir_heights;
 
 /* Flag: if set, try to omit the frame pointer if called by the backend */
 int be_omit_fp = 1;
@@ -212,7 +218,7 @@ be_abi_call_flags_t be_abi_call_get_flags(const be_abi_call_t *call)
  *
  * @return the new ABI call object
  */
-static be_abi_call_t *be_abi_call_new()
+static be_abi_call_t *be_abi_call_new(void)
 {
        be_abi_call_t *call = xmalloc(sizeof(call[0]));
        call->flags.val  = 0;
@@ -249,10 +255,10 @@ static void be_abi_call_free(be_abi_call_t *call)
      and the spills.
 */
 
-static int get_stack_entity_offset(be_stack_frame_t *frame, entity *ent, int bias)
+static int get_stack_entity_offset(be_stack_layout_t *frame, entity *ent, int bias)
 {
-       type *t = get_entity_owner(ent);
-       int ofs = get_entity_offset_bytes(ent);
+       ir_type *t = get_entity_owner(ent);
+       int ofs    = get_entity_offset_bytes(ent);
 
        int i, index;
 
@@ -278,12 +284,12 @@ static int get_stack_entity_offset(be_stack_frame_t *frame, entity *ent, int bia
 /**
  * Retrieve the entity with given offset from a frame type.
  */
-static entity *search_ent_with_offset(type *t, int offset)
+static entity *search_ent_with_offset(ir_type *t, int offset)
 {
        int i, n;
 
-       for(i = 0, n = get_class_n_members(t); i < n; ++i) {
-               entity *ent = get_class_member(t, i);
+       for(i = 0, n = get_compound_n_members(t); i < n; ++i) {
+               entity *ent = get_compound_member(t, i);
                if(get_entity_offset_bytes(ent) == offset)
                        return ent;
        }
@@ -291,16 +297,28 @@ static entity *search_ent_with_offset(type *t, int offset)
        return NULL;
 }
 
-static int stack_frame_compute_initial_offset(be_stack_frame_t *frame)
+static int stack_frame_compute_initial_offset(be_stack_layout_t *frame)
 {
-       type   *base = frame->stack_dir < 0 ? frame->between_type : frame->frame_type;
-       entity *ent  = search_ent_with_offset(base, 0);
+       ir_type *base = frame->stack_dir < 0 ? frame->between_type : frame->frame_type;
+       entity *ent   = search_ent_with_offset(base, 0);
        frame->initial_offset = 0;
        frame->initial_offset = get_stack_entity_offset(frame, ent, 0);
        return frame->initial_offset;
 }
 
-static be_stack_frame_t *stack_frame_init(be_stack_frame_t *frame, type *args, type *between, type *locals, int stack_dir)
+/**
+ * Initializes the frame layout from parts
+ *
+ * @param frame     the stack layout that will be initialized
+ * @param args      the stack argument layout type
+ * @param between   the between layout type
+ * @param locals    the method frame type
+ * @param stack_dir the stack direction
+ *
+ * @return the initialized stack layout
+ */
+static be_stack_layout_t *stack_frame_init(be_stack_layout_t *frame, ir_type *args,
+                                           ir_type *between, ir_type *locals, int stack_dir)
 {
        frame->arg_type       = args;
        frame->between_type   = between;
@@ -313,64 +331,30 @@ static be_stack_frame_t *stack_frame_init(be_stack_frame_t *frame, type *args, t
                frame->order[0] = args;
                frame->order[2] = locals;
        }
-
        else {
                frame->order[0] = locals;
                frame->order[2] = args;
        }
-
        return frame;
 }
 
-static void stack_frame_dump(FILE *file, be_stack_frame_t *frame)
+/** Dumps the stack layout to file. */
+static void stack_layout_dump(FILE *file, be_stack_layout_t *frame)
 {
        int i, j, n;
 
        ir_fprintf(file, "initial offset: %d\n", frame->initial_offset);
-       for(j = 0; j < N_FRAME_TYPES; ++j) {
-               type *t = frame->order[j];
+       for (j = 0; j < N_FRAME_TYPES; ++j) {
+               ir_type *t = frame->order[j];
 
-               ir_fprintf(file, "type %d: %Fm size: %d\n", j, t, get_type_size_bytes(t));
-               for(i = 0, n = get_class_n_members(t); i < n; ++i) {
-                       entity *ent = get_class_member(t, i);
+               ir_fprintf(file, "type %d: %F size: %d\n", j, t, get_type_size_bytes(t));
+               for (i = 0, n = get_compound_n_members(t); i < n; ++i) {
+                       entity *ent = get_compound_member(t, i);
                        ir_fprintf(file, "\t%F int ofs: %d glob ofs: %d\n", ent, get_entity_offset_bytes(ent), get_stack_entity_offset(frame, ent, 0));
                }
        }
 }
 
-/**
- * If irn is a Sel node computes the address of an entity
- * on the frame type return the entity, else NULL.
- */
-static INLINE entity *get_sel_ent(ir_node *irn)
-{
-       if(is_Sel(irn) && get_Sel_ptr(irn) == get_irg_frame(get_irn_irg(irn))) {
-               return get_Sel_entity(irn);
-       }
-
-       return NULL;
-}
-
-/**
- * Walker: Replaces Loads, Stores and Sels of frame type entities
- * by FrameLoad, FrameStore and FrameAdress.
- */
-static void lower_frame_sels_walker(ir_node *irn, void *data)
-{
-       ir_node *nw  = NULL;
-       entity *ent = get_sel_ent(irn);
-
-       if(ent != NULL) {
-               be_abi_irg_t *env = data;
-               ir_node *bl       = get_nodes_block(irn);
-               ir_graph *irg     = get_irn_irg(bl);
-               ir_node *frame    = get_irg_frame(irg);
-
-               nw = be_new_FrameAddr(env->isa->sp->reg_class, irg, bl, frame, ent);
-               exchange(irn, nw);
-       }
-}
-
 /**
  * Returns non-zero if the call argument at given position
  * is transfered on the stack.
@@ -399,7 +383,7 @@ static INLINE int is_on_stack(be_abi_call_t *call, int pos)
  * @param curr_sp The stack pointer node to use.
  * @return The stack pointer after the call.
  */
-static ir_node *adjust_call(be_abi_irg_t *env, ir_node *irn, ir_node *curr_sp)
+static ir_node *adjust_call(be_abi_irg_t *env, ir_node *irn, ir_node *curr_sp, ir_node *alloca_copy)
 {
        ir_graph *irg             = env->birg->irg;
        const arch_isa_t *isa     = env->birg->main_env->arch_env->isa;
@@ -487,7 +471,11 @@ static ir_node *adjust_call(be_abi_irg_t *env, ir_node *irn, ir_node *curr_sp)
                 * moving the stack pointer along the stack's direction.
                 */
                if(stack_dir < 0 && !do_seq && !no_alloc) {
-                       curr_sp = be_new_IncSP(sp, irg, bl, curr_sp, no_mem, stack_size, be_stack_dir_expand);
+                       curr_sp = be_new_IncSP(sp, irg, bl, curr_sp, stack_size);
+                       if(alloca_copy) {
+                               add_irn_dep(curr_sp, alloca_copy);
+                               alloca_copy = NULL;
+                       }
                }
 
                assert(mode_is_reference(mach_mode) && "machine mode must be pointer");
@@ -497,7 +485,7 @@ static ir_node *adjust_call(be_abi_irg_t *env, ir_node *irn, ir_node *curr_sp)
                        ir_node *param         = get_Call_param(irn, p);
                        ir_node *addr          = curr_sp;
                        ir_node *mem           = NULL;
-                       type *param_type       = get_method_param_type(mt, p);
+                       ir_type *param_type    = get_method_param_type(mt, p);
                        int param_size         = get_type_size_bytes(param_type) + arg->space_after;
 
                        /*
@@ -507,8 +495,12 @@ static ir_node *adjust_call(be_abi_irg_t *env, ir_node *irn, ir_node *curr_sp)
                         */
                        if (do_seq) {
                                curr_ofs = 0;
-                               addr = curr_sp = be_new_IncSP(sp, irg, bl, curr_sp, curr_mem,
-                                       param_size + arg->space_before, be_stack_dir_expand);
+                               addr = curr_sp = be_new_IncSP(sp, irg, bl, curr_sp, param_size + arg->space_before);
+                               if(alloca_copy) {
+                                       add_irn_dep(curr_sp, alloca_copy);
+                                       alloca_copy = NULL;
+                               }
+                               add_irn_dep(curr_sp, curr_mem);
                        }
                        else {
                                curr_ofs += arg->space_before;
@@ -635,6 +627,8 @@ static ir_node *adjust_call(be_abi_irg_t *env, ir_node *irn, ir_node *curr_sp)
        */
        be_node_set_reg_class(low_call, be_pos_Call_ptr, sp->reg_class);
 
+       DBG((env->dbg, LEVEL_3, "\tcreated backend call %+F\n", low_call));
+
        /* Set the register classes and constraints of the Call parameters. */
        for(i = 0; i < n_low_args; ++i) {
                int index = low_args[i];
@@ -704,8 +698,14 @@ static ir_node *adjust_call(be_abi_irg_t *env, ir_node *irn, ir_node *curr_sp)
                        mem_proj = new_r_Proj(irg, bl, low_call, mode_M, pn_Call_M);
 
                 /* Clean up the stack frame if we allocated it */
-               if(!no_alloc)
-                       curr_sp = be_new_IncSP(sp, irg, bl, curr_sp, mem_proj, stack_size, be_stack_dir_shrink);
+               if(!no_alloc) {
+                       curr_sp = be_new_IncSP(sp, irg, bl, curr_sp, -stack_size);
+                       add_irn_dep(curr_sp, mem_proj);
+                       if(alloca_copy) {
+                               add_irn_dep(curr_sp, alloca_copy);
+                               alloca_copy = NULL;
+                       }
+               }
        }
 
        be_abi_call_free(call);
@@ -720,7 +720,7 @@ static ir_node *adjust_call(be_abi_irg_t *env, ir_node *irn, ir_node *curr_sp)
  * Adjust an alloca.
  * The alloca is transformed into a back end alloca node and connected to the stack nodes.
  */
-static ir_node *adjust_alloc(be_abi_irg_t *env, ir_node *alloc, ir_node *curr_sp)
+static ir_node *adjust_alloc(be_abi_irg_t *env, ir_node *alloc, ir_node *curr_sp, ir_node **result_copy)
 {
        if (get_Alloc_where(alloc) == stack_alloc) {
                ir_node *bl        = get_nodes_block(alloc);
@@ -730,6 +730,8 @@ static ir_node *adjust_alloc(be_abi_irg_t *env, ir_node *alloc, ir_node *curr_sp
 
                const ir_edge_t *edge;
                ir_node *new_alloc;
+               ir_node *addr;
+               ir_node *copy;
 
                foreach_out_edge(alloc, edge) {
                        ir_node *irn = get_edge_src_irn(edge);
@@ -750,7 +752,7 @@ static ir_node *adjust_alloc(be_abi_irg_t *env, ir_node *alloc, ir_node *curr_sp
                /* Beware: currently Alloc nodes without a result might happen,
                   only escape analysis kills them and this phase runs only for object
                   oriented source. We kill the Alloc here. */
-               if (alloc_res == NULL) {
+               if (alloc_res == NULL && alloc_mem) {
                        exchange(alloc_mem, get_Alloc_mem(alloc));
                        return curr_sp;
                }
@@ -760,44 +762,61 @@ static ir_node *adjust_alloc(be_abi_irg_t *env, ir_node *alloc, ir_node *curr_sp
                env->call->flags.bits.try_omit_fp = 0;
                new_alloc = be_new_AddSP(env->isa->sp, irg, bl, curr_sp, get_Alloc_size(alloc));
 
-               exchange(alloc_res, env->isa->stack_dir < 0 ? new_alloc : curr_sp);
+               exchange(alloc, new_alloc);
 
                if(alloc_mem != NULL)
-                       exchange(alloc_mem, new_r_NoMem(irg));
+                       set_Proj_proj(alloc_mem, pn_be_AddSP_M);
+
+               /* fix projnum of alloca res */
+               set_Proj_proj(alloc_res, pn_be_AddSP_res);
+
+               addr = env->isa->stack_dir < 0 ? alloc_res : curr_sp;
 
-               curr_sp = new_alloc;
+               /* copy the address away, since it could be used after further stack pointer modifications. */
+               /* Let it point curr_sp just for the moment, I'll reroute it in a second. */
+               *result_copy = copy = be_new_Copy(env->isa->sp->reg_class, irg, bl, curr_sp);
+
+               /* Let all users of the Alloc() result now point to the copy. */
+               edges_reroute(alloc_res, copy, irg);
+
+               /* Rewire the copy appropriately. */
+               set_irn_n(copy, be_pos_Copy_op, addr);
+
+               curr_sp = alloc_res;
        }
 
        return curr_sp;
 }
 
+/* the following function is replaced by the usage of the heights module */
+#if 0
 /**
  * Walker for dependent_on().
  * This function searches a node tgt recursively from a given node
  * but is restricted to the given block.
  * @return 1 if tgt was reachable from curr, 0 if not.
  */
-static int check_dependence(ir_node *curr, ir_node *tgt, ir_node *bl, unsigned long visited_nr)
+static int check_dependence(ir_node *curr, ir_node *tgt, ir_node *bl)
 {
        int n, i;
 
-       if(get_irn_visited(curr) >= visited_nr)
-               return 0;
-
-       set_irn_visited(curr, visited_nr);
-       if(get_nodes_block(curr) != bl)
+       if (get_nodes_block(curr) != bl)
                return 0;
 
-       if(curr == tgt)
+       if (curr == tgt)
                return 1;
 
-       for(i = 0, n = get_irn_arity(curr); i < n; ++i) {
-               if(check_dependence(get_irn_n(curr, i), tgt, bl, visited_nr))
-                       return 1;
+       /* Phi functions stop the recursion inside a basic block */
+       if (! is_Phi(curr)) {
+               for(i = 0, n = get_irn_arity(curr); i < n; ++i) {
+                       if (check_dependence(get_irn_n(curr, i), tgt, bl))
+                               return 1;
+               }
        }
 
        return 0;
 }
+#endif /* if 0 */
 
 /**
  * Check if a node is somehow data dependent on another one.
@@ -809,12 +828,11 @@ static int check_dependence(ir_node *curr, ir_node *tgt, ir_node *bl, unsigned l
 static int dependent_on(ir_node *n1, ir_node *n2)
 {
        ir_node *bl   = get_nodes_block(n1);
-       ir_graph *irg = get_irn_irg(bl);
-       long vis_nr   = get_irg_visited(irg) + 1;
 
        assert(bl == get_nodes_block(n2));
-       set_irg_visited(irg, vis_nr);
-       return check_dependence(n1, n2, bl, vis_nr);
+
+       return heights_reachable_in_block(ir_heights, n1, n2);
+       //return check_dependence(n1, n2, bl);
 }
 
 static int cmp_call_dependecy(const void *c1, const void *c2)
@@ -828,17 +846,27 @@ static int cmp_call_dependecy(const void *c1, const void *c2)
                1  if second is "smaller" that first
                -1 if first is "smaller" that second
        */
-       return n1 == n2 ? 0 : (dependent_on(n1, n2) ? -1 : 1);
+       if (dependent_on(n1, n2))
+               return -1;
+
+       if (dependent_on(n2, n1))
+               return 1;
+
+       return 0;
 }
 
+/**
+ * Walker: links all Call nodes to the Block they are contained.
+ */
 static void link_calls_in_block_walker(ir_node *irn, void *data)
 {
-       if(is_Call(irn)) {
+       if(is_Call(irn) || (get_irn_opcode(irn) == iro_Alloc && get_Alloc_where(irn) == stack_alloc)) {
                be_abi_irg_t *env = data;
                ir_node *bl       = get_nodes_block(irn);
                void *save        = get_irn_link(bl);
 
-               env->call->flags.bits.irg_is_leaf = 0;
+               if (is_Call(irn))
+                       env->call->flags.bits.irg_is_leaf = 0;
 
                set_irn_link(irn, save);
                set_irn_link(bl, irn);
@@ -846,9 +874,10 @@ static void link_calls_in_block_walker(ir_node *irn, void *data)
 }
 
 /**
- * Process all call nodes inside a basic block.
+ * Block-walker:
+ * Process all Call nodes inside a basic block.
  * Note that the link field of the block must contain a linked list of all
- * Call nodes inside the block. We first order this list according to data dependency
+ * Call nodes inside the Block. We first order this list according to data dependency
  * and that connect the calls together.
  */
 static void process_calls_in_block(ir_node *bl, void *data)
@@ -865,6 +894,7 @@ static void process_calls_in_block(ir_node *bl, void *data)
        if(n > 0) {
                ir_node *keep;
                ir_node **nodes;
+               ir_node *copy = NULL;
                int i;
 
                nodes = obstack_finish(&env->obst);
@@ -875,12 +905,13 @@ static void process_calls_in_block(ir_node *bl, void *data)
                for(i = n - 1; i >= 0; --i) {
                        ir_node *irn = nodes[i];
 
+                       DBG((env->dbg, LEVEL_3, "\tprocessing call %+F\n", irn));
                        switch(get_irn_opcode(irn)) {
                        case iro_Call:
-                               curr_sp = adjust_call(env, irn, curr_sp);
+                               curr_sp = adjust_call(env, irn, curr_sp, copy);
                                break;
                        case iro_Alloc:
-                               curr_sp = adjust_alloc(env, irn, curr_sp);
+                               curr_sp = adjust_alloc(env, irn, curr_sp, &copy);
                                break;
                        default:
                                break;
@@ -907,7 +938,10 @@ static void process_calls(be_abi_irg_t *env)
 
        env->call->flags.bits.irg_is_leaf = 1;
        irg_walk_graph(irg, firm_clear_link, link_calls_in_block_walker, env);
+
+       ir_heights = heights_new(env->birg->irg);
        irg_block_walk_graph(irg, NULL, process_calls_in_block, env);
+       heights_free(ir_heights);
 }
 
 static void collect_return_walker(ir_node *irn, void *data)
@@ -935,7 +969,7 @@ static ir_node *setup_frame(be_abi_irg_t *env)
        int stack_nr       = get_Proj_proj(stack);
 
        if(flags.try_omit_fp) {
-               stack = be_new_IncSP(sp, irg, bl, stack, no_mem, BE_STACK_FRAME_SIZE, be_stack_dir_expand);
+               stack = be_new_IncSP(sp, irg, bl, stack, no_mem, BE_STACK_FRAME_SIZE_EXPAND);
                frame = stack;
        }
 
@@ -949,7 +983,7 @@ static ir_node *setup_frame(be_abi_irg_t *env)
                        arch_set_irn_register(env->birg->main_env->arch_env, frame, bp);
                }
 
-               stack = be_new_IncSP(sp, irg, bl, stack, frame, BE_STACK_FRAME_SIZE, be_stack_dir_expand);
+               stack = be_new_IncSP(sp, irg, bl, stack, frame, BE_STACK_FRAME_SIZE_EXPAND);
        }
 
        be_node_set_flags(env->reg_params, -(stack_nr + 1), arch_irn_flags_ignore);
@@ -974,7 +1008,7 @@ static void clearup_frame(be_abi_irg_t *env, ir_node *ret, pmap *reg_map, struct
        pmap_entry *ent;
 
        if(env->call->flags.bits.try_omit_fp) {
-               stack = be_new_IncSP(sp, irg, bl, stack, ret_mem, BE_STACK_FRAME_SIZE, be_stack_dir_shrink);
+               stack = be_new_IncSP(sp, irg, bl, stack, ret_mem, -BE_STACK_FRAME_SIZE_SHRINK);
        }
 
        else {
@@ -998,6 +1032,17 @@ static void clearup_frame(be_abi_irg_t *env, ir_node *ret, pmap *reg_map, struct
 */
 #endif
 
+/**
+ * Computes the stack argument layout type.
+ * Changes a possibly allocated value param type by moving
+ * entities to the stack layout type.
+ *
+ * @param env          the ABI environment
+ * @param call         the current call ABI
+ * @param method_type  the method type
+ *
+ * @return the stack argument layout type
+ */
 static ir_type *compute_arg_type(be_abi_irg_t *env, be_abi_call_t *call, ir_type *method_type)
 {
        int dir  = env->call->flags.bits.left_to_right ? 1 : -1;
@@ -1009,17 +1054,28 @@ static ir_type *compute_arg_type(be_abi_irg_t *env, be_abi_call_t *call, ir_type
        char buf[128];
        ir_type *res;
        int i;
+       ir_type *val_param_tp = get_method_value_param_type(method_type);
+       ident *id = get_entity_ident(get_irg_entity(env->birg->irg));
 
-       snprintf(buf, sizeof(buf), "%s_arg_type", get_entity_name(get_irg_entity(env->birg->irg)));
-       res = new_type_class(new_id_from_str(buf));
-
-       for(i = 0; i < n; ++i, curr += inc) {
-               type *param_type       = get_method_param_type(method_type, curr);
+       res = new_type_struct(mangle_u(id, new_id_from_chars("arg_type", 8)));
+       for (i = 0; i < n; ++i, curr += inc) {
+               ir_type *param_type    = get_method_param_type(method_type, curr);
                be_abi_call_arg_t *arg = get_call_arg(call, 0, curr);
 
-               if(arg->on_stack) {
-                       snprintf(buf, sizeof(buf), "param_%d", i);
-                       arg->stack_ent = new_entity(res, new_id_from_str(buf), param_type);
+               if (arg->on_stack) {
+                       if (val_param_tp) {
+                               /* the entity was already created, move it to the param type */
+                               arg->stack_ent = get_method_value_param_ent(method_type, i);
+                               remove_struct_member(val_param_tp, arg->stack_ent);
+                               set_entity_owner(arg->stack_ent, res);
+                               add_struct_member(res, arg->stack_ent);
+                               /* must be automatic to set a fixed layout */
+                               set_entity_allocation(arg->stack_ent, allocation_automatic);
+                       }
+                       else {
+                               snprintf(buf, sizeof(buf), "param_%d", i);
+                               arg->stack_ent = new_entity(res, new_id_from_str(buf), param_type);
+                       }
                        ofs += arg->space_before;
                        ofs = round_up2(ofs, arg->alignment);
                        set_entity_offset_bytes(arg->stack_ent, ofs);
@@ -1027,8 +1083,8 @@ static ir_type *compute_arg_type(be_abi_irg_t *env, be_abi_call_t *call, ir_type
                        ofs += get_type_size_bytes(param_type);
                }
        }
-
        set_type_size_bytes(res, ofs);
+       set_type_state(res, layout_fixed);
        return res;
 }
 
@@ -1113,8 +1169,8 @@ static reg_node_map_t *reg_map_to_arr(struct obstack *obst, pmap *reg_map)
 static ir_node *create_barrier(be_abi_irg_t *env, ir_node *bl, ir_node **mem, pmap *regs, int in_req)
 {
        ir_graph *irg = env->birg->irg;
+       int n_regs    = pmap_count(regs);
        int n;
-       int n_regs = pmap_count(regs);
        ir_node *irn;
        ir_node **in;
        reg_node_map_t *rm;
@@ -1134,9 +1190,10 @@ static ir_node *create_barrier(be_abi_irg_t *env, ir_node *bl, ir_node **mem, pm
        obstack_free(&env->obst, in);
 
        for(n = 0; n < n_regs; ++n) {
-               int pos = BE_OUT_POS(n);
-               ir_node *proj;
                const arch_register_t *reg = rm[n].reg;
+               int flags                  = 0;
+               int pos                    = BE_OUT_POS(n);
+               ir_node *proj;
 
                proj = new_r_Proj(irg, bl, irn, get_irn_mode(rm[n].irn), n);
                be_node_set_reg_class(irn, n, reg->reg_class);
@@ -1148,7 +1205,12 @@ static ir_node *create_barrier(be_abi_irg_t *env, ir_node *bl, ir_node **mem, pm
 
                /* if the proj projects a ignore register or a node which is set to ignore, propagate this property. */
                if(arch_register_type_is(reg, ignore) || arch_irn_is(env->birg->main_env->arch_env, in[n], ignore))
-                       be_node_set_flags(irn, pos, arch_irn_flags_ignore);
+                       flags |= arch_irn_flags_ignore;
+
+               if(arch_irn_is(env->birg->main_env->arch_env, in[n], modify_sp))
+                       flags |= arch_irn_flags_modify_sp;
+
+               be_node_set_flags(irn, pos, flags);
 
                pmap_insert(regs, (void *) reg, proj);
        }
@@ -1193,10 +1255,12 @@ static ir_node *create_be_return(be_abi_irg_t *env, ir_node *irn, ir_node *bl, i
        */
        stack = be_abi_reg_map_get(env->regs, isa->sp);
        if (keep) {
+               ir_node *bad = new_r_Bad(env->birg->irg);
                stack = get_irn_n(keep, 0);
-               exchange(keep, new_r_Bad(env->birg->irg));
+               set_nodes_block(keep, bad);
+               set_irn_n(keep, 0, bad);
+               // exchange(keep, new_r_Bad(env->birg->irg));
        }
-       be_abi_reg_map_set(reg_map, isa->sp, stack);
 
        /* Insert results for Return into the register map. */
        for(i = 0; i < n_res; ++i) {
@@ -1213,6 +1277,8 @@ static ir_node *create_be_return(be_abi_irg_t *env, ir_node *irn, ir_node *bl, i
                        pmap_insert(reg_map, ent->key, ent->value);
        }
 
+       be_abi_reg_map_set(reg_map, isa->sp, stack);
+
        /* Make the Epilogue node and call the arch's epilogue maker. */
        create_barrier(env, bl, &mem, reg_map, 1);
        call->cb->epilogue(env->cb, bl, &mem, reg_map);
@@ -1235,7 +1301,6 @@ static ir_node *create_be_return(be_abi_irg_t *env, ir_node *irn, ir_node *bl, i
        /* clear SP entry, since it has already been grown. */
        pmap_insert(reg_map, (void *) isa->sp, NULL);
        for(i = 0; i < n_res; ++i) {
-               ir_node *res           = get_Return_res(irn, i);
                be_abi_call_arg_t *arg = get_call_arg(call, 1, i);
 
                in[n]     = be_abi_reg_map_get(reg_map, arg->reg);
@@ -1268,6 +1333,149 @@ static ir_node *create_be_return(be_abi_irg_t *env, ir_node *irn, ir_node *bl, i
        return ret;
 }
 
+typedef struct lower_frame_sels_env_t {
+       be_abi_irg_t *env;
+       entity       *value_param_list;  /**< the list of all value param antities */
+} lower_frame_sels_env_t;
+
+/**
+ * Walker: Replaces Sels of frame type and
+ * value param type entities by FrameAddress.
+ */
+static void lower_frame_sels_walker(ir_node *irn, void *data)
+{
+       lower_frame_sels_env_t *ctx = data;
+
+       if (is_Sel(irn)) {
+               ir_graph *irg        = current_ir_graph;
+               ir_node  *frame      = get_irg_frame(irg);
+               ir_node  *param_base = get_irg_value_param_base(irg);
+               ir_node  *ptr        = get_Sel_ptr(irn);
+
+               if (ptr == frame || ptr == param_base) {
+                       be_abi_irg_t *env = ctx->env;
+                       entity       *ent = get_Sel_entity(irn);
+                       ir_node      *bl  = get_nodes_block(irn);
+                       ir_node      *nw;
+
+                       nw = be_new_FrameAddr(env->isa->sp->reg_class, irg, bl, frame, ent);
+                       exchange(irn, nw);
+
+                       if (ptr == param_base) {
+                               set_entity_link(ent, ctx->value_param_list);
+                               ctx->value_param_list = ent;
+                       }
+               }
+       }
+}
+
+/**
+ * Check if a value parameter is transmitted as a register.
+ * This might happen if the address of an parameter is taken which is
+ * transmitted in registers.
+ *
+ * Note that on some architectures this case must be handled specially
+ * because the place of the backing store is determined by their ABI.
+ *
+ * In the default case we move the entity to the frame type and create
+ * a backing store into the first block.
+ */
+static void fix_address_of_parameter_access(be_abi_irg_t *env, entity *value_param_list) {
+       be_abi_call_t *call = env->call;
+       ir_graph *irg       = env->birg->irg;
+       entity *ent, *next_ent, *new_list;
+       ir_type *frame_tp;
+       DEBUG_ONLY(firm_dbg_module_t *dbg = env->dbg;)
+
+       new_list = NULL;
+       for (ent = value_param_list; ent; ent = next_ent) {
+               int i = get_struct_member_index(get_entity_owner(ent), ent);
+               be_abi_call_arg_t *arg = get_call_arg(call, 0, i);
+
+               next_ent = get_entity_link(ent);
+               if (arg->in_reg) {
+                       DBG((dbg, LEVEL_2, "\targ #%d need backing store\n", i));
+                       set_entity_link(ent, new_list);
+                       new_list = ent;
+               }
+       }
+       if (new_list) {
+               /* ok, change the graph */
+               ir_node *start_bl = get_irg_start_block(irg);
+               ir_node *first_bl = NULL;
+               ir_node *frame, *imem, *nmem, *store, *mem, *args, *args_bl;
+               const ir_edge_t *edge;
+               optimization_state_t state;
+               int offset;
+
+               foreach_block_succ(start_bl, edge) {
+                       ir_node *succ = get_edge_src_irn(edge);
+                       if (start_bl != succ) {
+                               first_bl = succ;
+                               break;
+                       }
+               }
+               assert(first_bl);
+               /* we had already removed critical edges, so the following
+                  assertion should be always true. */
+               assert(get_Block_n_cfgpreds(first_bl) == 1);
+
+               /* now create backing stores */
+               frame = get_irg_frame(irg);
+               imem = get_irg_initial_mem(irg);
+
+               save_optimization_state(&state);
+               set_optimize(0);
+               nmem = new_r_Proj(irg, first_bl, get_irg_start(irg), mode_M, pn_Start_M);
+               restore_optimization_state(&state);
+
+               /* reroute all edges to the new memory source */
+               edges_reroute(imem, nmem, irg);
+
+               store   = NULL;
+               mem     = imem;
+               args    = get_irg_args(irg);
+               args_bl = get_nodes_block(args);
+               for (ent = new_list; ent; ent = get_entity_link(ent)) {
+                       int     i     = get_struct_member_index(get_entity_owner(ent), ent);
+                       ir_type *tp   = get_entity_type(ent);
+                       ir_mode *mode = get_type_mode(tp);
+                       ir_node *addr;
+
+                       /* address for the backing store */
+                       addr = be_new_FrameAddr(env->isa->sp->reg_class, irg, first_bl, frame, ent);
+
+                       if (store)
+                               mem = new_r_Proj(irg, first_bl, store, mode_M, pn_Store_M);
+
+                       /* the backing store itself */
+                       store = new_r_Store(irg, first_bl, mem, addr,
+                                           new_r_Proj(irg, args_bl, args, mode, i));
+               }
+               /* the new memory Proj gets the last Proj from store */
+               set_Proj_pred(nmem, store);
+               set_Proj_proj(nmem, pn_Store_M);
+
+               /* move all entities to the frame type */
+               frame_tp = get_irg_frame_type(irg);
+               offset   = get_type_size_bytes(frame_tp);
+               for (ent = new_list; ent; ent = get_entity_link(ent)) {
+                       ir_type *tp = get_entity_type(ent);
+                       int align = get_type_alignment_bytes(tp);
+
+                       offset += align - 1;
+                       offset &= -align;
+                       set_entity_owner(ent, frame_tp);
+                       add_class_member(frame_tp, ent);
+                       /* must be automatic to set a fixed layout */
+                       set_entity_allocation(ent, allocation_automatic);
+                       set_entity_offset_bytes(ent, offset);
+                       offset += get_type_size_bytes(tp);
+               }
+               set_type_size_bytes(frame_tp, offset);
+       }
+}
+
 /**
  * Modify the irg itself and the frame type.
  */
@@ -1279,14 +1487,12 @@ static void modify_irg(be_abi_irg_t *env)
        ir_graph *irg             = env->birg->irg;
        ir_node *bl               = get_irg_start_block(irg);
        ir_node *end              = get_irg_end_block(irg);
-       ir_node *arg_tuple        = get_irg_args(irg);
        ir_node *no_mem           = get_irg_no_mem(irg);
        ir_node *mem              = get_irg_initial_mem(irg);
-       type *method_type         = get_entity_type(get_irg_entity(irg));
+       ir_type *method_type      = get_entity_type(get_irg_entity(irg));
        pset *dont_save           = pset_new_ptr(8);
-       int n_params              = get_method_n_params(method_type);
-       int max_arg               = 0;
 
+       int n_params;
        int i, j, n;
 
        reg_node_map_t *rm;
@@ -1295,8 +1501,10 @@ static void modify_irg(be_abi_irg_t *env)
        ir_node *barrier;
        ir_node *reg_params_bl;
        ir_node **args;
+       ir_node *arg_tuple;
        const ir_edge_t *edge;
        ir_type *arg_type, *bet_type;
+       lower_frame_sels_env_t ctx;
 
        bitset_t *used_proj_nr;
        DEBUG_ONLY(firm_dbg_module_t *dbg = env->dbg;)
@@ -1304,24 +1512,32 @@ static void modify_irg(be_abi_irg_t *env)
        DBG((dbg, LEVEL_1, "introducing abi on %+F\n", irg));
 
        /* Convert the Sel nodes in the irg to frame load/store/addr nodes. */
-       irg_walk_graph(irg, lower_frame_sels_walker, NULL, env);
+       ctx.env              = env;
+       ctx.value_param_list = NULL;
+       irg_walk_graph(irg, lower_frame_sels_walker, NULL, &ctx);
 
        env->frame = obstack_alloc(&env->obst, sizeof(env->frame[0]));
        env->regs  = pmap_create();
 
-       /* Find the maximum proj number of the argument tuple proj */
-       foreach_out_edge(arg_tuple, edge)  {
-               ir_node *irn = get_edge_src_irn(edge);
-               int nr       = get_Proj_proj(irn);
-               max_arg      = MAX(max_arg, nr);
-       }
-
        used_proj_nr = bitset_alloca(1024);
-       max_arg      = MAX(max_arg + 1, n_params);
-       args         = obstack_alloc(&env->obst, max_arg * sizeof(args[0]));
-       memset(args, 0, max_arg * sizeof(args[0]));
+       n_params     = get_method_n_params(method_type);
+       args         = obstack_alloc(&env->obst, n_params * sizeof(args[0]));
+       memset(args, 0, n_params * sizeof(args[0]));
+
+       /* Check if a value parameter is transmitted as a register.
+        * This might happen if the address of an parameter is taken which is
+        * transmitted in registers.
+        *
+        * Note that on some architectures this case must be handled specially
+        * because the place of the backing store is determined by their ABI.
+        *
+        * In the default case we move the entity to the frame type and create
+        * a backing store into the first block.
+        */
+       fix_address_of_parameter_access(env, ctx.value_param_list);
 
        /* Fill the argument vector */
+       arg_tuple = get_irg_args(irg);
        foreach_out_edge(arg_tuple, edge) {
                ir_node *irn = get_edge_src_irn(edge);
                int nr       = get_Proj_proj(irn);
@@ -1374,10 +1590,12 @@ static void modify_irg(be_abi_irg_t *env)
        for(i = 0, n = pmap_count(env->regs); i < n; ++i) {
                arch_register_t *reg = (void *) rm[i].reg;
                ir_node *arg_proj    = rm[i].irn;
-               ir_node *proj;
                ir_mode *mode        = arg_proj ? get_irn_mode(arg_proj) : reg->reg_class->mode;
                long nr              = i;
                int pos              = BE_OUT_POS((int) nr);
+               int flags            = 0;
+
+               ir_node *proj;
 
                assert(nr >= 0);
                bitset_set(used_proj_nr, nr);
@@ -1391,7 +1609,12 @@ static void modify_irg(be_abi_irg_t *env)
                 * The Proj for that register shall also be ignored during register allocation.
                 */
                if(arch_register_type_is(reg, ignore))
-                       be_node_set_flags(env->reg_params, pos, arch_irn_flags_ignore);
+                       flags |= arch_irn_flags_ignore;
+
+               if(reg == sp)
+                       flags |= arch_irn_flags_modify_sp;
+
+               be_node_set_flags(env->reg_params, pos, flags);
 
                DBG((dbg, LEVEL_2, "\tregister save proj #%d -> reg %s\n", nr, reg->name));
        }
@@ -1403,7 +1626,7 @@ static void modify_irg(be_abi_irg_t *env)
        /* do the stack allocation BEFORE the barrier, or spill code
           might be added before it */
        env->init_sp  = be_abi_reg_map_get(env->regs, sp);
-       env->init_sp = be_new_IncSP(sp, irg, bl, env->init_sp, no_mem, BE_STACK_FRAME_SIZE, be_stack_dir_expand);
+       env->init_sp = be_new_IncSP(sp, irg, bl, env->init_sp, BE_STACK_FRAME_SIZE_EXPAND);
        be_abi_reg_map_set(env->regs, sp, env->init_sp);
 
        barrier = create_barrier(env, bl, &mem, env->regs, 0);
@@ -1416,7 +1639,7 @@ static void modify_irg(be_abi_irg_t *env)
        pset_insert_ptr(env->ignore_regs, fp_reg);
 
        /* Now, introduce stack param nodes for all parameters passed on the stack */
-       for(i = 0; i < max_arg; ++i) {
+       for(i = 0; i < n_params; ++i) {
                ir_node *arg_proj = args[i];
                ir_node *repl     = NULL;
 
@@ -1488,6 +1711,7 @@ be_abi_irg_t *be_abi_introduce(be_irg_t *birg)
 
        pmap_entry *ent;
        ir_node *dummy;
+       optimization_state_t state;
 
        obstack_init(&env->obst);
 
@@ -1501,17 +1725,26 @@ be_abi_irg_t *be_abi_introduce(be_irg_t *birg)
        env->dce_survivor     = new_survive_dce();
        env->birg             = birg;
        env->stack_phis       = pset_new_ptr(16);
+       /* Beware: later we replace this node by the real one, ensure it is not CSE'd
+          to another Unknown or the stack pointer gets used */
+       save_optimization_state(&state);
+       set_optimize(0);
        env->init_sp = dummy  = new_r_Unknown(irg, env->isa->sp->reg_class->mode);
+       restore_optimization_state(&state);
        FIRM_DBG_REGISTER(env->dbg, "firm.be.abi");
 
-       env->cb = env->call->cb->init(env->call, birg->main_env->arch_env, irg);
-
        memcpy(&env->irn_handler, &abi_irn_handler, sizeof(abi_irn_handler));
        env->irn_ops.impl = &abi_irn_ops;
 
        /* Lower all call nodes in the IRG. */
        process_calls(env);
 
+       /*
+               Beware: init backend abi call object after processing calls,
+               otherwise some information might be not yet available.
+       */
+       env->cb = env->call->cb->init(env->call, birg->main_env->arch_env, irg);
+
        /* Process the IRG */
        modify_irg(env);
 
@@ -1530,7 +1763,6 @@ be_abi_irg_t *be_abi_introduce(be_irg_t *birg)
        arch_env_push_irn_handler(env->birg->main_env->arch_env, &env->irn_handler);
 
        env->call->cb->done(env->cb);
-       be_liveness(irg);
        return env;
 }
 
@@ -1565,85 +1797,86 @@ void be_abi_put_ignore_regs(be_abi_irg_t *abi, const arch_register_class_t *cls,
 
 */
 
+struct fix_stack_walker_info {
+       nodeset *nodes;
+       const arch_env_t *aenv;
+};
+
 /**
  * Walker. Collect all stack modifying nodes.
  */
 static void collect_stack_nodes_walker(ir_node *irn, void *data)
 {
-       pset *s = data;
+       struct fix_stack_walker_info *info = data;
 
-       if(be_is_AddSP(irn)     || be_is_IncSP(irn)     || be_is_SetSP(irn))
-               pset_insert_ptr(s, irn);
+       if (is_Block(irn))
+               return;
+
+       if (arch_irn_is(info->aenv, irn, modify_sp)) {
+               assert(get_irn_mode(irn) != mode_M && get_irn_mode(irn) != mode_T);
+               pset_insert_ptr(info->nodes, irn);
+       }
 }
 
-void be_abi_fix_stack_nodes(be_abi_irg_t *env)
+void be_abi_fix_stack_nodes(be_abi_irg_t *env, be_lv_t *lv)
 {
        dom_front_info_t *df;
-       pset *stack_nodes;
+       pset *stack_nodes = pset_new_ptr(16);
+       struct fix_stack_walker_info info;
+
+       info.nodes = stack_nodes;
+       info.aenv  = env->birg->main_env->arch_env;
 
        /* We need dominance frontiers for fix up */
        df = be_compute_dominance_frontiers(env->birg->irg);
-       stack_nodes = pset_new_ptr(16);
+       irg_walk_graph(env->birg->irg, collect_stack_nodes_walker, NULL, &info);
        pset_insert_ptr(stack_nodes, env->init_sp);
-       irg_walk_graph(env->birg->irg, collect_stack_nodes_walker, NULL, stack_nodes);
-       be_ssa_constr_set_phis(df, stack_nodes, env->stack_phis);
+       be_ssa_constr_set_phis(df, lv, stack_nodes, env->stack_phis);
        del_pset(stack_nodes);
 
-       /* Liveness could have changed due to Phi nodes. */
-       be_liveness(env->birg->irg);
-
        /* free these dominance frontiers */
        be_free_dominance_frontiers(df);
 }
 
-/**
- * Translates a direction of an IncSP node (either be_stack_dir_shrink, or ...expand)
- * into -1 or 1, respectively.
- * @param irn The node.
- * @return 1, if the direction of the IncSP was along, -1 if against.
- */
-static int get_dir(ir_node *irn)
-{
-       return 1 - 2 * (be_get_IncSP_direction(irn) == be_stack_dir_shrink);
-}
-
 static int process_stack_bias(be_abi_irg_t *env, ir_node *bl, int bias)
 {
-       const arch_env_t *aenv = env->birg->main_env->arch_env;
+       const arch_env_t *arch_env = env->birg->main_env->arch_env;
        int omit_fp            = env->call->flags.bits.try_omit_fp;
        ir_node *irn;
 
        sched_foreach(bl, irn) {
 
                /*
-                       If the node modifies the stack pointer by a constant offset,
-                       record that in the bias.
-               */
-               if(be_is_IncSP(irn)) {
-                       int ofs = be_get_IncSP_offset(irn);
-                       int dir = get_dir(irn);
-
-                       if(ofs == BE_STACK_FRAME_SIZE) {
-                               ofs = get_type_size_bytes(get_irg_frame_type(env->birg->irg));
-                               be_set_IncSP_offset(irn, ofs);
-                       }
-
-                       if(omit_fp)
-                               bias += dir * ofs;
+                  Check, if the node relates to an entity on the stack frame.
+                  If so, set the true offset (including the bias) for that
+                  node.
+                */
+               entity *ent = arch_get_frame_entity(arch_env, irn);
+               if(ent) {
+                       int offset = get_stack_entity_offset(env->frame, ent, bias);
+                       arch_set_frame_offset(arch_env, irn, offset);
+                       DBG((env->dbg, LEVEL_2, "%F has offset %d (including bias %d)\n", ent, offset, bias));
                }
 
                /*
-                       Else check, if the node relates to an entity on the stack frame.
-                       If so, set the true offset (including the bias) for that
-                       node.
-               */
-               else {
-                       entity *ent = arch_get_frame_entity(aenv, irn);
-                       if(ent) {
-                               int offset = get_stack_entity_offset(env->frame, ent, bias);
-                               arch_set_frame_offset(aenv, irn, offset);
-                               DBG((env->dbg, LEVEL_2, "%F has offset %d\n", ent, offset));
+                  If the node modifies the stack pointer by a constant offset,
+                  record that in the bias.
+                */
+               if(arch_irn_is(arch_env, irn, modify_sp)) {
+                       int ofs = arch_get_sp_bias(arch_env, irn);
+
+                       if(be_is_IncSP(irn)) {
+                               if(ofs == BE_STACK_FRAME_SIZE_EXPAND) {
+                                       ofs = get_type_size_bytes(get_irg_frame_type(env->birg->irg));
+                                       be_set_IncSP_offset(irn, ofs);
+                               } else if(ofs == BE_STACK_FRAME_SIZE_SHRINK) {
+                                       ofs = - get_type_size_bytes(get_irg_frame_type(env->birg->irg));
+                                       be_set_IncSP_offset(irn, ofs);
+                               }
                        }
+
+                       if(omit_fp)
+                               bias += ofs;
                }
        }
 
@@ -1656,6 +1889,7 @@ static int process_stack_bias(be_abi_irg_t *env, ir_node *bl, int bias)
 struct bias_walk {
        be_abi_irg_t *env;     /**< The ABI irg environment. */
        int start_block_bias;  /**< The bias at the end of the start block. */
+       ir_node *start_block;  /**< The start block of the current graph. */
 };
 
 /**
@@ -1663,8 +1897,8 @@ struct bias_walk {
  */
 static void stack_bias_walker(ir_node *bl, void *data)
 {
-       if(bl != get_irg_start_block(get_irn_irg(bl))) {
-               struct bias_walk *bw = data;
+       struct bias_walk *bw = data;
+       if (bl != bw->start_block) {
                process_stack_bias(bw->env, bl, bw->start_block_bias);
        }
 }
@@ -1675,13 +1909,14 @@ void be_abi_fix_stack_bias(be_abi_irg_t *env)
        struct bias_walk bw;
 
        stack_frame_compute_initial_offset(env->frame);
-       // stack_frame_dump(stdout, env->frame);
+       // stack_layout_dump(stdout, env->frame);
 
        /* Determine the stack bias at the end of the start block. */
        bw.start_block_bias = process_stack_bias(env, get_irg_start_block(irg), 0);
 
        /* fix the bias is all other blocks */
        bw.env = env;
+       bw.start_block = get_irg_start_block(irg);
        irg_block_walk_graph(irg, stack_bias_walker, NULL, &bw);
 }
 
@@ -1764,7 +1999,7 @@ static arch_irn_class_t abi_classify(const void *_self, const ir_node *irn)
 
 static arch_irn_flags_t abi_get_flags(const void *_self, const ir_node *irn)
 {
-       return arch_irn_flags_ignore;
+       return arch_irn_flags_ignore | arch_irn_flags_modify_sp;
 }
 
 static entity *abi_get_frame_entity(const void *_self, const ir_node *irn)
@@ -1772,10 +2007,19 @@ static entity *abi_get_frame_entity(const void *_self, const ir_node *irn)
        return NULL;
 }
 
-static void abi_set_stack_bias(const void *_self, ir_node *irn, int bias)
+static void abi_set_frame_entity(const void *_self, ir_node *irn, entity *ent)
+{
+}
+
+static void abi_set_frame_offset(const void *_self, ir_node *irn, int bias)
 {
 }
 
+static int abi_get_sp_bias(const void *self, const ir_node *irn)
+{
+       return 0;
+}
+
 static const arch_irn_ops_if_t abi_irn_ops = {
        abi_get_irn_reg_req,
        abi_set_irn_reg,
@@ -1783,7 +2027,13 @@ static const arch_irn_ops_if_t abi_irn_ops = {
        abi_classify,
        abi_get_flags,
        abi_get_frame_entity,
-       abi_set_stack_bias
+       abi_set_frame_entity,
+       abi_set_frame_offset,
+       abi_get_sp_bias,
+       NULL,    /* get_inverse             */
+       NULL,    /* get_op_estimated_cost   */
+       NULL,    /* possible_memory_operand */
+       NULL,    /* perform_memory_operand  */
 };
 
 static const arch_irn_handler_t abi_irn_handler = {