diff --git a/clang-tools-extra/clangd/unittests/FindTargetTests.cpp b/clang-tools-extra/clangd/unittests/FindTargetTests.cpp index 1dab9d0935383c..2f690284b0c74e 100644 --- a/clang-tools-extra/clangd/unittests/FindTargetTests.cpp +++ b/clang-tools-extra/clangd/unittests/FindTargetTests.cpp @@ -326,9 +326,6 @@ TEST_F(TargetDeclTest, Types) { EXPECT_DECLS("TypedefTypeLoc", {"typedef ns::S X", Rel::Alias}, {"struct S", Rel::Underlying}); - // FIXME: Auto-completion in a template requires disabling delayed template - // parsing. - Flags = {"-fno-delayed-template-parsing"}; Code = R"cpp( template void foo() { [[T]] x; } @@ -336,9 +333,6 @@ TEST_F(TargetDeclTest, Types) { EXPECT_DECLS("TemplateTypeParmTypeLoc", "class T"); Flags.clear(); - // FIXME: Auto-completion in a template requires disabling delayed template - // parsing. - Flags = {"-fno-delayed-template-parsing"}; Code = R"cpp( template class T> void foo() { [[T]] x; } @@ -615,9 +609,6 @@ TEST_F(TargetDeclTest, Lambda) { } TEST_F(TargetDeclTest, OverloadExpr) { - // FIXME: Auto-completion in a template requires disabling delayed template - // parsing. - Flags = {"-fno-delayed-template-parsing"}; Flags.push_back("--target=x86_64-pc-linux-gnu"); Code = R"cpp( @@ -676,8 +667,6 @@ TEST_F(TargetDeclTest, OverloadExpr) { } TEST_F(TargetDeclTest, DependentExprs) { - Flags = {"-fno-delayed-template-parsing"}; - // Heuristic resolution of method of dependent field Code = R"cpp( struct A { void foo() {} }; @@ -758,8 +747,6 @@ TEST_F(TargetDeclTest, DependentExprs) { } TEST_F(TargetDeclTest, DependentTypes) { - Flags = {"-fno-delayed-template-parsing"}; - // Heuristic resolution of dependent type name Code = R"cpp( template @@ -981,7 +968,6 @@ class FindExplicitReferencesTest : public ::testing::Test { // FIXME: Auto-completion in a template requires disabling delayed template // parsing. - TU.ExtraArgs.push_back("-fno-delayed-template-parsing"); TU.ExtraArgs.push_back("-std=c++20"); TU.ExtraArgs.push_back("-xobjective-c++"); diff --git a/clang-tools-extra/clangd/unittests/ParsedASTTests.cpp b/clang-tools-extra/clangd/unittests/ParsedASTTests.cpp index 98a24832d568bf..a7328373f07f77 100644 --- a/clang-tools-extra/clangd/unittests/ParsedASTTests.cpp +++ b/clang-tools-extra/clangd/unittests/ParsedASTTests.cpp @@ -184,9 +184,6 @@ TEST(ParsedASTTest, template <> int foo = 0; )cpp"; - // FIXME: Auto-completion in a template requires disabling delayed template - // parsing. - TU.ExtraArgs.push_back("-fno-delayed-template-parsing"); auto AST = TU.build(); EXPECT_THAT( diff --git a/clang-tools-extra/clangd/unittests/RenameTests.cpp b/clang-tools-extra/clangd/unittests/RenameTests.cpp index f1f9e17a7b8b7a..b1cd8bc3dc4904 100644 --- a/clang-tools-extra/clangd/unittests/RenameTests.cpp +++ b/clang-tools-extra/clangd/unittests/RenameTests.cpp @@ -829,7 +829,6 @@ TEST(RenameTest, WithinFileRename) { SCOPED_TRACE(T); Annotations Code(T); auto TU = TestTU::withCode(Code.code()); - TU.ExtraArgs.push_back("-fno-delayed-template-parsing"); TU.ExtraArgs.push_back("-xobjective-c++"); auto AST = TU.build(); for (const auto &RenamePos : Code.points()) { diff --git a/clang-tools-extra/clangd/unittests/SelectionTests.cpp b/clang-tools-extra/clangd/unittests/SelectionTests.cpp index f187020b149872..e9c689f329ab9e 100644 --- a/clang-tools-extra/clangd/unittests/SelectionTests.cpp +++ b/clang-tools-extra/clangd/unittests/SelectionTests.cpp @@ -462,9 +462,6 @@ TEST(SelectionTest, CommonAncestor) { TestTU TU; TU.Code = std::string(Test.code()); - // FIXME: Auto-completion in a template requires disabling delayed template - // parsing. - TU.ExtraArgs.push_back("-fno-delayed-template-parsing"); TU.ExtraArgs.push_back("-xobjective-c++"); auto AST = TU.build(); diff --git a/clang-tools-extra/clangd/unittests/SemanticHighlightingTests.cpp b/clang-tools-extra/clangd/unittests/SemanticHighlightingTests.cpp index 917248628d49f6..eb848dddad20ad 100644 --- a/clang-tools-extra/clangd/unittests/SemanticHighlightingTests.cpp +++ b/clang-tools-extra/clangd/unittests/SemanticHighlightingTests.cpp @@ -74,9 +74,6 @@ void checkHighlightings(llvm::StringRef Code, TestTU TU; TU.Code = std::string(Test.code()); - // FIXME: Auto-completion in a template requires disabling delayed template - // parsing. - TU.ExtraArgs.push_back("-fno-delayed-template-parsing"); TU.ExtraArgs.push_back("-std=c++20"); for (auto File : AdditionalFiles) diff --git a/clang-tools-extra/clangd/unittests/XRefsTests.cpp b/clang-tools-extra/clangd/unittests/XRefsTests.cpp index 06bdb9fc4c8a94..a7715900ab979f 100644 --- a/clang-tools-extra/clangd/unittests/XRefsTests.cpp +++ b/clang-tools-extra/clangd/unittests/XRefsTests.cpp @@ -896,9 +896,6 @@ TEST(LocateSymbol, All) { TestTU TU; TU.Code = std::string(T.code()); - // FIXME: Auto-completion in a template requires disabling delayed template - // parsing. - TU.ExtraArgs.push_back("-fno-delayed-template-parsing"); TU.ExtraArgs.push_back("-xobjective-c++"); auto AST = TU.build(); diff --git a/clang-tools-extra/clangd/unittests/tweaks/DefineInlineTests.cpp b/clang-tools-extra/clangd/unittests/tweaks/DefineInlineTests.cpp index f190a314bc4a59..4c1bd3ab555670 100644 --- a/clang-tools-extra/clangd/unittests/tweaks/DefineInlineTests.cpp +++ b/clang-tools-extra/clangd/unittests/tweaks/DefineInlineTests.cpp @@ -176,9 +176,6 @@ TEST_F(DefineInlineTest, CheckForCanonDecl) { } TEST_F(DefineInlineTest, UsingShadowDecls) { - // Template body is not parsed until instantiation time on windows, which - // results in arbitrary failures as function body becomes NULL. - ExtraArgs.push_back("-fno-delayed-template-parsing"); EXPECT_UNAVAILABLE(R"cpp( namespace ns1 { void foo(int); } namespace ns2 { void foo(int*); } @@ -426,9 +423,6 @@ TEST_F(DefineInlineTest, TransformDependentTypes) { using namespace a; )cpp"; - // Template body is not parsed until instantiation time on windows, which - // results in arbitrary failures as function body becomes NULL. - ExtraArgs.push_back("-fno-delayed-template-parsing"); EXPECT_EQ(apply(Test), Expected); } @@ -512,9 +506,6 @@ TEST_F(DefineInlineTest, TransformFunctionTempls) { )cpp"}, }; - // Template body is not parsed until instantiation time on windows, which - // results in arbitrary failures as function body becomes NULL. - ExtraArgs.push_back("-fno-delayed-template-parsing"); for (const auto &Case : Cases) EXPECT_EQ(apply(Case.first), Case.second) << Case.first; } @@ -658,7 +649,6 @@ est); void foo(PARAM, TYPE b, TYPE c, TYPE d = BODY(x)){} )cpp"}, }; - ExtraArgs.push_back("-fno-delayed-template-parsing"); for (const auto &Case : Cases) EXPECT_EQ(apply(Case.first), Case.second) << Case.first; } @@ -689,7 +679,6 @@ TEST_F(DefineInlineTest, TransformTemplParamNames) { }; )cpp"; - ExtraArgs.push_back("-fno-delayed-template-parsing"); EXPECT_EQ(apply(Test), Expected); } @@ -936,7 +925,6 @@ TEST_F(DefineInlineTest, QualifyWithUsingDirectives) { } TEST_F(DefineInlineTest, AddInline) { - ExtraArgs.push_back("-fno-delayed-template-parsing"); llvm::StringMap EditedFiles; ExtraFiles["a.h"] = "void foo();"; apply(R"cpp(#include "a.h" diff --git a/clang-tools-extra/clangd/unittests/tweaks/DefineOutlineTests.cpp b/clang-tools-extra/clangd/unittests/tweaks/DefineOutlineTests.cpp index 1a49a375033546..fa627282e19372 100644 --- a/clang-tools-extra/clangd/unittests/tweaks/DefineOutlineTests.cpp +++ b/clang-tools-extra/clangd/unittests/tweaks/DefineOutlineTests.cpp @@ -99,9 +99,6 @@ TEST_F(DefineOutlineTest, ApplyTest) { llvm::StringMap EditedFiles; ExtraFiles["Test.cpp"] = ""; FileName = "Test.hpp"; - // Template body is not parsed until instantiation time on windows, which - // results in arbitrary failures as function body becomes NULL. - ExtraArgs.push_back("-fno-delayed-template-parsing"); struct { llvm::StringRef Test; diff --git a/clang-tools-extra/clangd/unittests/tweaks/ExpandAutoTypeTests.cpp b/clang-tools-extra/clangd/unittests/tweaks/ExpandAutoTypeTests.cpp index 5554b68b31c7e0..1b784f63f59cff 100644 --- a/clang-tools-extra/clangd/unittests/tweaks/ExpandAutoTypeTests.cpp +++ b/clang-tools-extra/clangd/unittests/tweaks/ExpandAutoTypeTests.cpp @@ -74,9 +74,6 @@ TEST_F(ExpandAutoTypeTest, Test) { // expanding types in structured bindings is syntactically invalid. EXPECT_UNAVAILABLE("const ^auto &[x,y] = (int[]){1,2};"); - // FIXME: Auto-completion in a template requires disabling delayed template - // parsing. - ExtraArgs.push_back("-fno-delayed-template-parsing"); // unknown types in a template should not be replaced EXPECT_THAT(apply("template void x() { ^auto y = T::z(); }"), StartsWith("fail: Could not deduce type for 'auto' type"));