[llvm-commits] [PATCH] va_arg alignment type

David Meyer pdox at google.com
Wed Oct 13 12:02:55 PDT 2010


Rafael,

The default on X86-32 is 4-byte alignment for doubles and long long (both in
structs and on the stack).
(gcc has the flag -malign-double to increase alignment to 8-bytes).

- David M

On Wed, Oct 13, 2010 at 11:47 AM, Rafael Espindola <espindola at google.com>wrote:

> > Could someone with more general target knowledge comment on the
> correctness
> > of this change?
>
> This looks right. It probably fixes doubles on x86 too. I will give it
> a try and commit if everything is ok.
>
> > Thank you,
> >   David Meyer
> > _______________________________________________
> > llvm-commits mailing list
> > llvm-commits at cs.uiuc.edu
> > http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits
> >
> >
>
> Cheers,
> --
> Rafael Ávila de Espíndola
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20101013/3bff1f87/attachment.html>


More information about the llvm-commits mailing list