[llvm] 3113bfb - AMDGPU/GlobalISel: Use getSubRegFromChannel (#100732)
via llvm-commits
llvm-commits at lists.llvm.org
Fri Jul 26 07:35:38 PDT 2024
Author: Matt Arsenault
Date: 2024-07-26T18:35:35+04:00
New Revision: 3113bfbfa7c6247dae2ae52d90b0487f7e31f33e
URL: https://github.com/llvm/llvm-project/commit/3113bfbfa7c6247dae2ae52d90b0487f7e31f33e
DIFF: https://github.com/llvm/llvm-project/commit/3113bfbfa7c6247dae2ae52d90b0487f7e31f33e.diff
LOG: AMDGPU/GlobalISel: Use getSubRegFromChannel (#100732)
Added:
Modified:
llvm/lib/Target/AMDGPU/AMDGPUInstructionSelector.cpp
Removed:
################################################################################
diff --git a/llvm/lib/Target/AMDGPU/AMDGPUInstructionSelector.cpp b/llvm/lib/Target/AMDGPU/AMDGPUInstructionSelector.cpp
index 63048c7d1a0a1..aaa292291334c 100644
--- a/llvm/lib/Target/AMDGPU/AMDGPUInstructionSelector.cpp
+++ b/llvm/lib/Target/AMDGPU/AMDGPUInstructionSelector.cpp
@@ -2169,27 +2169,6 @@ bool AMDGPUInstructionSelector::selectG_SELECT(MachineInstr &I) const {
return Ret;
}
-static int sizeToSubRegIndex(unsigned Size) {
- switch (Size) {
- case 32:
- return AMDGPU::sub0;
- case 64:
- return AMDGPU::sub0_sub1;
- case 96:
- return AMDGPU::sub0_sub1_sub2;
- case 128:
- return AMDGPU::sub0_sub1_sub2_sub3;
- case 256:
- return AMDGPU::sub0_sub1_sub2_sub3_sub4_sub5_sub6_sub7;
- default:
- if (Size < 32)
- return AMDGPU::sub0;
- if (Size > 256)
- return -1;
- return sizeToSubRegIndex(llvm::bit_ceil(Size));
- }
-}
-
bool AMDGPUInstructionSelector::selectG_TRUNC(MachineInstr &I) const {
Register DstReg = I.getOperand(0).getReg();
Register SrcReg = I.getOperand(1).getReg();
@@ -2293,8 +2272,9 @@ bool AMDGPUInstructionSelector::selectG_TRUNC(MachineInstr &I) const {
return false;
if (SrcSize > 32) {
- int SubRegIdx = sizeToSubRegIndex(DstSize);
- if (SubRegIdx == -1)
+ unsigned SubRegIdx =
+ DstSize < 32 ? AMDGPU::sub0 : TRI.getSubRegFromChannel(0, DstSize / 32);
+ if (SubRegIdx == AMDGPU::NoSubRegister)
return false;
// Deal with weird cases where the class only partially supports the subreg
More information about the llvm-commits
mailing list