[PATCH] D42214: libcxx: Move Windows threading support into a .cpp file.

Martin Storsjö via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sat Jan 20 09:50:33 PST 2018


mstorsjo added inline comments.


================
Comment at: libcxx/include/__threading_support:87
+typedef void* __libcpp_recursive_mutex_t[6];
+#elif defined(_M_AMD64)
+typedef void* __libcpp_recursive_mutex_t[5];
----------------
mstorsjo wrote:
> `|| defined(_M_ARM64)` here as well - there, it has the same size (40 bytes) as on x86_64.
Oh, also - please add checks for corresponding GCC defines, to keep things working for clang in mingw mode. (`__aarch64__` etc)


https://reviews.llvm.org/D42214





More information about the llvm-commits mailing list