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

[Guillaume Chatelet via Phabricator via llvm-commits]

gchatelet 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>
----------------
dmajor wrote:
> 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)`
>I believe the MSVC-ism for this is `defined(_M_IX86) || defined(_M_X64)`

Yes indeed
https://sourceforge.net/p/predef/wiki/Architectures/




Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D96498