[all-commits] [llvm/llvm-project] fde34d: sanitizer_common: remove debugging logic from the ...

Dmitry Vyukov via All-commits all-commits at lists.llvm.org
Mon Jul 12 04:02:52 PDT 2021


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: fde34d9f891b92539e0e8eff96057390ee7b1bdf
      https://github.com/llvm/llvm-project/commit/fde34d9f891b92539e0e8eff96057390ee7b1bdf
  Author: Dmitry Vyukov <dvyukov at google.com>
  Date:   2021-07-12 (Mon, 12 Jul 2021)

  Changed paths:
    M compiler-rt/lib/sanitizer_common/sanitizer_allocator.cpp

  Log Message:
  -----------
  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




More information about the All-commits mailing list