[PATCH] D125072: [MachineOutliner] Make getOutliningType partially target-independent

Wang Pengcheng via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sat May 7 02:33:42 PDT 2022


pcwang-thead added inline comments.


================
Comment at: llvm/include/llvm/CodeGen/TargetInstrInfo.h:1944
+public:
+  /// Returns how or if \p MI should be outlined.
+  outliner::InstrType
----------------
We may need to change `\p MI` to `\p MIT` or modify param name.


================
Comment at: llvm/lib/CodeGen/TargetInstrInfo.cpp:1458
+  // This includes MachineBasicBlock references, BlockAddressses,
+  // Constant pool indices, jump table indices, and frame indices.
+  //
----------------
"frame indices" could be excluded since we assert them now. :-)


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D125072



More information about the llvm-commits mailing list