[PATCH] D113492: [llvm-profgen] Fix bug of split range branch sample

Lei Wang via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Nov 11 09:46:48 PST 2021


wlei added inline comments.


================
Comment at: llvm/tools/llvm-profgen/ProfiledBinary.cpp:355
   if (!FuncRange->IsFuncEntry && FuncRange->getFuncName() == RangeSymName)
     FuncRange->IsFuncEntry = true;
 }
----------------
hoy wrote:
> Can the flag be set here based on `FuncRange->Func->Ranges.size() == 1` instead?
This is better place, thanks for the suggestion!


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D113492



More information about the llvm-commits mailing list