[PATCH] D110271: [llvm-profgen] Fix an out-of-range error during unwinding

Wenlei He via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Sep 22 17:08:42 PDT 2021


wenlei added inline comments.


================
Comment at: llvm/tools/llvm-profgen/PerfReader.cpp:81
+      if (!SameInlinee) {
+        State.switchToFrame(End);
         State.CurrentLeafFrame->recordRangeCount(PrevIP, End, Repeat);
----------------
This is a change beyond refactoring? Before, we use last address of old frame as frame address, but now it's going to be the first address of old frame. 

`State.switchToFrame(LeafAddr);`
->
`State.switchToFrame(End);`

Or do did you actually meant `State.switchToFrame(PrevIP);`?




Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D110271



More information about the llvm-commits mailing list