[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