[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