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

Vitaly Buka via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jan 18 14:16:59 PST 2019


vitalybuka marked 2 inline comments as done.
vitalybuka added inline comments.


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


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