From bd529fbb4a678b1a07cf769be71b1a8e41d527ec Mon Sep 17 00:00:00 2001 From: "Duncan P. N. Exon Smith" Date: Sat, 27 Feb 2016 19:23:34 +0000 Subject: [PATCH] CodeGen: Assert valid MI in AntiDepBreaker::UpdateDbgValue 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 | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/llvm/lib/CodeGen/AntiDepBreaker.h b/llvm/lib/CodeGen/AntiDepBreaker.h index 9f05200..2aa7716 100644 --- a/llvm/lib/CodeGen/AntiDepBreaker.h +++ b/llvm/lib/CodeGen/AntiDepBreaker.h @@ -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); } }; -- 2.7.4