[cfe-commits] r160570 - in /cfe/trunk: lib/Sema/SemaOverload.cpp test/SemaCXX/member-operator-expr.cpp

[Aaron Ballman]

Author: aaronballman
Date: Fri Jul 20 15:40:35 2012
New Revision: 160570

URL: http://llvm.org/viewvc/llvm-project?rev=160570&view=rev
Log:
No longer assuming the number of prototype arguments is always less than the number of formal parameters for a variadic function call.

Modified:
    cfe/trunk/lib/Sema/SemaOverload.cpp
    cfe/trunk/test/SemaCXX/member-operator-expr.cpp

Modified: cfe/trunk/lib/Sema/SemaOverload.cpp
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Sema/SemaOverload.cpp?rev=160570&r1=160569&r2=160570&view=diff
==============================================================================
--- cfe/trunk/lib/Sema/SemaOverload.cpp (original)
+++ cfe/trunk/lib/Sema/SemaOverload.cpp Fri Jul 20 15:40:35 2012
@@ -10998,7 +10998,7 @@
   // If this is a variadic call, handle args passed through "...".
   if (Proto->isVariadic()) {
     // Promote the arguments (C99 6.5.2.2p7).
-    for (unsigned i = NumArgsInProto; i != NumArgs; i++) {
+    for (unsigned i = NumArgsInProto; i < NumArgs; i++) {
       ExprResult Arg = DefaultVariadicArgumentPromotion(Args[i], VariadicMethod, 0);
       IsError |= Arg.isInvalid();
       TheCall->setArg(i + 1, Arg.take());

Modified: cfe/trunk/test/SemaCXX/member-operator-expr.cpp
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/test/SemaCXX/member-operator-expr.cpp?rev=160570&r1=160569&r2=160570&view=diff
==============================================================================
--- cfe/trunk/test/SemaCXX/member-operator-expr.cpp (original)
+++ cfe/trunk/test/SemaCXX/member-operator-expr.cpp Fri Jul 20 15:40:35 2012
@@ -27,3 +27,8 @@
   x->operator float(); // expected-error{{no member named 'operator float'}}
   x->operator; // expected-error{{expected a type}}
 }
+
+namespace pr13157 {
+  class A { public: void operator()(int x, int y = 2, ...) {} };
+  void f() { A()(1); }
+}
\ No newline at end of file