Default vaarg lowering should support indirect struct types.
authorJames Y Knight <jyknight@google.com>
Wed, 24 Feb 2016 02:59:33 +0000 (02:59 +0000)
committerJames Y Knight <jyknight@google.com>
Wed, 24 Feb 2016 02:59:33 +0000 (02:59 +0000)
commit29b5f086ca10fb3daf5fe0ad07d4403a1979d76f
treea1b6d79d5c20b488a2c55a6716c4fb5df6266b4d
parent3f18ea0e38f95a28309c55ba2a6acb84437cc526
Default vaarg lowering should support indirect struct types.

Fixes PR11517 for SPARC.

On most targets, clang lowers va_arg itself, eschewing the use of the
llvm vaarg instruction. This is necessary (at least for now) as the type
argument to the vaarg instruction cannot represent all the ABI
information that is needed to support complex calling conventions.

However, on targets with a simpler varrags ABIs, the LLVM instruction
can work just fine, and clang can simply lower to it. Unfortunately,
even on such targets, vaarg with a struct argument would fail, because
the default lowering to vaarg was naive: it didn't take into account the
ABI attribute computed by classifyArgumentType. In particular, for the
DefaultABIInfo, structs are supposed to be passed indirectly and so
llvm's vaarg instruction should be emitted with a pointer argument.

Now, vaarg instruction emission is able to use computed ABIArgInfo for
the provided argument type, which allows the default ABI support to work
for structs too.

I haven't touched the EmitVAArg implementation for PPC32_SVR4 or XCore,
although I believe both are now redundant, and could be switched over to
use the default implementation as well.

Differential Revision: http://reviews.llvm.org/D16154

llvm-svn: 261717
clang/lib/CodeGen/CGExprAgg.cpp
clang/lib/CodeGen/CGExprScalar.cpp
clang/lib/CodeGen/TargetInfo.cpp
clang/test/CodeGen/le32-vaarg.c
clang/test/CodeGen/sparc-vaarg.c [new file with mode: 0644]