[AST] RecursiveASTVisitor visits lambda classes when implicit visitation is on.
authorSam McCall <sam.mccall@gmail.com>
Mon, 14 Jan 2019 10:31:42 +0000 (10:31 +0000)
committerSam McCall <sam.mccall@gmail.com>
Mon, 14 Jan 2019 10:31:42 +0000 (10:31 +0000)
commite60151c9153602fe9dbf8e5d33aa48b95f2c124b
treef4780fa894a97a57e79cf7e6b5448e85534795a0
parent7d370a36bb9fee7bd2b2d2b9ae72314e32cffcdd
[AST] RecursiveASTVisitor visits lambda classes when implicit visitation is on.

Summary:
This fixes ASTContext's parent map for nodes in such classes (e.g. operator()).
https://bugs.llvm.org/show_bug.cgi?id=39949

This also changes the observed shape of the AST for implicit RAVs.
- this includes AST MatchFinder: cxxRecordDecl() now matches lambda classes,
functionDecl() matches the call operator, and the parent chain is body -> call
operator -> lambda class -> lambdaexpr rather than body -> lambdaexpr.
- this appears not to matter for the ASTImporterLookupTable builder
- this doesn't matter for the other RAVs in-tree.

In order to do this, we remove the TraverseLambdaBody hook. The problem is it's
hard/weird to ensure this hook is called when traversing via the implicit class.
There were just two users of this hook in-tree, who use it to skip bodies.
I replaced these with explicitly traversing the captures only. Another approach
would be recording the bodies when the lambda is visited, and then recognizing
them later.
I'd be open to suggestion on how to preserve this hook, instead.

Reviewers: aaron.ballman, JonasToth

Subscribers: cfe-commits, rsmith, jdennett

Differential Revision: https://reviews.llvm.org/D56444

llvm-svn: 351047
clang/include/clang/AST/RecursiveASTVisitor.h
clang/lib/CodeGen/CodeGenPGO.cpp
clang/lib/Sema/AnalysisBasedWarnings.cpp
clang/unittests/AST/ASTContextParentMapTest.cpp
clang/unittests/Tooling/RecursiveASTVisitorTests/LambdaExpr.cpp