[PATCH] D39244: [sanitizer] Random shuffling of chunks for the 32-bit Primary Allocator

Kostya Kortchinsky via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Oct 24 12:14:50 PDT 2017


cryptoad added inline comments.


================
Comment at: lib/sanitizer_common/sanitizer_allocator_primary32.h:273
+    char padding[kCacheLineSize - 2 * sizeof(uptr) -
                  sizeof(IntrusiveList<TransferBatch>)];
   };
----------------
alekseyshl wrote:
> How about using [kCacheLineSize - offsetof(SizeClassInfo, padding)] instead?
I can't seem to make that general idea to work one way or another.
The other possibility that some sanitizer parts use is just char padding[kCacheLinesize].


https://reviews.llvm.org/D39244





More information about the llvm-commits mailing list