[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