[PATCH] D30787: [builtins] Implement emulated TLS on Windows.

[Martin Storsjö via Phabricator via llvm-commits]

mstorsjo added inline comments.


================
Comment at: compiler-rt/trunk/lib/builtins/emutls.c:105
+#include <assert.h>
+#include <immintrin.h>
+
----------------
Was there any specific reason for including `immintrin.h` here? I don't see anything below that would require it. It breaks cross compilation for windows on non-x86 architectures (ARM), and simply skipping the include fixes the build, so I don't think it's needed on x86 either?


Repository:
  rL LLVM

https://reviews.llvm.org/D30787