[llvm] [AMDGPU][GlobalISel] Add register bank legalization for G_SMIN/G_SMAX/G_UMIN/G_UMAX (PR #159821)

Syadus Sefat via llvm-commits llvm-commits at lists.llvm.org
Wed Oct 1 10:26:07 PDT 2025


================
@@ -1329,7 +1363,13 @@ void RegBankLegalizeHelper::applyMappingTrivial(MachineInstr &MI) {
     B.setInstr(MI);
     for (unsigned i = NumDefs; i < NumOperands; ++i) {
       Register Reg = MI.getOperand(i).getReg();
-      if (MRI.getRegBank(Reg) != RB) {
+      // Helper to check if a register should be skipped for VGPR conversion
+      auto shouldSkipVGPRConversion = [&](Register Reg) {
+        MachineInstr *DefMI = MRI.getVRegDef(Reg);
+        // Skip if defining instruction is implicit_def
+        return DefMI && DefMI->getOpcode() == TargetOpcode::G_IMPLICIT_DEF;
+      };
----------------
mssefat wrote:

Reverted and inserted a fixme comment - please check. 

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


More information about the llvm-commits mailing list