[llvm] AMDGPU/NewPM Port SILoadStoreOptimizer to NPM (PR #106362)

via llvm-commits llvm-commits at lists.llvm.org
Thu Aug 29 00:01:22 PDT 2024


================
@@ -2571,3 +2580,19 @@ bool SILoadStoreOptimizer::runOnMachineFunction(MachineFunction &MF) {
 
   return Modified;
 }
+
+PreservedAnalyses
+SILoadStoreOptimizerPass::run(MachineFunction &MF,
+                              MachineFunctionAnalysisManager &MFAM) {
+  MFPropsModifier _(*this, MF);
----------------
paperchalice wrote:

There is no need to use them in legacy pass manager version. In legacy pass manager, `getRequiredProperties` etc. are virtual method and `MachineFunctionProperties` are handled by `MachineFunctionPass`(see `MachineFunctionPass::doInitialization` and `MachineFunctionPass::runOnFunction`).

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


More information about the llvm-commits mailing list