[PATCH] D93688: [ASTMatchers] Ensure that we can match inside lambdas

Aaron Ballman via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Jan 5 05:50:11 PST 2021


aaron.ballman accepted this revision.
aaron.ballman added a comment.
This revision is now accepted and ready to land.

LGTM



================
Comment at: clang/lib/ASTMatchers/ASTMatchFinder.cpp:526
+        TraverseStmt(LE->getTrailingRequiresClause());
+
+        TraverseStmt(LE->getBody());
----------------
steveire wrote:
> aaron.ballman wrote:
> > Do we also need to traverse attributes of the lambda?
> At least RAV doesn't do so. It would be a new feature which is orthogonal to this MR/patch.
Okay, SGTM


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D93688/new/

https://reviews.llvm.org/D93688



More information about the cfe-commits mailing list