[llvm] [AMDGPU] Drop const from return types (NFC) (PR #145640)
Kazu Hirata via llvm-commits
llvm-commits at lists.llvm.org
Tue Jun 24 23:01:05 PDT 2025
https://github.com/kazutakahirata created https://github.com/llvm/llvm-project/pull/145640
We don't need const on these return types.
>From d90ce7e28c7e49cc2c70f4443984cd9193f715ec Mon Sep 17 00:00:00 2001
From: Kazu Hirata <kazu at google.com>
Date: Tue, 24 Jun 2025 21:49:26 -0700
Subject: [PATCH] [AMDGPU] Drop const from return types (NFC)
We don't need const on these return types.
---
llvm/lib/Target/AMDGPU/AMDGPURegBankLegalizeHelper.cpp | 9 +++------
llvm/lib/Target/AMDGPU/AMDGPURegBankLegalizeHelper.h | 6 +++---
2 files changed, 6 insertions(+), 9 deletions(-)
diff --git a/llvm/lib/Target/AMDGPU/AMDGPURegBankLegalizeHelper.cpp b/llvm/lib/Target/AMDGPU/AMDGPURegBankLegalizeHelper.cpp
index b2ddc6e88966b..6a59a28b1d32c 100644
--- a/llvm/lib/Target/AMDGPU/AMDGPURegBankLegalizeHelper.cpp
+++ b/llvm/lib/Target/AMDGPU/AMDGPURegBankLegalizeHelper.cpp
@@ -171,8 +171,7 @@ void RegBankLegalizeHelper::lowerVccExtToSel(MachineInstr &MI) {
MI.eraseFromParent();
}
-const std::pair<Register, Register>
-RegBankLegalizeHelper::unpackZExt(Register Reg) {
+std::pair<Register, Register> RegBankLegalizeHelper::unpackZExt(Register Reg) {
auto PackedS32 = B.buildBitcast(SgprRB_S32, Reg);
auto Mask = B.buildConstant(SgprRB_S32, 0x0000ffff);
auto Lo = B.buildAnd(SgprRB_S32, PackedS32, Mask);
@@ -180,16 +179,14 @@ RegBankLegalizeHelper::unpackZExt(Register Reg) {
return {Lo.getReg(0), Hi.getReg(0)};
}
-const std::pair<Register, Register>
-RegBankLegalizeHelper::unpackSExt(Register Reg) {
+std::pair<Register, Register> RegBankLegalizeHelper::unpackSExt(Register Reg) {
auto PackedS32 = B.buildBitcast(SgprRB_S32, Reg);
auto Lo = B.buildSExtInReg(SgprRB_S32, PackedS32, 16);
auto Hi = B.buildAShr(SgprRB_S32, PackedS32, B.buildConstant(SgprRB_S32, 16));
return {Lo.getReg(0), Hi.getReg(0)};
}
-const std::pair<Register, Register>
-RegBankLegalizeHelper::unpackAExt(Register Reg) {
+std::pair<Register, Register> RegBankLegalizeHelper::unpackAExt(Register Reg) {
auto PackedS32 = B.buildBitcast(SgprRB_S32, Reg);
auto Lo = PackedS32;
auto Hi = B.buildLShr(SgprRB_S32, PackedS32, B.buildConstant(SgprRB_S32, 16));
diff --git a/llvm/lib/Target/AMDGPU/AMDGPURegBankLegalizeHelper.h b/llvm/lib/Target/AMDGPU/AMDGPURegBankLegalizeHelper.h
index 50bd86dc15a1f..08cc7d43bd78e 100644
--- a/llvm/lib/Target/AMDGPU/AMDGPURegBankLegalizeHelper.h
+++ b/llvm/lib/Target/AMDGPU/AMDGPURegBankLegalizeHelper.h
@@ -111,9 +111,9 @@ class RegBankLegalizeHelper {
SmallSet<Register, 4> &SgprWaterfallOperandRegs);
void lowerVccExtToSel(MachineInstr &MI);
- const std::pair<Register, Register> unpackZExt(Register Reg);
- const std::pair<Register, Register> unpackSExt(Register Reg);
- const std::pair<Register, Register> unpackAExt(Register Reg);
+ std::pair<Register, Register> unpackZExt(Register Reg);
+ std::pair<Register, Register> unpackSExt(Register Reg);
+ std::pair<Register, Register> unpackAExt(Register Reg);
void lowerUnpackBitShift(MachineInstr &MI);
void lowerV_BFE(MachineInstr &MI);
void lowerS_BFE(MachineInstr &MI);
More information about the llvm-commits
mailing list