Skip to content

Commit 0d71b3e

Browse files
authored
[CodeGen] Remove unused argument from getCoveringSubRegIndexes. NFC. (#122884)
1 parent 30f9a4f commit 0d71b3e

File tree

6 files changed

+9
-10
lines changed

6 files changed

+9
-10
lines changed

llvm/include/llvm/CodeGen/TargetRegisterInfo.h

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -426,8 +426,7 @@ class TargetRegisterInfo : public MCRegisterInfo {
426426
///
427427
/// If this is possible, returns true and appends the best matching set of
428428
/// indexes to \p Indexes. If this is not possible, returns false.
429-
bool getCoveringSubRegIndexes(const MachineRegisterInfo &MRI,
430-
const TargetRegisterClass *RC,
429+
bool getCoveringSubRegIndexes(const TargetRegisterClass *RC,
431430
LaneBitmask LaneMask,
432431
SmallVectorImpl<unsigned> &Indexes) const;
433432

llvm/lib/CodeGen/InitUndef.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -161,7 +161,7 @@ bool InitUndef::handleSubReg(MachineFunction &MF, MachineInstr &MI,
161161
});
162162

163163
SmallVector<unsigned> SubRegIndexNeedInsert;
164-
TRI->getCoveringSubRegIndexes(*MRI, TargetRegClass, NeedDef,
164+
TRI->getCoveringSubRegIndexes(TargetRegClass, NeedDef,
165165
SubRegIndexNeedInsert);
166166

167167
// It's not possible to create the INIT_UNDEF when there is no register

llvm/lib/CodeGen/SplitKit.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -568,7 +568,7 @@ SlotIndex SplitEditor::buildCopy(Register FromReg, Register ToReg,
568568
SmallVector<unsigned, 8> SubIndexes;
569569

570570
// Abort if we cannot possibly implement the COPY with the given indexes.
571-
if (!TRI.getCoveringSubRegIndexes(MRI, RC, LaneMask, SubIndexes))
571+
if (!TRI.getCoveringSubRegIndexes(RC, LaneMask, SubIndexes))
572572
report_fatal_error("Impossible to implement partial COPY");
573573

574574
SlotIndex Def;

llvm/lib/CodeGen/TargetRegisterInfo.cpp

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -556,8 +556,8 @@ TargetRegisterInfo::getRegSizeInBits(Register Reg,
556556
}
557557

558558
bool TargetRegisterInfo::getCoveringSubRegIndexes(
559-
const MachineRegisterInfo &MRI, const TargetRegisterClass *RC,
560-
LaneBitmask LaneMask, SmallVectorImpl<unsigned> &NeededIndexes) const {
559+
const TargetRegisterClass *RC, LaneBitmask LaneMask,
560+
SmallVectorImpl<unsigned> &NeededIndexes) const {
561561
SmallVector<unsigned, 8> PossibleIndexes;
562562
unsigned BestIdx = 0;
563563
unsigned BestCover = 0;

llvm/lib/CodeGen/VirtRegMap.cpp

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -660,9 +660,9 @@ void VirtRegRewriter::rewrite() {
660660

661661
// TODO: Just use one super register def if none of the lanes
662662
// are needed?
663-
if (!TRI->getCoveringSubRegIndexes(
664-
*MRI, MRI->getRegClass(VirtReg), LiveOutUndefLanes,
665-
CoveringIndexes))
663+
if (!TRI->getCoveringSubRegIndexes(MRI->getRegClass(VirtReg),
664+
LiveOutUndefLanes,
665+
CoveringIndexes))
666666
llvm_unreachable(
667667
"cannot represent required subregister defs");
668668

llvm/lib/Target/AMDGPU/SIFormMemoryClauses.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -366,7 +366,7 @@ bool SIFormMemoryClauses::runOnMachineFunction(MachineFunction &MF) {
366366

367367
SmallVector<unsigned> KilledIndexes;
368368
bool Success = TRI->getCoveringSubRegIndexes(
369-
*MRI, MRI->getRegClass(Reg), KilledMask, KilledIndexes);
369+
MRI->getRegClass(Reg), KilledMask, KilledIndexes);
370370
(void)Success;
371371
assert(Success && "Failed to find subregister mask to cover lanes");
372372
for (unsigned SubReg : KilledIndexes) {

0 commit comments

Comments
 (0)