[PATCH] D64857: [clangd] Force the required interpretation of #import on windows tests.

[Sam McCall via Phabricator via llvm-commits]

This revision was automatically updated to reflect the committed changes.
Closed by commit rL366321: [clangd] Force the required interpretation of #import on windows tests. (authored by sammccall, committed by ).

Changed prior to commit:
  https://reviews.llvm.org/D64857?vs=210279&id=210308#toc

Repository:
  rL LLVM

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D64857/new/

https://reviews.llvm.org/D64857

Files:
  clang-tools-extra/trunk/clangd/unittests/TestTU.cpp


Index: clang-tools-extra/trunk/clangd/unittests/TestTU.cpp
===================================================================
--- clang-tools-extra/trunk/clangd/unittests/TestTU.cpp
+++ clang-tools-extra/trunk/clangd/unittests/TestTU.cpp
@@ -38,6 +38,10 @@
     Cmd.push_back("-include");
     Cmd.push_back(ImplicitHeaderGuard ? ImportThunk.c_str()
                                       : FullHeaderName.c_str());
+    // ms-compatibility changes the meaning of #import.
+    // The default is OS-dependent (on on windows), ensure it's off.
+    if (ImplicitHeaderGuard)
+      Cmd.push_back("-fno-ms-compatibility");
   }
   Cmd.insert(Cmd.end(), ExtraArgs.begin(), ExtraArgs.end());
   // Put the file name at the end -- this allows the extra arg (-xc++) to


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D64857.210308.patch
Type: text/x-patch
Size: 763 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20190717/07bc228f/attachment.bin>