[llvm-commits] [PATCH] va_arg alignment type

David Meyer pdox at google.com
Fri Oct 15 07:32:28 PDT 2010


Let's cancel this patch...
Some changes to the data layout may be needed before this makes sense.

- David M

On Thu, Oct 14, 2010 at 6:27 PM, Rafael Espindola <espindola at google.com>wrote:

> On 14 October 2010 13:21, David Meyer <pdox at google.com> wrote:
> > Rafael,
> > What happens if you add s128:128:128 ?
>
> 4 cases:
>
> * No patch + s64:   align
> * No patch + s128: identical to the previous one (the 's' is ignored)
> * Patch + s64: alignment avoided (the 's' is used instead of the type
> alignment)
> * Patch + s128: Both the <2x i64> and the i32 are aligned (again the
> 's' is used instead of the type alignment (even for the i32))
>
> > - David M
>
> Cheers,
> --
> Rafael Ávila de Espíndola
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20101015/7a73bc59/attachment.html>


More information about the llvm-commits mailing list