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

Hongtao Yu via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jul 11 16:37:39 PDT 2022


hoy added inline comments.


================
Comment at: llvm/tools/llvm-profgen/ProfiledBinary.h:502
+  const SampleContextFrameVector &
+  getCachedFrameLocation(uint64_t Offset, bool UseProbeDiscriminator = false) {
     auto I = Offset2LocStackMap.emplace(Offset, SampleContextFrameVector());
----------------
nit: getCachedFrameLocation -> getCachedFrameLocationStack

Sorry for missing it previously.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D128859



More information about the llvm-commits mailing list