[PATCH] D96498: Fix exegesis build on aarch64-windows-msvc host

dmajor via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Feb 11 07:02:37 PST 2021


dmajor added inline comments.


================
Comment at: llvm/tools/llvm-exegesis/lib/X86/Target.cpp:29
 #include <vector>
-#if defined(_MSC_VER)
+#if defined(_MSC_VER) && (__i386__ || __x86_64__)
 #include <immintrin.h>
----------------
Is it supported to compile this with real MSVC? I don't think this would evaluate to true there. I believe the MSVC-ism for this is `defined(_M_IX86) || defined(_M_X64)`


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D96498



More information about the llvm-commits mailing list