[llvm] r294920 - NewGVN: Fix missed call that should be to shouldSwapOperands

Daniel Berlin via llvm-commits llvm-commits at lists.llvm.org
Sun Feb 12 14:02:47 PST 2017


Author: dannyb
Date: Sun Feb 12 16:02:47 2017
New Revision: 294920

URL: http://llvm.org/viewvc/llvm-project?rev=294920&view=rev
Log:
NewGVN: Fix missed call that should be to shouldSwapOperands

Modified:
    llvm/trunk/lib/Transforms/Scalar/NewGVN.cpp

Modified: llvm/trunk/lib/Transforms/Scalar/NewGVN.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Transforms/Scalar/NewGVN.cpp?rev=294920&r1=294919&r2=294920&view=diff
==============================================================================
--- llvm/trunk/lib/Transforms/Scalar/NewGVN.cpp (original)
+++ llvm/trunk/lib/Transforms/Scalar/NewGVN.cpp Sun Feb 12 16:02:47 2017
@@ -2425,7 +2425,6 @@ bool NewGVN::eliminateInstructions(Funct
 // we will simplify an operation with all constants so that it doesn't matter
 // what order they appear in.
 unsigned int NewGVN::getRank(const Value *V) const {
-  assert(V && "What?");
   if (isa<Constant>(V))
     return 0;
   else if (auto *A = dyn_cast<Argument>(V))




More information about the llvm-commits mailing list