[llvm] [AMDGPU][True16][CodeGen] update wmm reg sorting check condition (PR #135053)

via llvm-commits llvm-commits at lists.llvm.org
Thu Apr 10 06:53:58 PDT 2025


llvmbot wrote:


<!--LLVM PR SUMMARY COMMENT-->

@llvm/pr-subscribers-backend-amdgpu

Author: Brox Chen (broxigarchen)

<details>
<summary>Changes</summary>

We currently just need to shift down 32bit wmm registers. 

Update check condition to skip the 16bit register in wmm reg sorting

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


1 Files Affected:

- (modified) llvm/lib/Target/AMDGPU/SIFrameLowering.cpp (+1-1) 


``````````diff
diff --git a/llvm/lib/Target/AMDGPU/SIFrameLowering.cpp b/llvm/lib/Target/AMDGPU/SIFrameLowering.cpp
index 9c737b4f3e378..8f488f5154650 100644
--- a/llvm/lib/Target/AMDGPU/SIFrameLowering.cpp
+++ b/llvm/lib/Target/AMDGPU/SIFrameLowering.cpp
@@ -1650,7 +1650,7 @@ void SIFrameLowering::determineCalleeSaves(MachineFunction &MF,
     // are of 32-bit size. SIPreAllocateWWMRegs pass can add tuples into WWM
     // reserved registers.
     const TargetRegisterClass *RC = TRI->getPhysRegBaseClass(Reg);
-    if (TRI->getRegSizeInBits(*RC) > 32)
+    if (TRI->getRegSizeInBits(*RC) != 32)
       continue;
     SortedWWMVGPRs.push_back(Reg);
   }

``````````

</details>


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


More information about the llvm-commits mailing list