aboutsummaryrefslogtreecommitdiff
path: root/clang/lib/Frontend/CompilerInvocation.cpp
diff options
context:
space:
mode:
authorReid Kleckner <rnk@google.com>2019-09-06 22:55:26 +0000
committerReid Kleckner <rnk@google.com>2019-09-06 22:55:26 +0000
commit28328c3771e4241366ecb7aba8963dc92b3065d7 (patch)
treee2c953cb16c6e7461e3070b4d4d24e5e5f6c8aa2 /clang/lib/Frontend/CompilerInvocation.cpp
parentc1779194097f0553c292750195673ae65fdc4d32 (diff)
downloadllvm-28328c3771e4241366ecb7aba8963dc92b3065d7.zip
llvm-28328c3771e4241366ecb7aba8963dc92b3065d7.tar.gz
llvm-28328c3771e4241366ecb7aba8963dc92b3065d7.tar.bz2
Use musttail for variadic method thunks when possible
This avoids cloning variadic virtual methods when the target supports musttail and the return type is not covariant. I think we never implemented this previously because it doesn't handle the covariant case. But, in the MS ABI, there are some cases where vtable thunks must be emitted even when the variadic method defintion is not available, so it looks like we need to implement this. Do it for both ABIs, since it's a nice size improvement and simplification for Itanium. Emit an error when emitting thunks for variadic methods with a covariant return type. This case is essentially not implementable unless the ABI provides a way to perfectly forward variadic arguments without a tail call. Fixes PR43173. Differential Revision: https://reviews.llvm.org/D67028 llvm-svn: 371269
Diffstat (limited to 'clang/lib/Frontend/CompilerInvocation.cpp')
0 files changed, 0 insertions, 0 deletions