[PATCH] D132052: [BOLT] Emit LSDA call sites for all fragments

Fabian Parzefall via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Aug 24 17:17:39 PDT 2022


FPar marked an inline comment as done.
FPar added inline comments.


================
Comment at: bolt/include/bolt/Core/BinaryFunction.h:1428
+        CallSites.begin(), CallSites.end(), std::make_pair(F, CallSite()),
+        [](const std::pair<FragmentNum, CallSite> &Lhs,
+           const std::pair<FragmentNum, CallSite> &Rhs) {
----------------
Amir wrote:
> Why not using `llvm::less_first` here and below?
Because I was not aware of that function object. Thanks!


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D132052



More information about the llvm-commits mailing list