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

Frederich Munch via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun Jul 30 22:01:07 PDT 2017


marsupial added inline comments.


================
Comment at: compiler-rt/trunk/lib/builtins/emutls.c:105
+#include <assert.h>
+#include <immintrin.h>
+
----------------
mstorsjo wrote:
> 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?
It's for the atomic load/store operations arround line 200.


Repository:
  rL LLVM

https://reviews.llvm.org/D30787





More information about the llvm-commits mailing list