[llvm-commits] [PATCH] Fix default stack alignment on i386

Chandler Carruth chandlerc at google.com
Tue Jan 10 09:34:04 PST 2012


On Tue, Jan 10, 2012 at 8:51 AM, Joerg Sonnenberger <joerg at britannica.bec.de
> wrote:

> Hi all,
> the SYSV ABI says the stack alignment of i386 is 32bit, not 64bit.
> The attached patch fixes the default. Comments?
>
> This is PR 11725.
>

Stack alignment on 32-bit x86 has plagued us with issues before, but I
think this is still safe... However, I'd really like Nick to look at it, as
he is likely to remember details that I don't.

Just to understand correctly, this doesn't change the requested 16-byte
alignment of stacks on *linux* 32-bit x86? While I understand that that is
a divergence from the SYSV ABI, it is an important and ubiquitous one on
Linux.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20120110/0e281dce/attachment.html>


More information about the llvm-commits mailing list