[PATCH] D70773: [clangd] Handle the missing call expr in targetDecl.
Haojian Wu via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Wed Nov 27 07:31:53 PST 2019
This revision was automatically updated to reflect the committed changes.
Closed by commit rG939544add98e: [clangd] Handle the missing call expr in targetDecl. (authored by hokein).
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D70773/new/
https://reviews.llvm.org/D70773
Files:
clang-tools-extra/clangd/FindTarget.cpp
clang-tools-extra/clangd/unittests/FindTargetTests.cpp
Index: clang-tools-extra/clangd/unittests/FindTargetTests.cpp
===================================================================
--- clang-tools-extra/clangd/unittests/FindTargetTests.cpp
+++ clang-tools-extra/clangd/unittests/FindTargetTests.cpp
@@ -114,6 +114,23 @@
auto X = S() [[+]] S();
)cpp";
EXPECT_DECLS("DeclRefExpr", "S operator+(S) const");
+
+ Code = R"cpp(
+ int foo();
+ int s = foo[[()]];
+ )cpp";
+ EXPECT_DECLS("CallExpr", "int foo()");
+
+ Code = R"cpp(
+ struct X {
+ void operator()(int n);
+ };
+ void test() {
+ X x;
+ x[[(123)]];
+ }
+ )cpp";
+ EXPECT_DECLS("CXXOperatorCallExpr", "void operator()(int n)");
}
TEST_F(TargetDeclTest, UsingDecl) {
Index: clang-tools-extra/clangd/FindTarget.cpp
===================================================================
--- clang-tools-extra/clangd/FindTarget.cpp
+++ clang-tools-extra/clangd/FindTarget.cpp
@@ -175,6 +175,9 @@
RelSet Flags;
Visitor(TargetFinder &Outer, RelSet Flags) : Outer(Outer), Flags(Flags) {}
+ void VisitCallExpr(const CallExpr *CE) {
+ Outer.add(CE->getCalleeDecl(), Flags);
+ }
void VisitDeclRefExpr(const DeclRefExpr *DRE) {
const Decl *D = DRE->getDecl();
// UsingShadowDecl allows us to record the UsingDecl.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D70773.231260.patch
Type: text/x-patch
Size: 1314 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20191127/f9e015be/attachment-0001.bin>
More information about the cfe-commits
mailing list