[PATCH] D59443: Remove the outdated ThreadLocal class in favor of C++ thread_local.
Roman Lebedev via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Fri Mar 22 10:10:03 PDT 2019
lebedev.ri added inline comments.
================
Comment at: llvm/lib/Support/CrashRecoveryContext.cpp:21
-static ManagedStatic<
- sys::ThreadLocal<const CrashRecoveryContextImpl> > CurrentContext;
+static const thread_local CrashRecoveryContextImpl *CurrentContext;
----------------
resistor wrote:
> lebedev.ri wrote:
> > Are you **sure** `thread_local` is fully supported by all the compilers that are supported for building LLVM?
> > It wasn't supported by xcode for some time, maybe the newest version supports it though.
> `thread_local` support was added in Xcode 8, in 2016: https://asciiwwdc.com/2016/sessions/405#t=354.596
>
> As far as I'm aware, all other supported compiler (GCC, MSVC) had thread_local support earlier than that.
In particular, i would like to see a table of all the compilers that are officially supported by LLVM,
and a yes/no column for `thread_local` support.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D59443/new/
https://reviews.llvm.org/D59443
More information about the llvm-commits
mailing list