[PATCH] D125743: [outliner] Use profile data to avoid outlining hot blocks

Ellis Hoag via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jun 8 19:12:19 PDT 2022


ellis planned changes to this revision.
ellis added a comment.

I'm planning changes to this for now, see the `TODO` comments.



================
Comment at: llvm/lib/CodeGen/MachineOutliner.cpp:379
+    if (UseProfileData) {
+      AU.addRequired<BlockFrequencyInfoWrapperPass>();
+      AU.addRequired<ProfileSummaryInfoWrapperPass>();
----------------
wenlei wrote:
> Why do we require BFI instead of MBFI given this is dealing with MIR.
I would have liked to use `LazyMachineBlockFrequencyInfoPass` instead, but it seems that does not work here because this is a `ModulePass` instead of a `MachineFunctionPass`.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D125743



More information about the llvm-commits mailing list