[clang-tools-extra] [Clangd] Don't traverse ConceptDecl in typeForNode (PR #191654)
Younan Zhang via cfe-commits
cfe-commits at lists.llvm.org
Sat Apr 11 11:31:16 PDT 2026
https://github.com/zyn0217 created https://github.com/llvm/llvm-project/pull/191654
ConceptDecl doesn't have an associated template declaration, and it doesn't introduce a type either.
Fixes https://github.com/llvm/llvm-project/issues/188914
>From db6159610cf2542f74a845a160c2a21503a87c38 Mon Sep 17 00:00:00 2001
From: Younan Zhang <zyn7109 at gmail.com>
Date: Sun, 12 Apr 2026 02:27:40 +0800
Subject: [PATCH] [Clangd] Don't traverse ConceptDecl in typeForNode
ConceptDecl doesn't have an associated template declaration, and it
doesn't introduce a type either.
---
clang-tools-extra/clangd/XRefs.cpp | 1 +
clang-tools-extra/clangd/unittests/XRefsTests.cpp | 9 +++++++--
2 files changed, 8 insertions(+), 2 deletions(-)
diff --git a/clang-tools-extra/clangd/XRefs.cpp b/clang-tools-extra/clangd/XRefs.cpp
index 4b685b7bf793e..2247586b78c84 100644
--- a/clang-tools-extra/clangd/XRefs.cpp
+++ b/clang-tools-extra/clangd/XRefs.cpp
@@ -2060,6 +2060,7 @@ static QualType typeForNode(const ASTContext &Ctx, const HeuristicResolver *H,
QualType VisitTemplateDecl(const TemplateDecl *D) {
return Visit(D->getTemplatedDecl());
}
+ QualType VisitConceptDecl(const ConceptDecl *C) { return QualType(); }
} V(Ctx);
return V.Visit(D);
}
diff --git a/clang-tools-extra/clangd/unittests/XRefsTests.cpp b/clang-tools-extra/clangd/unittests/XRefsTests.cpp
index 8e97db7ab17c5..00ead63050c8d 100644
--- a/clang-tools-extra/clangd/unittests/XRefsTests.cpp
+++ b/clang-tools-extra/clangd/unittests/XRefsTests.cpp
@@ -2094,14 +2094,19 @@ TEST(FindType, All) {
TEST(FindType, Definition) {
Annotations A(R"cpp(
class $decl[[X]];
- X *^x;
+ X *$x^x;
class $def[[X]] {};
+
+ template <class T>
+ concept $Concept^True = true;
)cpp");
auto TU = TestTU::withCode(A.code().str());
+ TU.ExtraArgs.push_back("-std=c++20");
ParsedAST AST = TU.build();
- EXPECT_THAT(findType(AST, A.point(), nullptr),
+ EXPECT_THAT(findType(AST, A.point("x"), nullptr),
ElementsAre(sym("X", A.range("decl"), A.range("def"))));
+ EXPECT_THAT(findType(AST, A.point("Concept"), nullptr), IsEmpty());
}
TEST(FindType, Index) {
More information about the cfe-commits
mailing list