Skip to content

Commit 9bb1e21

Browse files
committed
[AMDGPU] Clean up calls to MachineOperand::setIsDead and friends. NFC.
1 parent ad89eb5 commit 9bb1e21

File tree

6 files changed

+10
-11
lines changed

6 files changed

+10
-11
lines changed

llvm/lib/Target/AMDGPU/GCNDPPCombine.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -711,7 +711,7 @@ bool GCNDPPCombine::combineDPPMov(MachineInstr &MovMI) const {
711711
continue;
712712
}
713713
while (!S.second.empty())
714-
S.first->getOperand(S.second.pop_back_val()).setIsUndef(true);
714+
S.first->getOperand(S.second.pop_back_val()).setIsUndef();
715715
}
716716
}
717717

llvm/lib/Target/AMDGPU/GCNHazardRecognizer.cpp

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1845,13 +1845,13 @@ bool GCNHazardRecognizer::fixShift64HighRegBug(MachineInstr *MI) {
18451845
Amt->setReg(NewAmt);
18461846
Amt->setIsKill(false);
18471847
// We do not update liveness, so verifier may see it as undef.
1848-
Amt->setIsUndef(true);
1848+
Amt->setIsUndef();
18491849
if (OverlappedDst)
18501850
MI->getOperand(0).setReg(NewReg);
18511851
if (OverlappedSrc) {
18521852
Src1->setReg(NewReg);
18531853
Src1->setIsKill(false);
1854-
Src1->setIsUndef(true);
1854+
Src1->setIsUndef();
18551855
}
18561856

18571857
return true;

llvm/lib/Target/AMDGPU/SIFrameLowering.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -328,7 +328,7 @@ void SIFrameLowering::emitEntryFunctionFlatScratchInit(
328328
AMDGPU::FLAT_SCR_HI)
329329
.addReg(FlatScrInitLo, RegState::Kill)
330330
.addImm(8);
331-
LShr->getOperand(3).setIsDead(true); // Mark SCC as dead.
331+
LShr->getOperand(3).setIsDead(); // Mark SCC as dead.
332332
}
333333

334334
// Note SGPRSpill stack IDs should only be used for SGPR spilling to VGPRs, not

llvm/lib/Target/AMDGPU/SIISelLowering.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4282,7 +4282,7 @@ MachineBasicBlock *SITargetLowering::EmitInstrWithCustomInserter(
42824282
const DebugLoc &DL = MI.getDebugLoc();
42834283
MachineInstr *Br = BuildMI(*BB, MI, DL, TII->get(AMDGPU::S_CBRANCH_SCC1))
42844284
.add(MI.getOperand(0));
4285-
Br->getOperand(1).setIsUndef(true); // read undef SCC
4285+
Br->getOperand(1).setIsUndef(); // read undef SCC
42864286
MI.eraseFromParent();
42874287
return BB;
42884288
}

llvm/lib/Target/AMDGPU/SIInstrInfo.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5588,7 +5588,7 @@ emitLoadSRsrcFromVGPRLoop(const SIInstrInfo &TII, MachineRegisterInfo &MRI,
55885588

55895589
// Update Rsrc operand to use the SGPR Rsrc.
55905590
Rsrc.setReg(SRsrc);
5591-
Rsrc.setIsKill(true);
5591+
Rsrc.setIsKill();
55925592

55935593
Register SaveExec = MRI.createVirtualRegister(BoolXExecRC);
55945594
MRI.setSimpleHint(SaveExec, CondReg);

llvm/lib/Target/AMDGPU/SIRegisterInfo.cpp

Lines changed: 4 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -228,7 +228,7 @@ struct SGPRSpillBuilder {
228228
auto I = BuildMI(*MBB, MI, DL, TII.get(NotOpc), ExecReg).addReg(ExecReg);
229229
if (!TmpVGPRLive)
230230
I.addReg(TmpVGPR, RegState::ImplicitDefine);
231-
I->getOperand(2).setIsDead(true); // Mark SCC as dead.
231+
I->getOperand(2).setIsDead(); // Mark SCC as dead.
232232
TRI.buildVGPRSpillLoadStore(*this, TmpVGPRIndex, 0, /*IsLoad*/ false);
233233
}
234234
}
@@ -263,7 +263,7 @@ struct SGPRSpillBuilder {
263263
auto I = BuildMI(*MBB, MI, DL, TII.get(NotOpc), ExecReg).addReg(ExecReg);
264264
if (!TmpVGPRLive)
265265
I.addReg(TmpVGPR, RegState::ImplicitKill);
266-
I->getOperand(2).setIsDead(true); // Mark SCC as dead.
266+
I->getOperand(2).setIsDead(); // Mark SCC as dead.
267267

268268
// Restore active lanes
269269
if (TmpVGPRLive)
@@ -2197,7 +2197,7 @@ void SIRegisterInfo::eliminateFrameIndex(MachineBasicBlock::iterator MI,
21972197

21982198
Register TmpReg = RS->scavengeRegister(RC, MI, 0, !UseSGPR);
21992199
FIOp.setReg(TmpReg);
2200-
FIOp.setIsKill(true);
2200+
FIOp.setIsKill();
22012201

22022202
if ((!FrameReg || !Offset) && TmpReg) {
22032203
unsigned Opc = UseSGPR ? AMDGPU::S_MOV_B32 : AMDGPU::V_MOV_B32_e32;
@@ -2270,8 +2270,7 @@ void SIRegisterInfo::eliminateFrameIndex(MachineBasicBlock::iterator MI,
22702270
.addImm(ST.getWavefrontSizeLog2())
22712271
.addReg(FrameReg);
22722272
if (IsSALU && !LiveSCC)
2273-
Shift.getInstr()->getOperand(3).setIsDead(
2274-
true); // Mark SCC as dead.
2273+
Shift.getInstr()->getOperand(3).setIsDead(); // Mark SCC as dead.
22752274
if (IsSALU && LiveSCC) {
22762275
Register NewDest =
22772276
RS->scavengeRegister(&AMDGPU::SReg_32RegClass, Shift, 0);

0 commit comments

Comments
 (0)