This is an archive of the discontinued LLVM Phabricator instance.

Fix llvm-for-windows-on-linux build after LLVM r272701.
ClosedPublic

Authored by thakis on Aug 3 2017, 1:08 PM.

Details

Reviewers
rnk
Summary

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.)

Diff Detail

Event Timeline

thakis created this revision.Aug 3 2017, 1:08 PM
rnk accepted this revision.Aug 3 2017, 1:09 PM

lgtm

This revision is now accepted and ready to land.Aug 3 2017, 1:09 PM
thakis closed this revision.Aug 3 2017, 1:11 PM

309980, thanks!