[PATCH] D143311: [MLGO] Add BB Profile Dump Pass for Regalloc Case

Aiden Grossman via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Feb 6 15:20:28 PST 2023


aidengrossman added inline comments.


================
Comment at: llvm/include/llvm/CodeGen/Passes.h:582
+  // When learning an eviction policy, export basic blocks and profile info
+  FunctionPass *createRegAllocBBProfileDumpPass();
+
----------------
mtrofin wrote:
> It's not really regalloc-tied, maybe `createFinalProfileDumpPass` (so "FinalProfileDump" being the pass)?
I renamed the pass to `MBBProfileDump` and changed all the associated init/creation functions. Also moved all the code into `MBBProfileDump.cpp`.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D143311



More information about the llvm-commits mailing list