[llvm] [AMDGPU][NFC] Simplify conditional return in SIInstrInfo.cpp (PR #222426)

Akash Dutta via llvm-commits llvm-commits at lists.llvm.org
Wed Sep 9 12:00:54 PDT 2026


https://github.com/akadutta created https://github.com/llvm/llvm-project/pull/222426

None

>From 5fc1f8dc0b64c91cabf178dbf8e56d8b5ba2ee2a Mon Sep 17 00:00:00 2001
From: Akash Dutta <137309513+akadutta at users.noreply.github.com>
Date: Wed, 9 Sep 2026 13:58:15 -0500
Subject: [PATCH] [AMDGPU][NFC] Simplify conditional return in SIInstrInfo.cpp

---
 llvm/lib/Target/AMDGPU/SIInstrInfo.cpp | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

diff --git a/llvm/lib/Target/AMDGPU/SIInstrInfo.cpp b/llvm/lib/Target/AMDGPU/SIInstrInfo.cpp
index 29a22a324447e..dd324962989e7 100644
--- a/llvm/lib/Target/AMDGPU/SIInstrInfo.cpp
+++ b/llvm/lib/Target/AMDGPU/SIInstrInfo.cpp
@@ -11541,9 +11541,7 @@ static bool foldableSelect(const MachineInstr &Def) {
       Def.getOperand(1).isImm() && Def.getOperand(1).getImm() != 0;
   bool Op2IsZeroImm =
       Def.getOperand(2).isImm() && Def.getOperand(2).getImm() == 0;
-  if (!Op1IsNonZeroImm || !Op2IsZeroImm)
-    return false;
-  return true;
+  return Op1IsNonZeroImm && Op2IsZeroImm;
 }
 
 static bool setsSCCIfResultIsZero(const MachineInstr &Def, bool &NeedInversion,



More information about the llvm-commits mailing list