[clang-tools-extra] 1a68c14 - [clangd] Restore -fno-ms-compatibility to tests

Sam McCall via cfe-commits cfe-commits at lists.llvm.org
Fri Dec 10 17:31:33 PST 2021


Author: Sam McCall
Date: 2021-12-11T02:31:26+01:00
New Revision: 1a68c14b577fd1b1ebfd6a064dd5faeda07042aa

URL: https://github.com/llvm/llvm-project/commit/1a68c14b577fd1b1ebfd6a064dd5faeda07042aa
DIFF: https://github.com/llvm/llvm-project/commit/1a68c14b577fd1b1ebfd6a064dd5faeda07042aa.diff

LOG: [clangd] Restore -fno-ms-compatibility to tests

Turns out these weren't obsolete after all...
http://45.33.8.238/win/50653/step_9.txt

Added: 
    

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

Removed: 
    


################################################################################
diff  --git a/clang-tools-extra/clangd/unittests/DiagnosticsTests.cpp b/clang-tools-extra/clangd/unittests/DiagnosticsTests.cpp
index c28efdf014e5..02d572ed926f 100644
--- a/clang-tools-extra/clangd/unittests/DiagnosticsTests.cpp
+++ b/clang-tools-extra/clangd/unittests/DiagnosticsTests.cpp
@@ -1084,6 +1084,8 @@ void g() {  ns::$[[scope]]::X_Y();  }
   )cpp");
   TestTU TU;
   TU.Code = std::string(Test.code());
+  // FIXME: Figure out why this is needed and remove it, PR43662.
+  TU.ExtraArgs.push_back("-fno-ms-compatibility");
   auto Index = buildIndexWithSymbol(
       SymbolWithHeader{"ns::scope::X_Y", "unittest:///x.h", "\"x.h\""});
   TU.ExternalIndex = Index.get();
@@ -1109,6 +1111,8 @@ void f() {
   )cpp");
   TestTU TU;
   TU.Code = std::string(Test.code());
+  // FIXME: Figure out why this is needed and remove it, PR43662.
+  TU.ExtraArgs.push_back("-fno-ms-compatibility");
   auto Index = buildIndexWithSymbol(
       {SymbolWithHeader{"clang::clangd::X", "unittest:///x.h", "\"x.h\""},
        SymbolWithHeader{"clang::clangd::ns::Y", "unittest:///y.h", "\"y.h\""}});


        


More information about the cfe-commits mailing list