[PATCH] D128859: [llvm-profgen] Do not cache the frame location stack during computing inlined context size

[Hongtao Yu via Phabricator via llvm-commits]

hoy added a comment.

Nice catch!



================
Comment at: llvm/tools/llvm-profgen/ProfiledBinary.h:503
   const SampleContextFrameVector &
   getFrameLocationStack(uint64_t Offset, bool UseProbeDiscriminator = false) {
     auto I = Offset2LocStackMap.emplace(Offset, SampleContextFrameVector());
----------------
How about extend this function with a parameter for caching or not?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D128859