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

Akshat Oke via llvm-commits llvm-commits at lists.llvm.org
Thu Aug 29 02:53:12 PDT 2024


https://github.com/Akshat-Oke created https://github.com/llvm/llvm-project/pull/106520

None

>From 7976458857815df9dc15482a3c4b3a3b02f41681 Mon Sep 17 00:00:00 2001
From: Akshat Oke <Akshat.Oke at amd.com>
Date: Thu, 29 Aug 2024 09:52:06 +0000
Subject: [PATCH] [AMDGPU][NewPM] Have consistent property changes in
 GCNDPPCombine

---
 llvm/lib/Target/AMDGPU/GCNDPPCombine.cpp | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

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();



More information about the llvm-commits mailing list