aboutsummaryrefslogtreecommitdiff
path: root/llvm/lib/Object/COFFModuleDefinition.cpp
diff options
context:
space:
mode:
authorShoaib Meenai <smeenai@fb.com>2017-12-06 18:33:07 +0000
committerShoaib Meenai <smeenai@fb.com>2017-12-06 18:33:07 +0000
commit6aa13adf0e4effbdcd3b8d55f4622871d3627793 (patch)
tree91f1c10949ca903686454703e6bb20b7e0a3a2bb /llvm/lib/Object/COFFModuleDefinition.cpp
parentf4c32398242723e090bd3047b9b4913407bfc342 (diff)
downloadllvm-6aa13adf0e4effbdcd3b8d55f4622871d3627793.zip
llvm-6aa13adf0e4effbdcd3b8d55f4622871d3627793.tar.gz
llvm-6aa13adf0e4effbdcd3b8d55f4622871d3627793.tar.bz2
[cmake] Remove unnecessary header include in atomics check
The header include was required to work around PR19898, as noted in that comment. That PR has since been marked resolved fixed, and the configuration check passes without the header inclusion both when compiling on Windows with cl and when cross-compiling on Linux using clang-cl. I noticed this because the inclusion was cased incorrectly (Intrin.h instead of intrin.h), which when cross-compiling on a case sensitive file system would cause the intrin.h from the Windows SDK to be included (which LLVM can't handle) instead of the one from clang's resource directory, making the check fail. This is the same issue as r309980. Correcting the case of the inclusion makes the check pass when cross compiling, but it seems better to get rid of the inclusion entirely, since it appears to be unnecessary now. Differential Revision: https://reviews.llvm.org/D40910 llvm-svn: 319917
Diffstat (limited to 'llvm/lib/Object/COFFModuleDefinition.cpp')
0 files changed, 0 insertions, 0 deletions