[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