diff options
author | Matthias Braun <matze@braunis.de> | 2017-01-27 18:53:05 +0000 |
---|---|---|
committer | Matthias Braun <matze@braunis.de> | 2017-01-27 18:53:05 +0000 |
commit | b162af3d1445759b9b3ac49700beec930ad4f7bc (patch) | |
tree | 4de5abfd1c71adcbb66a36bd520030a536cffade /lib/CodeGen/ScheduleDAGInstrs.cpp | |
parent | bb2a4cb623a0d2dc23cd191c1400847553f41b2b (diff) |
ScheduleDAGInstrs: Cleanup toggleKillFlag(); NFC
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@293323 91177308-0d34-0410-b5e6-96231b3b80d8
Diffstat (limited to 'lib/CodeGen/ScheduleDAGInstrs.cpp')
-rw-r--r-- | lib/CodeGen/ScheduleDAGInstrs.cpp | 21 |
1 files changed, 10 insertions, 11 deletions
diff --git a/lib/CodeGen/ScheduleDAGInstrs.cpp b/lib/CodeGen/ScheduleDAGInstrs.cpp index c07e7d7d5df..520a6ff7563 100644 --- a/lib/CodeGen/ScheduleDAGInstrs.cpp +++ b/lib/CodeGen/ScheduleDAGInstrs.cpp @@ -1154,32 +1154,32 @@ static void toggleBundleKillFlag(MachineInstr *MI, unsigned Reg, if ((--End)->addRegisterKilled(Reg, TRI, /* addIfNotFound= */ false)) return; } else - (--End)->clearRegisterKills(Reg, TRI); + (--End)->clearRegisterKills(Reg, TRI); } } -bool ScheduleDAGInstrs::toggleKillFlag(MachineInstr *MI, MachineOperand &MO) { +void ScheduleDAGInstrs::toggleKillFlag(MachineInstr &MI, MachineOperand &MO) { // Setting kill flag... if (!MO.isKill()) { MO.setIsKill(true); - toggleBundleKillFlag(MI, MO.getReg(), true, TRI); - return false; + toggleBundleKillFlag(&MI, MO.getReg(), true, TRI); + return; } // If MO itself is live, clear the kill flag... if (LiveRegs.test(MO.getReg())) { MO.setIsKill(false); - toggleBundleKillFlag(MI, MO.getReg(), false, TRI); - return false; + toggleBundleKillFlag(&MI, MO.getReg(), false, TRI); + return; } // If any subreg of MO is live, then create an imp-def for that // subreg and keep MO marked as killed. MO.setIsKill(false); - toggleBundleKillFlag(MI, MO.getReg(), false, TRI); + toggleBundleKillFlag(&MI, MO.getReg(), false, TRI); bool AllDead = true; const unsigned SuperReg = MO.getReg(); - MachineInstrBuilder MIB(MF, MI); + MachineInstrBuilder MIB(MF, &MI); for (MCSubRegIterator SubRegs(SuperReg, TRI); SubRegs.isValid(); ++SubRegs) { if (LiveRegs.test(*SubRegs)) { MIB.addReg(*SubRegs, RegState::ImplicitDefine); @@ -1189,9 +1189,8 @@ bool ScheduleDAGInstrs::toggleKillFlag(MachineInstr *MI, MachineOperand &MO) { if(AllDead) { MO.setIsKill(true); - toggleBundleKillFlag(MI, MO.getReg(), true, TRI); + toggleBundleKillFlag(&MI, MO.getReg(), true, TRI); } - return false; } void ScheduleDAGInstrs::fixupKills(MachineBasicBlock *MBB) { @@ -1265,7 +1264,7 @@ void ScheduleDAGInstrs::fixupKills(MachineBasicBlock *MBB) { if (MO.isKill() != kill) { DEBUG(dbgs() << "Fixing " << MO << " in "); - toggleKillFlag(&MI, MO); + toggleKillFlag(MI, MO); DEBUG(MI.dump()); DEBUG({ if (MI.getOpcode() == TargetOpcode::BUNDLE) { |