diff options
author | Sergei Barannikov <barannikov88@gmail.com> | 2023-05-01 05:39:30 +0300 |
---|---|---|
committer | Sergei Barannikov <barannikov88@gmail.com> | 2023-05-17 21:51:45 +0300 |
commit | 01a796744745d8413d0821c734caf2fbe19f2eca (patch) | |
tree | a48ea04230b38e6ea69d84fc537668f6eab4cc34 /llvm/lib/Target/AMDGPU/SIISelLowering.cpp | |
parent | dc3069dadf6fd4eece82936fe913dc8310a24cd0 (diff) | |
download | llvm-01a796744745d8413d0821c734caf2fbe19f2eca.tar.gz |
[CodeGen] Replace CCState's getNextStackOffset with getStackSize (NFC)
The term "next stack offset" is misleading because the next argument is
not necessarily allocated at this offset due to alignment constrains.
It also does not make much sense when allocating arguments at negative
offsets (introduced in a follow-up patch), because the returned offset
would be past the end of the next argument.
Reviewed By: arsenm
Differential Revision: https://reviews.llvm.org/D149566
Diffstat (limited to 'llvm/lib/Target/AMDGPU/SIISelLowering.cpp')
-rw-r--r-- | llvm/lib/Target/AMDGPU/SIISelLowering.cpp | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/llvm/lib/Target/AMDGPU/SIISelLowering.cpp b/llvm/lib/Target/AMDGPU/SIISelLowering.cpp index 6fad74caee6c..63176a249bf6 100644 --- a/llvm/lib/Target/AMDGPU/SIISelLowering.cpp +++ b/llvm/lib/Target/AMDGPU/SIISelLowering.cpp @@ -2658,7 +2658,7 @@ SDValue SITargetLowering::LowerFormalArguments( DAG.getPass()->getAnalysis<AMDGPUArgumentUsageInfo>(); ArgUsageInfo.setFuncArgInfo(Fn, Info->getArgInfo()); - unsigned StackArgSize = CCInfo.getNextStackOffset(); + unsigned StackArgSize = CCInfo.getStackSize(); Info->setBytesInStackArgArea(StackArgSize); return Chains.empty() ? Chain : @@ -3114,7 +3114,7 @@ bool SITargetLowering::isEligibleForTailCallOptimization( // If the stack arguments for this call do not fit into our own save area then // the call cannot be made tail. // TODO: Is this really necessary? - if (CCInfo.getNextStackOffset() > FuncInfo->getBytesInStackArgArea()) + if (CCInfo.getStackSize() > FuncInfo->getBytesInStackArgArea()) return false; const MachineRegisterInfo &MRI = MF.getRegInfo(); @@ -3221,7 +3221,7 @@ SDValue SITargetLowering::LowerCall(CallLoweringInfo &CLI, CCInfo.AnalyzeCallOperands(Outs, AssignFn); // Get a count of how many bytes are to be pushed on the stack. - unsigned NumBytes = CCInfo.getNextStackOffset(); + unsigned NumBytes = CCInfo.getStackSize(); if (IsSibCall) { // Since we're not changing the ABI to make this a tail call, the memory |