[llvm-commits] [patch] Remember the correct alignment when splitting an VAARG

Rafael Espindola espindola at google.com
Fri Jun 25 10:54:07 PDT 2010


Currently if the target doesn't supports i64 (for example) we will
split a i64 VAARG in two 32 bit ones. The problem is that we forget
that the first one had to be aligned. The attached patch adds
alignment information to the VAARGS nodes. Another option would be to
do the alignment when splitting, but this looked cleaner.

With this patch we generate inefficient but correct code. The FE's
don't produce va_arg, but since it is in the language spec, it is good
if the backends support it.

Is the patch OK?

Cheers,
-- 
Rafael Ávila de Espíndola
-------------- next part --------------
A non-text attachment was scrubbed...
Name: va_arg.patch
Type: text/x-patch
Size: 5212 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20100625/abe1dfed/attachment.bin>


More information about the llvm-commits mailing list