diff options
author | Douglas Gregor <dgregor@apple.com> | 2011-01-05 23:23:17 +0000 |
---|---|---|
committer | Douglas Gregor <dgregor@apple.com> | 2011-01-05 23:23:17 +0000 |
commit | 86bea351a9abbf88df9e70768450d30ed0682d27 (patch) | |
tree | 44891d26ff278a23fd7f0a49ce0603631fb3984e | |
parent | f6272cd7e9e066b92cd3486bff1f86f855182100 (diff) | |
download | bcm5719-llvm-86bea351a9abbf88df9e70768450d30ed0682d27.tar.gz bcm5719-llvm-86bea351a9abbf88df9e70768450d30ed0682d27.zip |
Fast-path an arity check when performing template argument deduction that compares two parameter-type-lists. No functionality change.
llvm-svn: 122928
-rw-r--r-- | clang/lib/Sema/SemaTemplateDeduction.cpp | 8 |
1 files changed, 6 insertions, 2 deletions
diff --git a/clang/lib/Sema/SemaTemplateDeduction.cpp b/clang/lib/Sema/SemaTemplateDeduction.cpp index e762d341176..603a7efea5f 100644 --- a/clang/lib/Sema/SemaTemplateDeduction.cpp +++ b/clang/lib/Sema/SemaTemplateDeduction.cpp @@ -541,8 +541,12 @@ DeduceTemplateArguments(Sema &S, TemplateDeductionInfo &Info, llvm::SmallVectorImpl<DeducedTemplateArgument> &Deduced, unsigned TDF) { - // FIXME: Fast-path check with NumParams != NumArgs and there are no - // pack expansions around. + // Fast-path check to see if we have too many/too few arguments. + if (NumParams != NumArgs && + !(NumParams && isa<PackExpansionType>(Params[NumParams - 1])) && + !(NumArgs && isa<PackExpansionType>(Args[NumArgs - 1]))) + return NumArgs < NumParams ? Sema::TDK_TooFewArguments + : Sema::TDK_TooManyArguments; // C++0x [temp.deduct.type]p10: // Similarly, if P has a form that contains (T), then each parameter type |