[llvm] r294925 - NewGVN: Use shouldSwapOperands in one more place

[Daniel Berlin via llvm-commits]

Author: dannyb
Date: Sun Feb 12 17:24:42 2017
New Revision: 294925

URL: http://llvm.org/viewvc/llvm-project?rev=294925&view=rev
Log:
NewGVN: Use shouldSwapOperands in one more place

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=294925&r1=294924&r2=294925&view=diff
==============================================================================
--- llvm/trunk/lib/Transforms/Scalar/NewGVN.cpp (original)
+++ llvm/trunk/lib/Transforms/Scalar/NewGVN.cpp Sun Feb 12 17:24:42 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);
   }