[llvm-commits] PATCH: llvm-gcc option to emit "va_arg" instruction

Rafael Espíndola rafael.espindola at gmail.com
Tue Nov 2 07:25:15 PDT 2010


2010/11/2 Duncan Sands <baldrick at free.fr>:
> The attached patch just passes va_arg through.  Unfortunately bootstrap
> fails,
> not because the compiler crashes but because the genmddeps tool built by the
> new compiler crashes, i.e. there is a miscompilation.  But perhaps that
> would
> also happen with the builtin version?

My first impression is that they are equivalent, other than the fact
that your patches enables it :-)

It is too soon to enable it for all architectures as the va_arg
implementation is incomplete. We need some way to do it in a target by
target basis. Not sure if va_arg is feature complete in any target at
the moment...

> Ciao,
>
> Duncan.
>

Cheers,
Rafael




More information about the llvm-commits mailing list