[clang-tools-extra] aa3c763 - [clang] Traverse init-captures while indexing
Nathan Ridge via cfe-commits
cfe-commits at lists.llvm.org
Mon Sep 21 23:32:20 PDT 2020
Author: Nathan Ridge
Date: 2020-09-22T02:32:09-04:00
New Revision: aa3c7638ba1648e020cf65a424389e093f2b3a1a
URL: https://github.com/llvm/llvm-project/commit/aa3c7638ba1648e020cf65a424389e093f2b3a1a
DIFF: https://github.com/llvm/llvm-project/commit/aa3c7638ba1648e020cf65a424389e093f2b3a1a.diff
LOG: [clang] Traverse init-captures while indexing
Fixes https://github.com/clangd/clangd/issues/496
Differential Revision: https://reviews.llvm.org/D87257
Added:
Modified:
clang-tools-extra/clangd/unittests/XRefsTests.cpp
clang/lib/Index/IndexBody.cpp
clang/test/Index/cxx14-lambdas.cpp
Removed:
################################################################################
diff --git a/clang-tools-extra/clangd/unittests/XRefsTests.cpp b/clang-tools-extra/clangd/unittests/XRefsTests.cpp
index d2337dcbd7b3..2176b59ab9ed 100644
--- a/clang-tools-extra/clangd/unittests/XRefsTests.cpp
+++ b/clang-tools-extra/clangd/unittests/XRefsTests.cpp
@@ -1582,6 +1582,12 @@ TEST(FindReferences, WithinAST) {
f.[[^~]]Foo();
}
)cpp",
+ R"cpp(// Lambda capture initializer
+ void foo() {
+ int [[w^aldo]] = 42;
+ auto lambda = [x = [[waldo]]](){};
+ }
+ )cpp",
};
for (const char *Test : Tests) {
Annotations T(Test);
diff --git a/clang/lib/Index/IndexBody.cpp b/clang/lib/Index/IndexBody.cpp
index 01cf559d7057..e4944fd0fc3b 100644
--- a/clang/lib/Index/IndexBody.cpp
+++ b/clang/lib/Index/IndexBody.cpp
@@ -391,11 +391,13 @@ class BodyIndexer : public RecursiveASTVisitor<BodyIndexer> {
if (C->capturesThis() || C->capturesVLAType())
return true;
+ if (!base::TraverseStmt(Init))
+ return false;
+
if (C->capturesVariable() && IndexCtx.shouldIndexFunctionLocalSymbols())
return IndexCtx.handleReference(C->getCapturedVar(), C->getLocation(),
Parent, ParentDC, SymbolRoleSet());
- // FIXME: Lambda init-captures.
return true;
}
diff --git a/clang/test/Index/cxx14-lambdas.cpp b/clang/test/Index/cxx14-lambdas.cpp
index abca65b518bc..0c9aaec6c88e 100644
--- a/clang/test/Index/cxx14-lambdas.cpp
+++ b/clang/test/Index/cxx14-lambdas.cpp
@@ -28,7 +28,9 @@ struct X {
// CHECK-LOAD: cxx14-lambdas.cpp:8:7: ReturnStmt= Extent=[8:7 - 8:29]
// RUN: env CINDEXTEST_INDEXLOCALSYMBOLS=1 c-index-test -index-file -std=c++14 %s | FileCheck -check-prefix=CHECK-INDEX %s
+// CHECK-INDEX: [indexEntityReference]: kind: variable | name: localA | USR: c:cxx14-lambdas.cpp at 100@S at X@F at f#@localA | lang: C | cursor: DeclRefExpr=localA:6:9 | loc: 7:27
// CHECK-INDEX: [indexEntityReference]: kind: variable | name: ptr | USR: c:cxx14-lambdas.cpp at 139@S at X@F at f#@Sa at F@operator()#I#1 at ptr | lang: C | cursor: VariableRef=ptr:7:20 | loc: 7:20
+// CHECK-INDEX: [indexEntityReference]: kind: variable | name: localB | USR: c:cxx14-lambdas.cpp at 100@S at X@F at f#@localB | lang: C | cursor: DeclRefExpr=localB:6:17 | loc: 7:42
// CHECK-INDEX: [indexEntityReference]: kind: variable | name: copy | USR: c:cxx14-lambdas.cpp at 154@S at X@F at f#@Sa at F@operator()#I#1 at copy | lang: C | cursor: VariableRef=copy:7:35 | loc: 7:35
// CHECK-INDEX: [indexDeclaration]: kind: variable | name: x | USR: c:cxx14-lambdas.cpp at 170@S at X@F at f#@Sa at F@operator()#I#1 at x | lang: C | cursor: ParmDecl=x:7:59 (Definition) | loc: 7:59
// CHECK-INDEX: [indexEntityReference]: kind: typedef | name: Integer | USR: c:cxx14-lambdas.cpp at T@Integer | lang: C | cursor: TypeRef=Integer:3:13 | loc: 7:51
More information about the cfe-commits
mailing list