diff --git a/clang-tools-extra/clangd/Hover.cpp b/clang-tools-extra/clangd/Hover.cpp --- a/clang-tools-extra/clangd/Hover.cpp +++ b/clang-tools-extra/clangd/Hover.cpp @@ -26,6 +26,7 @@ #include "clang/AST/ExprCXX.h" #include "clang/AST/PrettyPrinter.h" #include "clang/AST/Type.h" +#include "clang/Basic/Specifiers.h" #include "clang/Index/IndexSymbol.h" #include "llvm/ADT/None.h" #include "llvm/ADT/Optional.h" @@ -205,11 +206,18 @@ // Returns the decl that should be used for querying comments, either from index // or AST. const NamedDecl *getDeclForComment(const NamedDecl *D) { - if (const auto *CTSD = llvm::dyn_cast(D)) - if (const auto *TIP = CTSD->getTemplateInstantiationPattern()) + if (const auto *TSD = llvm::dyn_cast(D)) { + // We might get a non-instantiated decl, e.g. + // template struct X {}; + // X^* x; + // Fallback to primary template in such cases. + if (TSD->getTemplateSpecializationKind() == TSK_Undeclared) + return TSD->getSpecializedTemplate(); + if (const auto *TIP = TSD->getTemplateInstantiationPattern()) return TIP; - if (const auto *VTSD = llvm::dyn_cast(D)) - if (const auto *TIP = VTSD->getTemplateInstantiationPattern()) + } + if (const auto *TSD = llvm::dyn_cast(D)) + if (const auto *TIP = TSD->getTemplateInstantiationPattern()) return TIP; if (const auto *FD = D->getAsFunction()) if (const auto *TIP = FD->getTemplateInstantiationPattern()) diff --git a/clang-tools-extra/clangd/unittests/HoverTests.cpp b/clang-tools-extra/clangd/unittests/HoverTests.cpp --- a/clang-tools-extra/clangd/unittests/HoverTests.cpp +++ b/clang-tools-extra/clangd/unittests/HoverTests.cpp @@ -556,15 +556,22 @@ }}, { R"cpp( + // comment template class Foo {}; - Foo* bar(); + // comment2 + template class Foo {}; void foo() { - [[^auto]] *x = bar(); + [[Fo^o]] *x = nullptr; } )cpp", [](HoverInfo &HI) { - HI.Name = "Foo"; + HI.Name = "Foo"; HI.Kind = index::SymbolKind::Class; + HI.NamespaceScope = ""; + HI.Definition = "template <> class Foo"; + // FIXME: Maybe force instantiation to make use of real template + // pattern. + HI.Documentation = "comment"; }}, }; for (const auto &Case : Cases) {