diff --git a/clang/docs/LibASTMatchersReference.html b/clang/docs/LibASTMatchersReference.html --- a/clang/docs/LibASTMatchersReference.html +++ b/clang/docs/LibASTMatchersReference.html @@ -1191,7 +1191,17 @@
Matches lambda captures. + +Given + int main() { + int x; + auto f = [x](){}; + auto g = [x = 1](){}; + } +In the matcher `lambdaExpr(hasAnyCapture(lambdaCapture()))`, +`lambdaCapture()` matches `x` and `x=1`. +
Matches each lambda capture in a lambda expression. + +Given + int main() { + int x, y; + float z; + auto f = [=]() { return x + y + z; }; + } +lambdaExpr(forEachLambdaCapture( + lambdaCapture(capturesVar(varDecl(hasType(isInteger())))))) +will trigger two matches, binding for 'x' and 'y' respectively. +
Matches any capture in a lambda expression. @@ -5666,6 +5691,15 @@
Matches a variable serving as the implicit variable for a lambda init- +capture. + +Example matches x (matcher = varDecl(isInitCapture())) +auto f = [x=3]() { return x; }; +
Matches a static variable with local scope. diff --git a/clang/include/clang/ASTMatchers/ASTMatchers.h b/clang/include/clang/ASTMatchers/ASTMatchers.h --- a/clang/include/clang/ASTMatchers/ASTMatchers.h +++ b/clang/include/clang/ASTMatchers/ASTMatchers.h @@ -4205,6 +4205,45 @@ InnerMatcher.matches(*Initializer, Finder, Builder)); } +/// Matches a variable serving as the implicit variable for a lambda init- +/// capture. +/// +/// Example matches x (matcher = varDecl(isInitCapture())) +/// \code +/// auto f = [x=3]() { return x; }; +/// \endcode +AST_MATCHER(VarDecl, isInitCapture) { return Node.isInitCapture(); } + +/// Matches each lambda capture in a lambda expression. +/// +/// Given +/// \code +/// int main() { +/// int x, y; +/// float z; +/// auto f = [=]() { return x + y + z; }; +/// } +/// \endcode +/// lambdaExpr(forEachLambdaCapture( +/// lambdaCapture(capturesVar(varDecl(hasType(isInteger())))))) +/// will trigger two matches, binding for 'x' and 'y' respectively. +AST_MATCHER_P(LambdaExpr, forEachLambdaCapture, LambdaCaptureMatcher, + InnerMatcher) { + BoundNodesTreeBuilder Result; + bool Matched = false; + for (const auto &Capture : Node.captures()) { + if (Finder->isTraversalIgnoringImplicitNodes() && Capture.isImplicit()) + continue; + BoundNodesTreeBuilder CaptureBuilder(*Builder); + if (InnerMatcher.matches(Capture, Finder, &CaptureBuilder)) { + Matched = true; + Result.addMatch(CaptureBuilder); + } + } + *Builder = std::move(Result); + return Matched; +} + /// \brief Matches a static variable with local scope. /// /// Example matches y (matcher = varDecl(isStaticLocal())) @@ -4590,6 +4629,18 @@ return false; } +/// Matches lambda captures. +/// +/// Given +/// \code +/// int main() { +/// int x; +/// auto f = [x](){}; +/// auto g = [x = 1](){}; +/// } +/// \endcode +/// In the matcher `lambdaExpr(hasAnyCapture(lambdaCapture()))`, +/// `lambdaCapture()` matches `x` and `x=1`. extern const internal::VariadicAllOfMatcherlambdaCapture; /// Matches any capture in a lambda expression. diff --git a/clang/lib/ASTMatchers/Dynamic/Registry.cpp b/clang/lib/ASTMatchers/Dynamic/Registry.cpp --- a/clang/lib/ASTMatchers/Dynamic/Registry.cpp +++ b/clang/lib/ASTMatchers/Dynamic/Registry.cpp @@ -246,6 +246,7 @@ REGISTER_MATCHER(forEachArgumentWithParamType); REGISTER_MATCHER(forEachConstructorInitializer); REGISTER_MATCHER(forEachDescendant); + REGISTER_MATCHER(forEachLambdaCapture); REGISTER_MATCHER(forEachOverridden); REGISTER_MATCHER(forEachSwitchCase); REGISTER_MATCHER(forField); @@ -424,6 +425,7 @@ REGISTER_MATCHER(isImplicit); REGISTER_MATCHER(isInStdNamespace); REGISTER_MATCHER(isInTemplateInstantiation); + REGISTER_MATCHER(isInitCapture); REGISTER_MATCHER(isInline); REGISTER_MATCHER(isInstanceMessage); REGISTER_MATCHER(isInstanceMethod); diff --git a/clang/unittests/ASTMatchers/ASTMatchersNarrowingTest.cpp b/clang/unittests/ASTMatchers/ASTMatchersNarrowingTest.cpp --- a/clang/unittests/ASTMatchers/ASTMatchersNarrowingTest.cpp +++ b/clang/unittests/ASTMatchers/ASTMatchersNarrowingTest.cpp @@ -1439,6 +1439,22 @@ EXPECT_TRUE(notMatches("int X;", M)); } +TEST_P(ASTMatchersTest, IsInitCapture) { + if (!GetParam().isCXX11OrLater()) { + return; + } + auto M = varDecl(hasName("vd"), isInitCapture()); + EXPECT_TRUE(notMatches( + "int main() { int vd = 3; auto f = [vd]() { return vd; }; }", M)); + + if (!GetParam().isCXX14OrLater()) { + return; + } + EXPECT_TRUE(matches("int main() { auto f = [vd=3]() { return vd; }; }", M)); + EXPECT_TRUE(matches( + "int main() { int x = 3; auto f = [vd=x]() { return vd; }; }", M)); +} + TEST_P(ASTMatchersTest, StorageDuration) { StringRef T = "void f() { int x; static int y; } int a;static int b;extern int c;"; diff --git a/clang/unittests/ASTMatchers/ASTMatchersTraversalTest.cpp b/clang/unittests/ASTMatchers/ASTMatchersTraversalTest.cpp --- a/clang/unittests/ASTMatchers/ASTMatchersTraversalTest.cpp +++ b/clang/unittests/ASTMatchers/ASTMatchersTraversalTest.cpp @@ -4769,6 +4769,66 @@ cxxConstructorDecl(forEachConstructorInitializer(cxxCtorInitializer())))); } +TEST(ForEachLambdaCapture, MatchesCaptures) { + EXPECT_TRUE(matches( + "int main() { int x, y; auto f = [x, y]() { return x + y; }; }", + lambdaExpr(forEachLambdaCapture(lambdaCapture())), langCxx11OrLater())); + auto matcher = lambdaExpr(forEachLambdaCapture( + lambdaCapture(capturesVar(varDecl(hasType(isInteger())))).bind("LC"))); + EXPECT_TRUE(matchAndVerifyResultTrue( + "int main() { int x, y; float z; auto f = [=]() { return x + y + z; }; }", + matcher, std::make_unique >("LC", 2))); + EXPECT_TRUE(matchAndVerifyResultTrue( + "int main() { int x, y; float z; auto f = [x, y, z]() { return x + y + " + "z; }; }", + matcher, std::make_unique >("LC", 2))); +} + +TEST(ForEachLambdaCapture, IgnoreUnlessSpelledInSource) { + auto matcher = + traverse(TK_IgnoreUnlessSpelledInSource, + lambdaExpr(forEachLambdaCapture( + lambdaCapture(capturesVar(varDecl(hasType(isInteger())))) + .bind("LC")))); + EXPECT_TRUE( + notMatches("int main() { int x, y; auto f = [=]() { return x + y; }; }", + matcher, langCxx11OrLater())); + EXPECT_TRUE( + notMatches("int main() { int x, y; auto f = [&]() { return x + y; }; }", + matcher, langCxx11OrLater())); + EXPECT_TRUE(matchAndVerifyResultTrue( + R"cc( + int main() { + int x, y; + float z; + auto f = [=, &y]() { return x + y + z; }; + } + )cc", + matcher, std::make_unique >("LC", 1))); +} + +TEST(ForEachLambdaCapture, MatchImplicitCapturesOnly) { + auto matcher = + lambdaExpr(forEachLambdaCapture(lambdaCapture(isImplicit()).bind("LC"))); + EXPECT_TRUE(matchAndVerifyResultTrue( + "int main() { int x, y, z; auto f = [=, &z]() { return x + y + z; }; }", + matcher, std::make_unique >("LC", 2))); + EXPECT_TRUE(matchAndVerifyResultTrue( + "int main() { int x, y, z; auto f = [&, z]() { return x + y + z; }; }", + matcher, std::make_unique >("LC", 2))); +} + +TEST(ForEachLambdaCapture, MatchExplicitCapturesOnly) { + auto matcher = lambdaExpr( + forEachLambdaCapture(lambdaCapture(unless(isImplicit())).bind("LC"))); + EXPECT_TRUE(matchAndVerifyResultTrue( + "int main() { int x, y, z; auto f = [=, &z]() { return x + y + z; }; }", + matcher, std::make_unique >("LC", 1))); + EXPECT_TRUE(matchAndVerifyResultTrue( + "int main() { int x, y, z; auto f = [&, z]() { return x + y + z; }; }", + matcher, std::make_unique >("LC", 1))); +} + TEST(HasConditionVariableStatement, DoesNotMatchCondition) { EXPECT_TRUE(notMatches( "void x() { if(true) {} }",