[PATCH] D31774: [sanitizer] Introduce tid_t as a typedef for OS-provided thread IDs
Kuba (Brecka) Mracek via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Apr 6 10:46:21 PDT 2017
kubamracek added inline comments.
================
Comment at: lib/sanitizer_common/sanitizer_stoptheworld.h:21
namespace __sanitizer {
-typedef int SuspendedThreadID;
----------------
fjricci wrote:
> Will it be problematic to move `SuspendedThreadID` from `int` to `uptr` on Linux?
I don't see any problems besides using a little bit more space in SuspendedThreadsList below.
I guess we could #ifdef `tid_t` to `int`/`pid_t` on Linux, because that's what `gettid` returns.
Repository:
rL LLVM
https://reviews.llvm.org/D31774
More information about the llvm-commits
mailing list