diff options
author | Sanjoy Das <sanjoy@playingwithpointers.com> | 2016-11-17 07:29:43 +0000 |
---|---|---|
committer | Sanjoy Das <sanjoy@playingwithpointers.com> | 2016-11-17 07:29:43 +0000 |
commit | 43ccb38bb50c052a95f364444bef52c914db82ed (patch) | |
tree | 5aaac183b34915d2cd2924ee6b1b95154cf7ba09 /llvm/lib/CodeGen | |
parent | 4a8fe09040de2dbb7aa865c590a1bb18bf2e4a30 (diff) | |
download | bcm5719-llvm-43ccb38bb50c052a95f364444bef52c914db82ed.tar.gz bcm5719-llvm-43ccb38bb50c052a95f364444bef52c914db82ed.zip |
Delete dead code and add asserts instead; NFC
llvm-svn: 287214
Diffstat (limited to 'llvm/lib/CodeGen')
-rw-r--r-- | llvm/lib/CodeGen/ImplicitNullChecks.cpp | 10 |
1 files changed, 3 insertions, 7 deletions
diff --git a/llvm/lib/CodeGen/ImplicitNullChecks.cpp b/llvm/lib/CodeGen/ImplicitNullChecks.cpp index 31d65e69844..3caa01e8c6d 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 |