[llvm] [AMDGPU] Merge two V_CNDMASK instructions into V_DUAL_CNDMASK (PR #135007)

Jay Foad via llvm-commits llvm-commits at lists.llvm.org
Wed Apr 9 07:00:07 PDT 2025


================
@@ -1459,13 +1482,73 @@ bool SIFoldOperandsImpl::tryConstantFoldOp(MachineInstr *MI) const {
   return false;
 }
 
+bool SIFoldOperandsImpl::shouldSwitchOperands(MachineRegisterInfo &MRI,
+                                              MachineInstr &MI,
+                                              const SIInstrInfo &TII) const {
+  auto allUses = MRI.use_nodbg_operands(MI.getOperand(5).getReg());
----------------
jayfoad wrote:

All variable names should start with an upper case letter.

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


More information about the llvm-commits mailing list