[PATCH] D39153: [Sanitizers] New sanitizer API to purge allocator quarantine.

Kostya Kortchinsky via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Oct 23 09:07:46 PDT 2017


cryptoad accepted this revision.
cryptoad added a comment.
This revision is now accepted and ready to land.

LGTM with a nit.



================
Comment at: lib/sanitizer_common/sanitizer_quarantine.h:120
     if (cache_.Size() > GetSize() && recycle_mutex_.TryLock())
-      Recycle(cb);
+      Recycle(atomic_load(&min_size_, memory_order_relaxed), cb);
+  }
----------------
Up to you, but maybe use atomic_load_relaxed?


https://reviews.llvm.org/D39153





More information about the llvm-commits mailing list