[llvm] [NewPM][CodeGen] Port MachineLICM to NPM (PR #107376)

via llvm-commits llvm-commits at lists.llvm.org
Thu Sep 5 05:27:43 PDT 2024


================
@@ -352,6 +386,11 @@ bool MachineLICMBase::runOnMachineFunction(MachineFunction &MF) {
   MRI = &MF.getRegInfo();
   SchedModel.init(&ST);
 
+  // FIXME: Remove this assignment or convert to an assert?
+  // MachineLICM and PostRAMachineLICM were distinguished by introducing
+  // EarlyMachineLICM and MachineLICM respectively to avoid "using an unreliable
+  // MRI::isSSA() check to determine whether register allocation has happened"
+  // (See 4a7c8e7).
   PreRegAlloc = MRI->isSSA();
----------------
paperchalice wrote:

What about `MachineFunctionProperties::Property::IsSSA` or `MachineFunctionProperties::Property::NoVRegs`?

https://github.com/llvm/llvm-project/pull/107376


More information about the llvm-commits mailing list