[libcxx-commits] [PATCH] D91875: [SystemZ][ZOS] Fix the usage of pthread_t within libc++

Louis Dionne via Phabricator via libcxx-commits libcxx-commits at lists.llvm.org
Fri Nov 20 11:31:23 PST 2020


ldionne added a comment.

Properly separating `__libcpp_thread_t` from `__libcpp_thread_id` seems like a conceptual "bugfix" to me, so I like it.

However, POSIX says of `pthread_t`:

> Used to identify a thread.

It's not much, but I would expect `pthread_t` to look like a thread ID of some sort, like something you can compare. Are you sure the `pthread_t` on zOS is conforming?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D91875



More information about the libcxx-commits mailing list