[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