X-Git-Url: http://nsz.repo.hu/git/?a=blobdiff_plain;f=ir%2Fbe%2Fbespill.c;h=65f87e3dc61792690049aebd453f682a336c929b;hb=76f6a8961f9cd4fe5e4f9fda8b600e52cc9fe9da;hp=016a2d776de04a402bdb83afe03d0bfea7d3c99a;hpb=3767f52182607e5de9f0ed7e48d88132317c12d8;p=libfirm

diff --git a/ir/be/bespill.c b/ir/be/bespill.c
index 016a2d776..65f87e3dc 100644
--- a/ir/be/bespill.c
+++ b/ir/be/bespill.c
@@ -20,6 +20,7 @@
 #include "debug.h"
 #include "irgwalk.h"
 #include "array.h"
+#include "pdeq.h"
 
 #include "belive_t.h"
 #include "besched_t.h"
@@ -27,7 +28,7 @@
 #include "benode_t.h"
 #include "bechordal_t.h"
 
-#undef REMAT
+#define REMAT
 /* This enables re-computation of values. Current state: Unfinished and buggy. */
 #undef BUGGY_REMAT
 
@@ -57,41 +58,70 @@ struct _spill_env_t {
 	set *spill_ctxs;
 	set *spills;				/**< all spill_info_t's, which must be placed */
 	pset *mem_phis;				/**< set of all special spilled phis. allocated and freed separately */
-	decide_irn_t is_mem_phi;	/**< callback func to decide if a phi needs special spilling */
+	decide_irn_t is_spilled_phi;/**< callback func to decide if a phi needs special spilling */
 	void *data;					/**< data passed to all callbacks */
 	DEBUG_ONLY(firm_dbg_module_t *dbg;)
 };
 
+/* associated Phi -> Spill*/
+typedef struct _phi_spill_assoc_t {
+	ir_node *phi;
+	ir_node *spill;
+} phi_spill_assoc_t;
+
+/**
+ * Compare two Phi->Spill associations.
+ */
+static int cmp_phi_spill_assoc(const void *a, const void *b, size_t n) {
+	const phi_spill_assoc_t *p1 = a;
+	const phi_spill_assoc_t *p2 = b;
+	return p1->phi != p2->phi;
+}
+
+/**
+ * compare two spill contexts.
+ */
 static int cmp_spillctx(const void *a, const void *b, size_t n) {
 	const spill_ctx_t *p = a;
 	const spill_ctx_t *q = b;
-	return !(p->user == q->user && p->spilled == q->spilled);
+	return p->user != q->user || p->spilled != q->spilled;
 }
 
+/**
+ * Compare two spill infos.
+ */
 static int cmp_spillinfo(const void *x, const void *y, size_t size) {
 	const spill_info_t *xx = x;
 	const spill_info_t *yy = y;
-	return ! (xx->spilled_node == yy->spilled_node);
+	return xx->spilled_node != yy->spilled_node;
 }
 
 DEBUG_ONLY(
+/* Sets the debug module of a spill environment. */
 void be_set_spill_env_dbg_module(spill_env_t *env, firm_dbg_module_t *dbg) {
 	env->dbg = dbg;
 }
-);
-
-spill_env_t *be_new_spill_env(const be_chordal_env_t *chordal_env, decide_irn_t is_mem_phi, void *data) {
-	spill_env_t *env = xmalloc(sizeof(env[0]));
-	env->spill_ctxs  = new_set(cmp_spillctx, 1024);
-	env->spills      = new_set(cmp_spillinfo, 1024);
-	env->cls         = chordal_env->cls;
-	env->is_mem_phi  = is_mem_phi;
-	env->data        = data;
-	env->chordal_env = chordal_env;
+)
+
+/* Creates a new spill environment. */
+spill_env_t *be_new_spill_env(const be_chordal_env_t *chordal_env, decide_irn_t is_spilled_phi, void *data) {
+	spill_env_t *env	= xmalloc(sizeof(env[0]));
+	env->spill_ctxs		= new_set(cmp_spillctx, 1024);
+	env->spills			= new_set(cmp_spillinfo, 1024);
+	env->cls			= chordal_env->cls;
+	env->is_spilled_phi	= is_spilled_phi;
+	env->data			= data;
+	env->chordal_env	= chordal_env;
 	obstack_init(&env->obst);
 	return env;
 }
 
+void be_set_is_spilled_phi(spill_env_t *env, decide_irn_t is_spilled_phi, void *data) {
+	env->is_spilled_phi	= is_spilled_phi;
+	env->data			= data;
+}
+
+/* Deletes a spill environment. */
 void be_delete_spill_env(spill_env_t *senv) {
 	del_set(senv->spill_ctxs);
 	del_set(senv->spills);
@@ -99,6 +129,15 @@ void be_delete_spill_env(spill_env_t *senv) {
 	free(senv);
 }
 
+/**
+ * Returns a spill context. If the context did not exists, create one.
+ *
+ * @param sc        the set containing all spill contexts
+ * @param to_spill  the node that should be spilled
+ * @param ctx_irn   an user of the spilled node
+ *
+ * @return a spill context.
+ */
 static spill_ctx_t *be_get_spill_ctx(set *sc, ir_node *to_spill, ir_node *ctx_irn) {
 	spill_ctx_t templ;
 
@@ -109,86 +148,154 @@ static spill_ctx_t *be_get_spill_ctx(set *sc, ir_node *to_spill, ir_node *ctx_ir
 	return set_insert(sc, &templ, sizeof(templ), HASH_COMBINE(HASH_PTR(to_spill), HASH_PTR(ctx_irn)));
 }
 
+/**
+ * Creates a spill.
+ *
+ * @param senv      the spill environment
+ * @param irn       the node that should be spilled
+ * @param ctx_irn   an user of the spilled node
+ *
+ * @return a be_Spill node
+ */
 static ir_node *be_spill_irn(spill_env_t *senv, ir_node *irn, ir_node *ctx_irn) {
 	spill_ctx_t *ctx;
+	const be_main_env_t *env = senv->chordal_env->birg->main_env;
 	DBG((senv->dbg, LEVEL_1, "%+F in ctx %+F\n", irn, ctx_irn));
 
+	// Has the value already been spilled?
 	ctx = be_get_spill_ctx(senv->spill_ctxs, irn, ctx_irn);
-	if(!ctx->spill) {
-		const be_main_env_t *env = senv->chordal_env->birg->main_env;
-		ctx->spill = be_spill(env->arch_env, irn, ctx_irn);
+	if(ctx->spill)
+		return ctx->spill;
+
+	/* Trying to spill an already spilled value, no need for a new spill
+	 * node then, we can simply connect to the same one for this reload
+	 */
+	if(be_is_Reload(irn)) {
+		return get_irn_n(irn, be_pos_Reload_mem);
 	}
 
+	ctx->spill = be_spill(env->arch_env, irn, ctx_irn);
 	return ctx->spill;
 }
 
 /**
- * If the first usage of a phi result would be out of memory
+ * If the first usage of a Phi result would be out of memory
  * there is no sense in allocating a register for it.
  * Thus we spill it and all its operands to the same spill slot.
  * Therefore the phi/dataB becomes a phi/Memory
+ *
+ * @param senv      the spill environment
+ * @param phi       the Phi node that should be spilled
+ * @param ctx_irn   an user of the spilled node
+ *
+ * @return a be_Spill node
  */
-static ir_node *be_spill_phi(spill_env_t *senv, ir_node *phi, ir_node *ctx_irn) {
-	int i, n = get_irn_arity(phi);
-	ir_node **ins, *bl = get_nodes_block(phi);
-	ir_graph *irg = senv->chordal_env->irg;
+static ir_node *be_spill_phi(spill_env_t *senv, ir_node *phi, ir_node *ctx_irn, unsigned visited_nr, set *already_visited_phis) {
+	int         i, n      = get_irn_arity(phi);
+	ir_graph    *irg      = senv->chordal_env->irg;
+	ir_node     *bl       = get_nodes_block(phi);
+	ir_node     **ins, *phi_spill;
+	phi_spill_assoc_t key;
 	spill_ctx_t *ctx;
 
 	assert(is_Phi(phi));
 	DBG((senv->dbg, LEVEL_1, "%+F in ctx %+F\n", phi, ctx_irn));
 
+	/* build a new PhiM */
+	NEW_ARR_A(ir_node *, ins, n);
+	for (i = 0; i < n; ++i) {
+		ins[i]    = new_r_Bad(irg);
+	}
+	phi_spill = new_r_Phi(senv->chordal_env->irg, bl, n, ins, mode_M);
+	key.phi   = phi;
+	key.spill = phi_spill;
+	set_insert(already_visited_phis, &key, sizeof(key), HASH_PTR(phi));
+
 	/* search an existing spill for this context */
 	ctx = be_get_spill_ctx(senv->spill_ctxs, phi, ctx_irn);
 
 	/* if not found spill the phi */
-	if(!ctx->spill) {
-		/* build a new PhiM with dummy in-array */
-    NEW_ARR_A(ir_node *, ins, n);
-		for(i=0; i<n; ++i)
-			ins[i] = new_r_Unknown(irg, mode_M);
-		ctx->spill = new_r_Phi(senv->chordal_env->irg, bl, n, ins, mode_M);
-
-		/* re-wire the phiM */
-		for(i=0; i<n; ++i) {
+	if (! ctx->spill) {
+        set_irn_visited(phi, visited_nr);
+
+		/* collect all arguments of the phi */
+		for (i = 0; i < n; ++i) {
 			ir_node *arg = get_irn_n(phi, i);
 			ir_node *sub_res;
-
-			if(is_Phi(arg) && pset_find_ptr(senv->mem_phis, arg))
-				sub_res = be_spill_phi(senv, arg, ctx_irn);
+			phi_spill_assoc_t *entry;
+
+			if(is_Phi(arg) && pset_find_ptr(senv->mem_phis, arg)) {
+				if (get_irn_visited(arg) < visited_nr)
+					sub_res = be_spill_phi(senv, arg, ctx_irn, visited_nr, already_visited_phis);
+				else {
+					/* we already visited the argument phi: get it's spill */
+					key.phi   = arg;
+					key.spill = NULL;
+					entry     = set_find(already_visited_phis, &key, sizeof(key), HASH_PTR(arg));
+					assert(entry && "argument phi already visited, but no spill found?!?");
+					sub_res   = entry->spill;
+					assert(sub_res && "spill missing?!?");
+				}
+			}
 			else
 				sub_res = be_spill_irn(senv, arg, ctx_irn);
 
-			set_irn_n(ctx->spill, i, sub_res);
+			set_irn_n(phi_spill, i, sub_res);
 		}
+
+		ctx->spill = phi_spill;
 	}
 	return ctx->spill;
 }
 
-static ir_node *be_spill_node(spill_env_t *senv, ir_node *to_spill) {
+/**
+ * Spill a node.
+ *
+ * @param senv      the spill environment
+ * @param to_spill  the node that should be spilled
+ *
+ * @return a be_Spill node
+ */
+static ir_node *be_spill_node(spill_env_t *senv, ir_node *to_spill, unsigned visited_nr) {
+	ir_graph *irg                  = get_irn_irg(to_spill);
+	int      save_optimize         = get_optimize();
+	int      save_normalize        = get_opt_normalize();
+	set      *already_visited_phis = new_set(cmp_phi_spill_assoc, 10);
 	ir_node *res;
+
+	/*
+	 * Disable optimization so that the phi functions do not
+	 * disappear.
+	 */
+	set_optimize(0);
+	set_opt_normalize(0);
+
 	if (pset_find_ptr(senv->mem_phis, to_spill))
-		res = be_spill_phi(senv, to_spill, to_spill);
+		res = be_spill_phi(senv, to_spill, to_spill, visited_nr, already_visited_phis);
 	else
 		res = be_spill_irn(senv, to_spill, to_spill);
 
-	return res;
-}
+	del_set(already_visited_phis);
 
-static void phi_walker(ir_node *irn, void *env) {
-	spill_env_t *senv = env;
-	const arch_env_t *arch = senv->chordal_env->birg->main_env->arch_env;
+	/* reset the optimizations */
+	set_optimize(save_optimize);
+	set_opt_normalize(save_normalize);
 
-	if (is_Phi(irn) && arch_irn_has_reg_class(arch, irn, 0, senv->cls)
-			&& senv->is_mem_phi(irn, senv->data)) {
-		DBG((senv->dbg, LEVEL_1, "  %+F\n", irn));
-		pset_insert_ptr(senv->mem_phis, irn);
-	}
+	return res;
 }
 
 #ifdef REMAT
 
 #ifdef BUGGY_REMAT
 
+/**
+ * Check if a spilled node could be rematerialized.
+ *
+ * @param senv      the spill environment
+ * @param spill     the Spill node
+ * @param spilled   the node that was spilled
+ * @param reloader  a irn that requires a reload
+ */
 static int check_remat_conditions(spill_env_t *senv, ir_node *spill, ir_node *spilled, ir_node *reloader) {
 	int pos, max;
 
@@ -243,6 +350,14 @@ is_alive:	;
 
 #else /* BUGGY_REMAT */
 
+/**
+ * A very simple rematerialization checker.
+ *
+ * @param senv      the spill environment
+ * @param spill     the Spill node
+ * @param spilled   the node that was spilled
+ * @param reloader  a irn that requires a reload
+ */
 static int check_remat_conditions(spill_env_t *senv, ir_node *spill, ir_node *spilled, ir_node *reloader) {
 	const arch_env_t *aenv = senv->chordal_env->birg->main_env->arch_env;
 
@@ -253,6 +368,15 @@ static int check_remat_conditions(spill_env_t *senv, ir_node *spill, ir_node *sp
 
 #endif /* BUGGY_REMAT */
 
+#endif /* REMAT */
+
+/**
+ * Re-materialize a node.
+ *
+ * @param senv      the spill environment
+ * @param spilled   the node that was spilled
+ * @param reloader  a irn that requires a reload
+ */
 static ir_node *do_remat(spill_env_t *senv, ir_node *spilled, ir_node *reloader) {
 	ir_node *res;
 	ir_node *bl = (is_Block(reloader)) ? reloader : get_nodes_block(reloader);
@@ -262,7 +386,7 @@ static ir_node *do_remat(spill_env_t *senv, ir_node *spilled, ir_node *reloader)
 		get_irn_op(spilled),
 		get_irn_mode(spilled),
 		get_irn_arity(spilled),
-		get_irn_in(spilled));
+		get_irn_in(spilled) + 1);
 	copy_node_attr(spilled, res);
 
 	DBG((senv->dbg, LEVEL_1, "Insert remat %+F before reloader %+F\n", res, reloader));
@@ -278,11 +402,27 @@ static ir_node *do_remat(spill_env_t *senv, ir_node *spilled, ir_node *reloader)
 	return res;
 }
 
-#endif
+/**
+ * Walker: fills the mem_phis set by evaluating Phi nodes
+ * using the is_spilled_phi() callback.
+ */
+static void phi_walker(ir_node *irn, void *env) {
+	spill_env_t *senv = env;
+
+	if (is_Phi(irn)) {
+		const arch_env_t *arch = senv->chordal_env->birg->main_env->arch_env;
+		if (arch_irn_has_reg_class(arch, irn, 0, senv->cls) &&
+			senv->is_spilled_phi(irn, senv->data)) {
+			DBG((senv->dbg, LEVEL_1, "  %+F\n", irn));
+			pset_insert_ptr(senv->mem_phis, irn);
+		}
+	}
+}
 
-void be_insert_spills_reloads(spill_env_t *senv, pset *reload_set) {
+void be_insert_spills_reloads(spill_env_t *senv) {
 	const arch_env_t *aenv = senv->chordal_env->birg->main_env->arch_env;
 	ir_graph *irg          = senv->chordal_env->irg;
+	unsigned visited_nr;
 	ir_node *irn;
 	spill_info_t *si;
 
@@ -307,11 +447,15 @@ void be_insert_spills_reloads(spill_env_t *senv, pset *reload_set) {
 		}
 	}
 
+	visited_nr = get_irg_visited(irg) + 1;
+	set_irg_visited(irg, visited_nr);
+
 	/* process each spilled node */
 	DBG((senv->dbg, LEVEL_1, "Insert spills and reloads:\n"));
 	for(si = set_first(senv->spills); si; si = set_next(senv->spills)) {
 		reloader_t *rld;
 		ir_mode *mode = get_irn_mode(si->spilled_node);
+		//ir_node *value;
 		pset *values = pset_new_ptr(16);
 
 		/* go through all reloads for this spill */
@@ -319,11 +463,13 @@ void be_insert_spills_reloads(spill_env_t *senv, pset *reload_set) {
 			ir_node *new_val;
 
 			/* the spill for this reloader */
-			ir_node *spill   = be_spill_node(senv, si->spilled_node);
+			ir_node *spill   = be_spill_node(senv, si->spilled_node, visited_nr);
 
 #ifdef REMAT
-			if (check_remat_conditions(senv, spill, si->spilled_node, rld->reloader))
+			if (check_remat_conditions(senv, spill, si->spilled_node, rld->reloader)) {
 				new_val = do_remat(senv, si->spilled_node, rld->reloader);
+				//pdeq_putl(possibly_dead, spill);
+			}
 			else
 #endif
 				/* do a reload */
@@ -331,8 +477,6 @@ void be_insert_spills_reloads(spill_env_t *senv, pset *reload_set) {
 
 			DBG((senv->dbg, LEVEL_1, " %+F of %+F before %+F\n", new_val, si->spilled_node, rld->reloader));
 			pset_insert_ptr(values, new_val);
-			if(reload_set)
-				pset_insert_ptr(reload_set, new_val);
 		}
 
 		/* introduce copies, rewire the uses */
@@ -343,14 +487,11 @@ void be_insert_spills_reloads(spill_env_t *senv, pset *reload_set) {
 		del_pset(values);
 	}
 
-	for(irn = pset_first(senv->mem_phis); irn; irn = pset_next(senv->mem_phis)) {
-		int i, n;
-		for(i = 0, n = get_irn_arity(irn); i < n; ++i)
-			set_irn_n(irn, i, new_r_Bad(senv->chordal_env->irg));
-		sched_remove(irn);
-	}
-
 	del_pset(senv->mem_phis);
+
+	// reloads are placed now, but we might reuse the spill environment for further spilling decisions
+	del_set(senv->spills);
+	senv->spills = new_set(cmp_spillinfo, 1024);
 }
 
 void be_add_reload(spill_env_t *senv, ir_node *to_spill, ir_node *before) {
@@ -395,6 +536,7 @@ typedef struct _ss_env_t {
 	DEBUG_ONLY(firm_dbg_module_t *dbg;)
 } ss_env_t;
 
+
 /**
  * Walker: compute the spill slots
  */
@@ -413,9 +555,7 @@ static void compute_spill_slots_walker(ir_node *spill, void *env) {
 
 	if (!entry) {
 		struct _arch_env_t *arch_env     = ssenv->cenv->birg->main_env->arch_env;
-		ir_node *spilled                 = get_irn_n(spill, be_pos_Spill_val);
-		const arch_register_t *reg       = arch_get_irn_register(arch_env, spilled);
-		const arch_register_class_t *cls = arch_register_get_class(reg);
+		const arch_register_class_t *cls = arch_get_irn_reg_class(arch_env, spill, be_pos_Spill_val);
 		ir_mode *largest_mode            = arch_register_class_mode(cls);
 
 		/* this is a new spill context */
@@ -426,24 +566,36 @@ static void compute_spill_slots_walker(ir_node *spill, void *env) {
 		ss->align        = arch_isa_get_reg_class_alignment(arch_env->isa, cls);
 		pmap_insert(ssenv->slots, ctx, ss);
 	} else {
-		ir_node *irn;
 		/* values with the same spill_ctx must go into the same spill slot */
 		ss = entry->value;
-		assert(ss->size == (unsigned)get_mode_size_bytes(get_irn_mode(get_irn_n(spill, be_pos_Spill_val))) && "Different sizes for the same spill slot are not allowed yet.");
-		for (irn = pset_first(ss->members); irn; irn = pset_next(ss->members)) {
-			/* use values_interfere here, because it uses the dominance check,
-			   which does work for values in memory */
-			assert(!values_interfere(spill, irn) && "Spills for the same spill slot must not interfere!");
+
+#ifndef NDEBUG
+		/* ugly mega assert :-) */
+		{
+			ir_node *irn;
+			struct _arch_env_t *arch_env     = ssenv->cenv->birg->main_env->arch_env;
+			const arch_register_class_t *cls = arch_get_irn_reg_class(arch_env, spill, be_pos_Spill_val);
+			int size = get_mode_size_bytes(arch_register_class_mode(cls));
+			assert((int) ss->size == size && "Different sizes for the same spill slot are not allowed.");
+			for (irn = pset_first(ss->members); irn; irn = pset_next(ss->members)) {
+				/* use values_interfere here, because it uses the dominance check,
+					 which does work for values in memory */
+				assert(!values_interfere(spill, irn) && "Spills for the same spill slot must not interfere!");
+			}
 		}
+#endif /* NDEBUG */
 	}
 
 	pset_insert_ptr(ss->members, spill);
 }
 
+/**
+ * qsort compare function, sort spill slots by size.
+ */
 static int ss_sorter(const void *v1, const void *v2) {
-	const spill_slot_t *ss1 = v1;
-	const spill_slot_t *ss2 = v2;
-	return ((int) ss2->size) - ((int) ss1->size);
+	const spill_slot_t **ss1 = (const spill_slot_t **)v1;
+	const spill_slot_t **ss2 = (const spill_slot_t **)v2;
+	return ((int) (*ss2)->size) - ((int) (*ss1)->size);
 }
 
 
@@ -458,7 +610,7 @@ static int ss_sorter(const void *v1, const void *v2) {
  *
  * @return An array of spill slots @p ass in specific order
  **/
-static void optimize_slots(ss_env_t *ssenv, int size, spill_slot_t **ass) {
+static void optimize_slots(ss_env_t *ssenv, int size, spill_slot_t *ass[]) {
 	int i, o, used_slots;
 	pmap_entry *entr;
 
@@ -530,24 +682,41 @@ interf_detected: /*nothing*/ ;
 /**
  * Returns a spill type for a mode. Keep them in a map to reduce
  * the number of types.
+ *
+ * @param types  a map containing all created types
+ * @param ss     the spill slot
+ *
+ * Note that type types should are identical for every mode.
+ * This rule might break if two different register classes return the same
+ * mode but different alignments.
  */
-static ir_type *get_spill_type(pmap *types, ir_mode *mode) {
-  pmap_entry *e = pmap_find(types, mode);
+static ir_type *get_spill_type(pmap *types, spill_slot_t *ss) {
+  pmap_entry *e = pmap_find(types, ss->largest_mode);
   ir_type *res;
 
   if (! e) {
 		char buf[64];
-    snprintf(buf, sizeof(buf), "spill_slot_type_%s", get_mode_name(mode));
-    res = new_type_primitive(new_id_from_str(buf), mode);
-    pmap_insert(types, mode, res);
+    snprintf(buf, sizeof(buf), "spill_slot_type_%s", get_mode_name(ss->largest_mode));
+    res = new_type_primitive(new_id_from_str(buf), ss->largest_mode);
+		set_type_alignment_bytes(res, ss->align);
+    pmap_insert(types, ss->largest_mode, res);
   }
-  else
+  else {
     res = e->value;
+		assert(get_type_alignment_bytes(res) == (int)ss->align);
+	}
   return res;
 }
 
-static void assign_entities(ss_env_t *ssenv, int n, spill_slot_t **ss) {
-	int i, offset;
+/**
+ * Create spill slot entities on the frame type.
+ *
+ * @param ssenv   the spill environment
+ * @param n       number of spill slots
+ * @param ss      array of spill slots
+ */
+static void assign_entities(ss_env_t *ssenv, int n_slots, spill_slot_t *ss[]) {
+	int i, offset, frame_align;
 	ir_type *frame = get_irg_frame_type(ssenv->cenv->irg);
 
 	/* aligning by increasing frame size */
@@ -556,7 +725,7 @@ static void assign_entities(ss_env_t *ssenv, int n, spill_slot_t **ss) {
 	set_type_size_bytes(frame, -1);
 
 	/* create entities and assign offsets according to size and alignment*/
-	for (i=0; i<n; ++i) {
+	for (i = 0; i < n_slots; ++i) {
 		char buf[64];
 		ident *name;
 		entity *spill_ent;
@@ -566,21 +735,22 @@ static void assign_entities(ss_env_t *ssenv, int n, spill_slot_t **ss) {
 		snprintf(buf, sizeof(buf), "spill_slot_%d", i);
 		name = new_id_from_str(buf);
 
-		spill_ent = new_entity(frame, name, get_spill_type(ssenv->types, ss[i]->largest_mode));
+		spill_ent = new_entity(frame, name, get_spill_type(ssenv->types, ss[i]));
 
 		/* align */
 		offset = round_up2(offset, ss[i]->align);
 		/* set */
 		set_entity_offset_bytes(spill_ent, offset);
 		/* next possible offset */
-		offset += ss[i]->size;
+		offset += round_up2(ss[i]->size, ss[i]->align);
 
 		pset_foreach(ss[i]->members, irn)
 			be_set_Spill_entity(irn, spill_ent);
 	}
 
 	/* set final size of stack frame */
-	set_type_size_bytes(frame, offset);
+	frame_align = get_type_alignment_bytes(frame);
+	set_type_size_bytes(frame, round_up2(offset, frame_align));
 }
 
 void be_compute_spill_offsets(be_chordal_env_t *cenv) {