[llvm] [AMDGPU][NewPM] Have consistent property changes in GCNDPPCombine (PR #106520)

via llvm-commits llvm-commits at lists.llvm.org
Mon Sep 2 22:05:32 PDT 2024


llvmbot wrote:


<!--LLVM PR SUMMARY COMMENT-->

@llvm/pr-subscribers-backend-amdgpu

Author: Akshat Oke (Akshat-Oke)

<details>
<summary>Changes</summary>



---
Full diff: https://github.com/llvm/llvm-project/pull/106520.diff


1 Files Affected:

- (modified) llvm/lib/Target/AMDGPU/GCNDPPCombine.cpp (+2-2) 


``````````diff
diff --git a/llvm/lib/Target/AMDGPU/GCNDPPCombine.cpp b/llvm/lib/Target/AMDGPU/GCNDPPCombine.cpp
index 3e1a79062ff0cc..833cd6dfa4ba2e 100644
--- a/llvm/lib/Target/AMDGPU/GCNDPPCombine.cpp
+++ b/llvm/lib/Target/AMDGPU/GCNDPPCombine.cpp
@@ -795,11 +795,11 @@ bool GCNDPPCombine::run(MachineFunction &MF) {
 
 PreservedAnalyses GCNDPPCombinePass::run(MachineFunction &MF,
                                          MachineFunctionAnalysisManager &) {
+  MFPropsModifier _(*this, MF);
+
   if (MF.getFunction().hasOptNone())
     return PreservedAnalyses::all();
 
-  MFPropsModifier _(*this, MF);
-
   bool Changed = GCNDPPCombine().run(MF);
   if (!Changed)
     return PreservedAnalyses::all();

``````````

</details>


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


More information about the llvm-commits mailing list