[PATCH] D111749: [llvm-profgen] Also ignore the first external branch

Wenlei He via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Oct 13 12:36:13 PDT 2021


wenlei added inline comments.


================
Comment at: llvm/tools/llvm-profgen/PerfReader.cpp:493
     if (IsExternal) {
-      if (PrevTrDst)
+      if (PrevTrDst || LBRStack.empty())
         continue;
----------------
As we discussed, we shouldn't relax this for CS profile - it can't handle leading external address during unwinding.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D111749



More information about the llvm-commits mailing list