[PATCH] D120430: [memprof] Symbolize and cache stack frames.

Snehasish Kumar via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Feb 23 12:44:10 PST 2022


snehasish created this revision.
snehasish added a reviewer: davidxl.
Herald added a subscriber: hiraditya.
snehasish requested review of this revision.
Herald added a project: LLVM.
Herald added a subscriber: llvm-commits.

Currently, symbolization of stack frames occurs on demand when the instrprof writer
iterates over all the records in the raw memprof reader. With this
change we symbolize and cache the frames immediately after reading the
raw profiles. For a large internal binary this results in a runtime
reduction of ~50% (2m -> 48s) when merging a memprof raw profile with a
raw instr profile to generate an indexed profile. This change also makes
it simpler in the future to generate additional calling context
metadata to attach to each memprof record.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D120430

Files:
  llvm/include/llvm/ProfileData/MemProf.h
  llvm/include/llvm/ProfileData/RawMemProfReader.h
  llvm/lib/ProfileData/RawMemProfReader.cpp
  llvm/unittests/ProfileData/MemProfTest.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D120430.410914.patch
Type: text/x-patch
Size: 7027 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20220223/7744cd7a/attachment.bin>


More information about the llvm-commits mailing list