CodeGen: Assert valid MI in AntiDepBreaker::UpdateDbgValue
authorDuncan P. N. Exon Smith <dexonsmith@apple.com>
Sat, 27 Feb 2016 19:23:34 +0000 (19:23 +0000)
committerDuncan P. N. Exon Smith <dexonsmith@apple.com>
Sat, 27 Feb 2016 19:23:34 +0000 (19:23 +0000)
This already assumes a valid MI, since it dereferences the MI in an
assertion before checking for null.  At an explicit assert.

llvm-svn: 262144

llvm/lib/CodeGen/AntiDepBreaker.h

index 9f05200..2aa7716 100644 (file)
@@ -56,8 +56,9 @@ public:
   /// Update DBG_VALUE if dependency breaker is updating
   /// other machine instruction to use NewReg.
   void UpdateDbgValue(MachineInstr *MI, unsigned OldReg, unsigned NewReg) {
+    assert(MI && "Expected valid instruction");
     assert (MI->isDebugValue() && "MI is not DBG_VALUE!");
-    if (MI && MI->getOperand(0).isReg() && MI->getOperand(0).getReg() == OldReg)
+    if (MI->getOperand(0).isReg() && MI->getOperand(0).getReg() == OldReg)
       MI->getOperand(0).setReg(NewReg);
   }
 };