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

via llvm-commits llvm-commits at lists.llvm.org
Wed Aug 28 06:50:12 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:

Not all passes in CodeGen pipeline set them although some of them should. `MFPropsModifier ` is new pass manager only, we should ensure `PassName(...).run(...)` also set properties correctly. If pass require some properties, just use it or do something manually.

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


More information about the llvm-commits mailing list