[PATCH] D36281: Fix llvm-for-windows-on-linux build after LLVM r272701.
Nico Weber via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Aug 3 13:08:31 PDT 2017
thakis created this revision.
The file is called "intrin.h". When building targeting Windows on a Linux system, with the SDK mounted in a case-insensitive file system, "Intrin.h" will miss clang's intrin.h header (because that's not in a case-insensitive file system) but then find intrin.h in the Microsoft SDK. clang can't handle the SDK's intrin.h.
(I'm not really trying to build llvm-for-windows on linux, but I'm trying to build chrome-for-windows on linux, and parts of the chrome build depend on parts of the llvm build, so I want to fix this upstream.)
https://reviews.llvm.org/D36281
Files:
lib/Support/Atomic.cpp
Index: lib/Support/Atomic.cpp
===================================================================
--- lib/Support/Atomic.cpp
+++ lib/Support/Atomic.cpp
@@ -17,9 +17,9 @@
using namespace llvm;
#if defined(_MSC_VER)
-#include <Intrin.h>
+#include <intrin.h>
-// We must include windows.h after Intrin.h.
+// We must include windows.h after intrin.h.
#include <windows.h>
#undef MemoryFence
#endif
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D36281.109618.patch
Type: text/x-patch
Size: 404 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20170803/d491c7ed/attachment.bin>
More information about the llvm-commits
mailing list