sparc: Keep stack pointer 8 bytes aligned when generating code for MemPerm.

diff --git a/ir/be/sparc/sparc_emitter.c b/ir/be/sparc/sparc_emitter.c
index 413b59a..d958c5e 100644 (file)
--- a/ir/be/sparc/sparc_emitter.c
+++ b/ir/be/sparc/sparc_emitter.c
@@ -572,11 +572,20 @@ static void emit_be_Perm(const ir_node *irn)
        be_emit_finish_line_gas(irn);
 }
 
+/* The stack pointer must always be 8 bytes aligned, so get the next bigger
+ * integer that's evenly divisible by 8. */
+static unsigned get_aligned_sp_change(int memperm_arity)
+{
+       const unsigned bytes = ((unsigned) memperm_arity) * 4;
+       return (bytes + 7) & ~7U;
+}
+
 static void emit_be_MemPerm(const ir_node *node)
 {
-       int i;
-       int memperm_arity;
-       int sp_change = 0;
+       int      i;
+       int      memperm_arity;
+       unsigned aligned_sp_change;
+       int      sp_change = 0;
        ir_graph          *irg    = get_irn_irg(node);
        be_stack_layout_t *layout = be_get_irg_stack_layout(irg);
 
@@ -591,7 +600,8 @@ static void emit_be_MemPerm(const ir_node *node)
        if (memperm_arity > 8)
                panic("memperm with more than 8 inputs not supported yet");
 
-       be_emit_irprintf("\tsub %%sp, %d, %%sp", memperm_arity*4);
+       aligned_sp_change = get_aligned_sp_change(memperm_arity);
+       be_emit_irprintf("\tsub %%sp, %u, %%sp", aligned_sp_change);
        be_emit_finish_line_gas(node);
 
        for (i = 0; i < memperm_arity; ++i) {
@@ -622,7 +632,7 @@ static void emit_be_MemPerm(const ir_node *node)
                be_emit_finish_line_gas(node);
        }
 
-       be_emit_irprintf("\tadd %%sp, %d, %%sp", memperm_arity*4);
+       be_emit_irprintf("\tadd %%sp, %u, %%sp", aligned_sp_change);
        be_emit_finish_line_gas(node);
 
        assert(sp_change == 0);