sanitizer_common: remove debugging logic from the internal allocator
authorDmitry Vyukov <dvyukov@google.com>
Sun, 11 Jul 2021 12:51:58 +0000 (14:51 +0200)
committerDmitry Vyukov <dvyukov@google.com>
Mon, 12 Jul 2021 11:02:38 +0000 (13:02 +0200)
commitfde34d9f891b92539e0e8eff96057390ee7b1bdf
tree4f78af300b6e22ee8664352f76fd5f40929d3b7d
parentde59f564400de1b0fe30ae07f3c800562a025e27
sanitizer_common: remove debugging logic from the internal allocator

The internal allocator adds 8-byte header for debugging purposes.
The problem with it is that it's not possible to allocate nicely-sized
objects without a significant overhead. For example, if we allocate
512-byte objects, that will be rounded up to 768 or something.
This logic migrated from tsan where it was added during initial development,
I don't remember that it ever caught anything (we don't do bugs!).
Remove it so that it's possible to allocate nicely-sized objects
without overheads.

Reviewed By: melver

Differential Revision: https://reviews.llvm.org/D105777
compiler-rt/lib/sanitizer_common/sanitizer_allocator.cpp