r192030 - Sema: Cleanup redundant variable NumArgsToCheck

Benjamin Kramer benny.kra at googlemail.com
Sat Oct 5 03:03:01 PDT 2013


Author: d0k
Date: Sat Oct  5 05:03:01 2013
New Revision: 192030

URL: http://llvm.org/viewvc/llvm-project?rev=192030&view=rev
Log:
Sema: Cleanup redundant variable NumArgsToCheck

Patch by Daniel Marjamäki!

Modified:
    cfe/trunk/lib/Sema/SemaOverload.cpp

Modified: cfe/trunk/lib/Sema/SemaOverload.cpp
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Sema/SemaOverload.cpp?rev=192030&r1=192029&r2=192030&view=diff
==============================================================================
--- cfe/trunk/lib/Sema/SemaOverload.cpp (original)
+++ cfe/trunk/lib/Sema/SemaOverload.cpp Sat Oct  5 05:03:01 2013
@@ -11396,7 +11396,6 @@ Sema::BuildCallToObjectOfClassType(Scope
     Method->getType()->getAs<FunctionProtoType>();
 
   unsigned NumArgsInProto = Proto->getNumArgs();
-  unsigned NumArgsToCheck = std::max<unsigned>(Args.size(), NumArgsInProto);
 
   DeclarationNameInfo OpLocInfo(
                Context.DeclarationNames.getCXXOperatorName(OO_Call), LParenLoc);
@@ -11434,8 +11433,6 @@ Sema::BuildCallToObjectOfClassType(Scope
   // slots in the call for them.
   if (Args.size() < NumArgsInProto)
     TheCall->setNumArgs(Context, NumArgsInProto + 1);
-  else if (Args.size() > NumArgsInProto)
-    NumArgsToCheck = NumArgsInProto;
 
   bool IsError = false;
 
@@ -11450,7 +11447,7 @@ Sema::BuildCallToObjectOfClassType(Scope
   TheCall->setArg(0, Object.take());
 
   // Check the argument types.
-  for (unsigned i = 0; i != NumArgsToCheck; i++) {
+  for (unsigned i = 0; i != NumArgsInProto; i++) {
     Expr *Arg;
     if (i < Args.size()) {
       Arg = Args[i];





More information about the cfe-commits mailing list