[PATCH] D151891: Allow configuring BOLT runtime instrumentation library max. allocation size

[Rafael Auler via Phabricator via llvm-commits]

rafauler added inline comments.


================
Comment at: bolt/runtime/instr.cpp:1473
 
   BumpPtrAllocator Alloc;
   const uint8_t *FuncDesc = Ctx.FuncDescriptions;
----------------
Kobzol wrote:
> rafauler wrote:
> > Note: This is the only arena that does not call setMaxSize, so that's the one you're parameterizing. It is used to write function profile.
> Hmm, I wonder, since the arena size situation is a bit complex, maybe a simpler solution would just be to set this specific arena to 100 MiB too?
Yes, that makes sense


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D151891