[PATCH] D56886: [safestack] Remove dependency of SafeStack on sanitizer_common

Kamil Rytarowski via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jan 17 21:07:37 PST 2019


krytarowski added inline comments.


================
Comment at: compiler-rt/trunk/lib/safestack/safestack.cc:154
   pid_t pid;
-  tid_t tid;
+  pid_t tid;
 };
----------------
Can we please add a dedicated (u64) type for TID? Assuming that PID==TID is linuxism.


Repository:
  rL LLVM

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

https://reviews.llvm.org/D56886





More information about the llvm-commits mailing list