[PATCH] D128474: [BOLT] Support multiple parents for split jump table

Huan Nguyen via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jun 24 00:11:00 PDT 2022


nhuhuan marked 5 inline comments as done.
nhuhuan added inline comments.


================
Comment at: bolt/lib/Core/BinaryContext.cpp:705
+  std::vector<BinaryFunction *> FragmentQueue;
   // Copy the functions back to FragmentsToSkip.
+  FragmentQueue.assign(FragmentsToSkip.begin(), FragmentsToSkip.end());
----------------
rafauler wrote:
> outdated coment
Thanks. I removed UniqueFunctions because it is the same as FragmentsToSkip. Also updated the comment.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D128474



More information about the llvm-commits mailing list