[libcxx-commits] [PATCH] D138461: [libcxxabi] Always use thread_local for cxa_exception_storage

Shoaib Meenai via Phabricator via libcxx-commits libcxx-commits at lists.llvm.org
Mon Nov 21 18:40:06 PST 2022


smeenai added a comment.

In D138461#3942614 <https://reviews.llvm.org/D138461#3942614>, @MaskRay wrote:

> LGTM. Clang has some targets that do `TLSSupported = false` but they are not supported by libc++.

Do those targets support emulated TLS? Android didn't support ELF TLS until fairly recently, for example, so the `thread_local` path would end up using emulated TLS (which is using pthread under the hood), but this saves libc++abi from having to worry about those details or manually do the pthread equivalent.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D138461/new/

https://reviews.llvm.org/D138461



More information about the libcxx-commits mailing list