[PATCH] D74098: scudo: Add a dump of primary allocation sizes to malloc_info output.

Peter Collingbourne via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Feb 6 09:19:44 PST 2020


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


================
Comment at: compiler-rt/lib/scudo/standalone/wrappers_c.inc:185
+      decltype(SCUDO_ALLOCATOR)::PrimaryT::SizeClassMap::MaxSize;
+  scudo::uptr sizes[max_size] = {};
+  auto callback = [](uintptr_t, size_t size, void* arg) {
----------------
cryptoad wrote:
> Isn't this going to be huge? 128k*sizeof(uptr), so like 1mb of stack?
Yeah, I'll move it to the heap.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D74098/new/

https://reviews.llvm.org/D74098





More information about the llvm-commits mailing list