[PATCH] D28068: Whenever reasonable, merge ASAN quarantine batches to save memory.

[Aleksey Shlyapnikov via Phabricator via llvm-commits]

alekseyshl marked an inline comment as done.
alekseyshl added inline comments.


================
Comment at: lib/sanitizer_common/sanitizer_quarantine.h:38
+    batch[0] = ptr;
+    this->size = size + sizeof(QuarantineBatch);  // Account for the batch size.
+  }
----------------
eugenis wrote:
> would it make the code simpler if we removed sizeof(QuarantineBatch) from here?
I guess, you mean to do not account for the quarantine batch size here and calculate the overhead every time we check whether the quarantine limit is reached? It will simplify the code a little bit, but we will have to perform this calculation on every Put() call or, if we still going to roll it into size_, EnqueueBatch, DequeueBatch and MergeBatches will have to be aware of it (not that great too).


https://reviews.llvm.org/D28068