ilya-biryukov created this revision. ilya-biryukov added a reviewer: kadircet. Herald added subscribers: usaxena95, arphaman, jkorous, MaskRay. Herald added a project: clang.
Repository: rG LLVM Github Monorepo https://reviews.llvm.org/D70481 Files: clang-tools-extra/clangd/ExpectedTypes.cpp clang-tools-extra/clangd/unittests/CodeCompleteTests.cpp Index: clang-tools-extra/clangd/unittests/CodeCompleteTests.cpp =================================================================== --- clang-tools-extra/clangd/unittests/CodeCompleteTests.cpp +++ clang-tools-extra/clangd/unittests/CodeCompleteTests.cpp @@ -1030,6 +1030,16 @@ SnippetSuffix("(${1:int A})")))); } +TEST(CompletionTest, NoCrashWithTemplateParamsAndPreferredTypes) { + auto Completions = completions(R"cpp( +template <template <class> class TT> int foo() { + int a = ^ +} +)cpp") + .Completions; + EXPECT_THAT(Completions, Contains(Named("TT"))); +} + SignatureHelp signatures(llvm::StringRef Text, Position Point, std::vector<Symbol> IndexSymbols = {}) { std::unique_ptr<SymbolIndex> Index; Index: clang-tools-extra/clangd/ExpectedTypes.cpp =================================================================== --- clang-tools-extra/clangd/ExpectedTypes.cpp +++ clang-tools-extra/clangd/ExpectedTypes.cpp @@ -44,12 +44,10 @@ static llvm::Optional<QualType> typeOfCompletion(const CodeCompletionResult &R) { const NamedDecl *D = R.Declaration; - if (!D) - return llvm::None; // Templates do not have a type on their own, look at the templated decl. - if (auto *Template = dyn_cast<TemplateDecl>(D)) + if (auto *Template = dyn_cast_or_null<TemplateDecl>(D)) D = Template->getTemplatedDecl(); - auto *VD = dyn_cast<ValueDecl>(D); + auto *VD = dyn_cast_or_null<ValueDecl>(D); if (!VD) return llvm::None; // We handle only variables and functions below. auto T = VD->getType();
Index: clang-tools-extra/clangd/unittests/CodeCompleteTests.cpp =================================================================== --- clang-tools-extra/clangd/unittests/CodeCompleteTests.cpp +++ clang-tools-extra/clangd/unittests/CodeCompleteTests.cpp @@ -1030,6 +1030,16 @@ SnippetSuffix("(${1:int A})")))); } +TEST(CompletionTest, NoCrashWithTemplateParamsAndPreferredTypes) { + auto Completions = completions(R"cpp( +template <template <class> class TT> int foo() { + int a = ^ +} +)cpp") + .Completions; + EXPECT_THAT(Completions, Contains(Named("TT"))); +} + SignatureHelp signatures(llvm::StringRef Text, Position Point, std::vector<Symbol> IndexSymbols = {}) { std::unique_ptr<SymbolIndex> Index; Index: clang-tools-extra/clangd/ExpectedTypes.cpp =================================================================== --- clang-tools-extra/clangd/ExpectedTypes.cpp +++ clang-tools-extra/clangd/ExpectedTypes.cpp @@ -44,12 +44,10 @@ static llvm::Optional<QualType> typeOfCompletion(const CodeCompletionResult &R) { const NamedDecl *D = R.Declaration; - if (!D) - return llvm::None; // Templates do not have a type on their own, look at the templated decl. - if (auto *Template = dyn_cast<TemplateDecl>(D)) + if (auto *Template = dyn_cast_or_null<TemplateDecl>(D)) D = Template->getTemplatedDecl(); - auto *VD = dyn_cast<ValueDecl>(D); + auto *VD = dyn_cast_or_null<ValueDecl>(D); if (!VD) return llvm::None; // We handle only variables and functions below. auto T = VD->getType();
_______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits