[libcxx-commits] [PATCH] D119966: [SystemZ][z/OS] Add OPEN_THREAD version of pthread function call

Louis Dionne via Phabricator via libcxx-commits libcxx-commits at lists.llvm.org
Thu Sep 15 12:59:01 PDT 2022


ldionne requested changes to this revision.
ldionne added a comment.
This revision now requires changes to proceed.

Let's try not add obscure platform-specific `#ifdefs` like that. `pthread_getspecific_d8_np` doesn't even return something on Google, so this is pretty obscure as far as this open source project is concerned.

This whole code block is under `_LIBCPP_HAS_THREAD_API_PTHREAD`, and z/OS doesn't seem to support pthread since it's missing these functions. Instead, z/OS should either provide that API, or you can introduce another "threading back-end" other than the pthread or C11 one.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D119966



More information about the libcxx-commits mailing list