[PATCH] D34289: [Sanitizer] Remove CombinedAllocator::Allocate's 'cleared' parameter

Kostya Serebryany via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jun 16 12:35:47 PDT 2017


kcc accepted this revision.
kcc added a comment.
This revision is now accepted and ready to land.

LGTM

Interesting. The intent was to optimize the calloc implementation to avoid memset of freshly mmap-ed regions.
Apparently, this optimization when somewhere else (to asan_allocator.cc? or disappeared?)
Anyway, this looks like NFC, so LGTM


https://reviews.llvm.org/D34289





More information about the llvm-commits mailing list