[PATCH] D12001: Implement __emutls_get_address
Dan Albert via llvm-commits
llvm-commits at lists.llvm.org
Thu Aug 27 20:27:02 PDT 2015
danalbert added inline comments.
================
Comment at: compiler-rt/lib/builtins/CMakeLists.txt:44
@@ -43,2 +43,3 @@
divxc3.c
+ emutls.c
enable_execute_stack.c
----------------
compnerd wrote:
> Please don't add this to GENERIC_SOURCES. This is really not applicable to all targets. It uses pthread, so it can't be used on say Windows.
>
> This makes me wonder if this actually belongs in the compiler builtins in the first place.
GCC keeps it in libgcc, and it's a function that's only ever emitted by the compiler as a replacement for `__thread`. Seems like this is the right place to me. A windows equivalent could be implemented as well, but perhaps it's best to just wrap all this in an `#if !defined(_WIN32)` for now.
http://reviews.llvm.org/D12001
More information about the llvm-commits
mailing list