[llvm] r329632 - [MemorySSA] Remove redundant assert; NFC

George Burgess IV via llvm-commits llvm-commits at lists.llvm.org
Mon Apr 9 15:45:15 PDT 2018


Author: gbiv
Date: Mon Apr  9 15:45:14 2018
New Revision: 329632

URL: http://llvm.org/viewvc/llvm-project?rev=329632&view=rev
Log:
[MemorySSA] Remove redundant assert; NFC

The `if (!Def && !Use) return nullptr;` right above this assert sort of
defeats the purpose.

Modified:
    llvm/trunk/lib/Analysis/MemorySSA.cpp

Modified: llvm/trunk/lib/Analysis/MemorySSA.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Analysis/MemorySSA.cpp?rev=329632&r1=329631&r2=329632&view=diff
==============================================================================
--- llvm/trunk/lib/Analysis/MemorySSA.cpp (original)
+++ llvm/trunk/lib/Analysis/MemorySSA.cpp Mon Apr  9 15:45:14 2018
@@ -1566,9 +1566,6 @@ MemoryUseOrDef *MemorySSA::createNewAcce
   if (!Def && !Use)
     return nullptr;
 
-  assert((Def || Use) &&
-         "Trying to create a memory access with a non-memory instruction");
-
   MemoryUseOrDef *MUD;
   if (Def)
     MUD = new MemoryDef(I->getContext(), nullptr, I, I->getParent(), NextID++);




More information about the llvm-commits mailing list