TargetInstrInfo::commuteInstruction() doesn't actually return a new instruction

unless it was requested to with an optional parameter that defaults to false, so
we don't need to handle that case in TwoAddressInstructionPass.

llvm-svn: 175974
This commit is contained in:
Cameron Zwarich 2013-02-23 23:13:28 +00:00
parent 87e7870bee
commit f8a7b64dfc

View File

@ -581,19 +581,9 @@ commuteInstruction(MachineBasicBlock::iterator &mi,
}
DEBUG(dbgs() << "2addr: COMMUTED TO: " << *NewMI);
// If the instruction changed to commute it, update livevar.
if (NewMI != MI) {
if (LV)
// Update live variables
LV->replaceKillInstruction(RegC, MI, NewMI);
if (LIS)
LIS->ReplaceMachineInstrInMaps(MI, NewMI);
MBB->insert(mi, NewMI); // Insert the new inst
MBB->erase(mi); // Nuke the old inst.
mi = NewMI;
DistanceMap.insert(std::make_pair(NewMI, Dist));
}
assert(NewMI == MI &&
"TargetInstrInfo::commuteInstruction() should not return a new "
"instruction unless it was requested.");
// Update source register map.
unsigned FromRegC = getMappedReg(RegC, SrcRegMap);