]> granicus.if.org Git - clang/commit
Fix the required args count for variadic blocks.
authorJohn McCall <rjmccall@apple.com>
Fri, 7 Dec 2012 07:03:17 +0000 (07:03 +0000)
committerJohn McCall <rjmccall@apple.com>
Fri, 7 Dec 2012 07:03:17 +0000 (07:03 +0000)
commite56bb36e8eea89bae7dfe6eb6ea0455af126bf4a
treec6ac1420506da704a1f788684944e958214e818e
parent8fda1ab0fbfeac24489906d0bd6714c337f66e0f
Fix the required args count for variadic blocks.

We were emitting calls to blocks as if all arguments were
required --- i.e. with signature (A,B,C,D,...) rather than
(A,B,...).  This patch fixes that and accounts for the
implicit block-context argument as a required argument.
In addition, this patch changes the function type under which
we call unprototyped functions on platforms like x86-64 that
guarantee compatibility of variadic functions with unprototyped
function types;  previously we would always call such functions
under the LLVM type T (...)*, but now we will call them under
the type T (A,B,C,D,...)*.  This last change should have no
material effect except for making the type conventions more
explicit;  it was a side-effect of the most convenient implementation.

git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@169588 91177308-0d34-0410-b5e6-96231b3b80d8
lib/CodeGen/CGBlocks.cpp
lib/CodeGen/CGCall.cpp
lib/CodeGen/CGCall.h
lib/CodeGen/CGExpr.cpp
lib/CodeGen/CodeGenTypes.h
lib/CodeGen/TargetInfo.h
test/CodeGen/x86_64-arguments.c
test/CodeGenObjC/blocks.m