[llvm] r294919 - testing
Daniel Berlin via llvm-commits
llvm-commits at lists.llvm.org
Sun Feb 12 14:02:20 PST 2017
Author: dannyb
Date: Sun Feb 12 16:02:20 2017
New Revision: 294919
URL: http://llvm.org/viewvc/llvm-project?rev=294919&view=rev
Log:
testing
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=294919&r1=294918&r2=294919&view=diff
==============================================================================
--- llvm/trunk/lib/Transforms/Scalar/NewGVN.cpp (original)
+++ llvm/trunk/lib/Transforms/Scalar/NewGVN.cpp Sun Feb 12 16:02:20 2017
@@ -571,7 +571,7 @@ const Expression *NewGVN::createExpressi
// numbers. Since all commutative instructions have two operands it is more
// efficient to sort by hand rather than using, say, std::sort.
assert(I->getNumOperands() == 2 && "Unsupported commutative instruction!");
- if (E->getOperand(0) > E->getOperand(1))
+ if (shouldSwapOperands(E->getOperand(0), E->getOperand(1)))
E->swapOperands(0, 1);
}
@@ -2425,6 +2425,7 @@ 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