[compiler-rt] [scudo] Add two missing locks to enable/disable. (PR #79670)
Evgenii Stepanov via llvm-commits
llvm-commits at lists.llvm.org
Mon Jan 29 13:26:39 PST 2024
================
@@ -136,6 +136,14 @@ class StackDepot {
u64 operator[](uptr RingPos) const {
return atomic_load_relaxed(&Ring[RingPos & RingMask]);
}
+
+ void disable() NO_THREAD_SAFETY_ANALYSIS {
+ RingEndMu.lock();
+ }
+
+ void enable() NO_THREAD_SAFETY_ANALYSIS {
+ RingEndMu.unlock();
+ }
----------------
eugenis wrote:
Great point about the BufferPool lock, removed it from the pull request.
Added a comment in StackDepot.
https://github.com/llvm/llvm-project/pull/79670
More information about the llvm-commits
mailing list