[libcxx-commits] [PATCH] D135464: [SystemZ][z/OS] Account for no __cxa_thread_atexit_impl on z/OS

[Louis Dionne via Phabricator via libcxx-commits]

ldionne added a comment.

How is this code going to be correct if there is no thread-local storage? z/OS does support multiple threads, so does this mean that this is now a race condition?



================
Comment at: libcxxabi/src/cxa_thread_atexit.cpp:25-28
 #ifndef HAVE___CXA_THREAD_ATEXIT_IMPL
   // A weak symbol is used to detect this function's presence in the C library
   // at runtime, even if libc++ is built against an older libc
   _LIBCXXABI_WEAK
----------------
Why doesn't that handle the z/OS case?


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

https://reviews.llvm.org/D135464