[clang-tidy] readability-named-parameter: don't complain about implicit parameters

Fixes http://llvm.org/PR24464.

llvm-svn: 252248

2 files changed, 6 insertions, 0 deletions

diff --git a/clang-tools-extra/clang-tidy/readability/NamedParameterCheck.cpp b/clang-tools-extra/clang-tidy/readability/NamedParameterCheck.cpp
index 2409a8b08b5..ffdc813beeb 100644
--- a/clang-tools-extra/clang-tidy/readability/NamedParameterCheck.cpp
+++ b/clang-tools-extra/clang-tidy/readability/NamedParameterCheck.cpp
@@ -45,6 +45,8 @@ void NamedParameterCheck::check(const MatchFinder::MatchResult &Result) {
   //       arguments in the same position.
   for (unsigned I = 0, E = Function->getNumParams(); I != E; ++I) {
     const ParmVarDecl *Parm = Function->getParamDecl(I);
+    if (Parm->isImplicit())
+      continue;
     // Look for unnamed parameters.
     if (!Parm->getName().empty())
       continue;
diff --git a/clang-tools-extra/test/clang-tidy/readability-named-parameter.cpp b/clang-tools-extra/test/clang-tidy/readability-named-parameter.cpp
index f2d31e26999..af2c1954bc5 100644
--- a/clang-tools-extra/test/clang-tidy/readability-named-parameter.cpp
+++ b/clang-tools-extra/test/clang-tidy/readability-named-parameter.cpp
@@ -127,3 +127,7 @@ typedef decltype(nullptr) nullptr_t;
 }
 
 void f(std::nullptr_t) {}
+
+typedef void (F)(int);
+F f;
+void f(int x) {}