[llvm] Remove constant local variable (PR #83850)
Martin Wehking via llvm-commits
llvm-commits at lists.llvm.org
Tue Mar 5 10:15:29 PST 2024
https://github.com/MartinWehking updated https://github.com/llvm/llvm-project/pull/83850
>From cd4895c7c4630601718eebc9aed25841e0a590c8 Mon Sep 17 00:00:00 2001
From: Martin Wehking <martin.wehking at codeplay.com>
Date: Mon, 4 Mar 2024 14:23:55 +0000
Subject: [PATCH 1/2] Remove constant local variable
Remove isThisReturn which has always the value false
Replace its uses with false directly.
---
llvm/lib/Target/AMDGPU/SIISelLowering.cpp | 4 +---
1 file changed, 1 insertion(+), 3 deletions(-)
diff --git a/llvm/lib/Target/AMDGPU/SIISelLowering.cpp b/llvm/lib/Target/AMDGPU/SIISelLowering.cpp
index 0df0b5cdf0f392..9739dee6ba591b 100644
--- a/llvm/lib/Target/AMDGPU/SIISelLowering.cpp
+++ b/llvm/lib/Target/AMDGPU/SIISelLowering.cpp
@@ -3599,7 +3599,6 @@ SDValue SITargetLowering::LowerCall(CallLoweringInfo &CLI,
bool &IsTailCall = CLI.IsTailCall;
bool IsVarArg = CLI.IsVarArg;
bool IsSibCall = false;
- bool IsThisReturn = false;
MachineFunction &MF = DAG.getMachineFunction();
if (Callee.isUndef() || isNullConstant(Callee)) {
@@ -3892,8 +3891,7 @@ SDValue SITargetLowering::LowerCall(CallLoweringInfo &CLI,
// Handle result values, copying them out of physregs into vregs that we
// return.
return LowerCallResult(Chain, InGlue, CallConv, IsVarArg, Ins, DL, DAG,
- InVals, IsThisReturn,
- IsThisReturn ? OutVals[0] : SDValue());
+ InVals, false, SDValue());
}
// This is identical to the default implementation in ExpandDYNAMIC_STACKALLOC,
>From f0f1e5061a334cecc1d0eaf8f7fc2b98cb7ed0c3 Mon Sep 17 00:00:00 2001
From: Martin Wehking <martin.wehking at codeplay.com>
Date: Tue, 5 Mar 2024 18:13:42 +0000
Subject: [PATCH 2/2] Name the parameter value
Stick to the LLVM coding standards.
---
llvm/lib/Target/AMDGPU/SIISelLowering.cpp | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/llvm/lib/Target/AMDGPU/SIISelLowering.cpp b/llvm/lib/Target/AMDGPU/SIISelLowering.cpp
index 9739dee6ba591b..5f6ce12ae1fbab 100644
--- a/llvm/lib/Target/AMDGPU/SIISelLowering.cpp
+++ b/llvm/lib/Target/AMDGPU/SIISelLowering.cpp
@@ -3891,7 +3891,7 @@ SDValue SITargetLowering::LowerCall(CallLoweringInfo &CLI,
// Handle result values, copying them out of physregs into vregs that we
// return.
return LowerCallResult(Chain, InGlue, CallConv, IsVarArg, Ins, DL, DAG,
- InVals, false, SDValue());
+ InVals, /*IsThisReturn=*/false, SDValue());
}
// This is identical to the default implementation in ExpandDYNAMIC_STACKALLOC,
More information about the llvm-commits
mailing list