From: Sanjoy Das <sanjoy@playingwithpointers.com>
Date: Thu, 17 Nov 2016 07:29:43 +0000 (+0000)
Subject: Delete dead code and add asserts instead; NFC
X-Git-Tag: llvmorg-4.0.0-rc1~4315
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=43ccb38bb50c052a95f364444bef52c914db82ed;p=platform%2Fupstream%2Fllvm.git

Delete dead code and add asserts instead; NFC

llvm-svn: 287214
---

diff --git a/llvm/lib/CodeGen/ImplicitNullChecks.cpp b/llvm/lib/CodeGen/ImplicitNullChecks.cpp
index 31d65e6..3caa01e 100644
--- a/llvm/lib/CodeGen/ImplicitNullChecks.cpp
+++ b/llvm/lib/CodeGen/ImplicitNullChecks.cpp
@@ -266,15 +266,11 @@ bool HazardDetector::isSafeToHoist(MachineInstr *MI,
             return true;
           if (MO.isUse())
             return false;
-          assert((!MO.isDef() || RegDefs.count(MO.getReg())) &&
+          assert(MO.isDef() &&
+                 "Register MachineOperands must either be uses or be defs.");
+          assert(RegDefs.count(MO.getReg()) &&
                  "All defs must be tracked in RegDefs by now!");
 
-          if (!MO.isDef()) {
-            // FIXME: This is unnecessary, we should be able to
-            // assert(MO.isDef()) here.
-            return true;
-          }
-
           for (unsigned Reg : RegUses)
             if (TRI.regsOverlap(Reg, MO.getReg()))
               return false; // We found a write-after-read