RPCS3/llvm-mirror
1
0
Fork 0
mirror of https://github.com/RPCS3/llvm-mirror.git synced 2024-11-22 18:54:02 +01:00
Code Issues Projects Releases Wiki Activity

Fix gcc5 build failure (NFC)

Browse Source 
The loop index was shadowing the container name.
It seems that we can just not use a for-range loop here since there is
an induction variable anyway.

Differential Revision: https://reviews.llvm.org/D94254
This commit is contained in:
Mehdi Amini 2021-01-07 19:59:37 +00:00
parent d069b95364
commit af57c8dcb7
1 changed files with 6 additions and 9 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

15
lib/Target/AMDGPU/AMDGPUCallLowering.cpp
View File

@ -330,11 +330,11 @@ void AMDGPUCallLowering::processSplitArgs(
// FIXME: This is mostly nasty pre-processing before handleAssignments. Most
// of this should be performed by handleAssignments.
int SplitIdx = 0;
for (const ArgInfo &SplitArg : SplitArg) {
for (int SplitIdx = 0, e = SplitArg.size(); SplitIdx != e; ++SplitIdx) {
const ArgInfo &CurSplitArg = SplitArg[SplitIdx];
Register Reg = OrigArg.Regs[SplitIdx];
EVT VT = EVT::getEVT(SplitArg.Ty);
LLT LLTy = getLLTForType(*SplitArg.Ty, DL);
EVT VT = EVT::getEVT(CurSplitArg.Ty);
LLT LLTy = getLLTForType(*CurSplitArg.Ty, DL);
unsigned NumParts = TLI.getNumRegistersForCallingConv(Ctx, CallConv, VT);
MVT RegVT = TLI.getRegisterTypeForCallingConv(Ctx, CallConv, VT);
@ -342,9 +342,8 @@ void AMDGPUCallLowering::processSplitArgs(
if (NumParts == 1) {
// No splitting to do, but we want to replace the original type (e.g. [1 x
// double] -> double).
SplitArgs.emplace_back(Reg, SplitArg.Ty, OrigArg.Flags, OrigArg.IsFixed);
++SplitIdx;
SplitArgs.emplace_back(Reg, CurSplitArg.Ty, OrigArg.Flags,
OrigArg.IsFixed);
continue;
}
@ -362,8 +361,6 @@ void AMDGPUCallLowering::processSplitArgs(
}
PerformArgSplit(SplitRegs, Reg, LLTy, PartLLT, SplitIdx);
++SplitIdx;
}
}