[clang-tools-extra] 7830c2d - [clangd] Get rid of delayed template parsing

Kadir Cetinkaya via cfe-commits cfe-commits at lists.llvm.org
Wed Jan 29 03:17:16 PST 2020


Author: Kadir Cetinkaya
Date: 2020-01-29T12:12:45+01:00
New Revision: 7830c2d44f531bbe09f997436b6608a140db46fb

URL: https://github.com/llvm/llvm-project/commit/7830c2d44f531bbe09f997436b6608a140db46fb
DIFF: https://github.com/llvm/llvm-project/commit/7830c2d44f531bbe09f997436b6608a140db46fb.diff

LOG: [clangd] Get rid of delayed template parsing

Summary:
No need to pass fno-delayed-template-parsing as the opposite flag is
only passed to cc1 when abi is set to msvc. Sending as a follow-up to D73613 to
keep changes in the release branch minimal.

Reviewers: sammccall

Subscribers: ilya-biryukov, MaskRay, jkorous, arphaman, usaxena95, cfe-commits

Tags: #clang

Differential Revision: https://reviews.llvm.org/D73615

Added: 
    

Modified: 
    clang-tools-extra/clangd/unittests/HoverTests.cpp

Removed: 
    


################################################################################
diff  --git a/clang-tools-extra/clangd/unittests/HoverTests.cpp b/clang-tools-extra/clangd/unittests/HoverTests.cpp
index 5f8f0560a3ee..894de7dfc251 100644
--- a/clang-tools-extra/clangd/unittests/HoverTests.cpp
+++ b/clang-tools-extra/clangd/unittests/HoverTests.cpp
@@ -580,9 +580,6 @@ class Foo {})cpp";
     Annotations T(Case.Code);
     TestTU TU = TestTU::withCode(T.code());
     TU.ExtraArgs.push_back("-std=c++17");
-    // FIXME: This is no longer necessary, as the default behavior is no delayed
-    // parsing in the triplet below.
-    TU.ExtraArgs.push_back("-fno-delayed-template-parsing");
     // Types might be 
diff erent depending on the target triplet, we chose a
     // fixed one to make sure tests passes on 
diff erent platform.
     TU.ExtraArgs.push_back("--target=x86_64-pc-linux-gnu");


        


More information about the cfe-commits mailing list