[PATCH] D71719: [scudo][standalone] Implement TSD registry disabling

Evgenii Stepanov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Dec 19 11:22:47 PST 2019


eugenis added inline comments.


================
Comment at: compiler-rt/lib/scudo/standalone/tsd_exclusive.h:51
   ALWAYS_INLINE TSD<Allocator> *getTSDAndLock(bool *UnlockRequired) {
-    if (LIKELY(State == ThreadState::Initialized)) {
+    if (LIKELY(State == ThreadState::Initialized && !Disabled)) {
       *UnlockRequired = false;
----------------
is not this a data race?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D71719





More information about the llvm-commits mailing list