[PATCH] D64183: [NewPM] Port MachineModuleInfo to the new pass manager.

Matt Arsenault via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jul 15 10:31:23 PDT 2019


arsenm added inline comments.


================
Comment at: llvm/lib/CodeGen/IfConversion.cpp:361
+        MF, TII, ST.getRegisterInfo(),
+        &getAnalysisIfAvailable<MachineModuleInfoWrapperPass>()->getMMI());
   }
----------------
Won't this crash if MachineModuleInfoWrapperPass is not available?


================
Comment at: llvm/lib/CodeGen/IfConversion.cpp:502
+        MF, TII, MF.getSubtarget().getRegisterInfo(),
+        &getAnalysisIfAvailable<MachineModuleInfoWrapperPass>()->getMMI());
   }
----------------
Ditto


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D64183





More information about the llvm-commits mailing list