commit 2fe8ae4f6cab83b753fded814a0992b0cddf1609
author Jacques Pienaar <jpienaar@google.com> Sat May 04 02:48:57 2019
committer Mehdi Amini <joker.eph@gmail.com> Mon May 06 15:28:20 2019
tree 6e5c7effb9b5bb604347b30f5b99db0325f1137f
parent 4db06d84d8fdb45f025737f0d616bc53bd84ec2e

    Fix up some mixed sign warnings.

--

PiperOrigin-RevId: 246614498

mlir/lib/Transforms/LoopFusion.cpp

diff --git a/mlir/lib/Transforms/LoopFusion.cpp b/mlir/lib/Transforms/LoopFusion.cpp
index f8db53a..fda6574 100644
--- a/mlir/lib/Transforms/LoopFusion.cpp
+++ b/mlir/lib/Transforms/LoopFusion.cpp
@@ -1676,7 +1676,7 @@
                             << "   fused mem: " << fusedMem << "\n"
                             << "   slice mem: " << sliceMemEstimate << "\n");
 
-    if (fusedMem > srcMemSizeVal + dstMemSizeVal) {
+    if (static_cast<long>(fusedMem) > srcMemSizeVal + dstMemSizeVal) {
       LLVM_DEBUG(llvm::dbgs() << "Fusion is not profitable; NOT fusing.\n");
       return false;
     }

mlir/lib/Transforms/Utils/GreedyPatternRewriteDriver.cpp

diff --git a/mlir/lib/Transforms/Utils/GreedyPatternRewriteDriver.cpp b/mlir/lib/Transforms/Utils/GreedyPatternRewriteDriver.cpp
index 8c6a932..3fd44c7 100644
--- a/mlir/lib/Transforms/Utils/GreedyPatternRewriteDriver.cpp
+++ b/mlir/lib/Transforms/Utils/GreedyPatternRewriteDriver.cpp
@@ -53,7 +53,7 @@
 
   /// Perform the rewrites. Return true if the rewrite converges in
   /// `maxIterations`.
-  bool simplifyFunction(unsigned maxIterations);
+  bool simplifyFunction(int maxIterations);
 
   void addToWorklist(Operation *op) {
     // Check to see if the worklist already contains this op.
@@ -146,7 +146,7 @@
 }; // end anonymous namespace
 
 /// Perform the rewrites.
-bool GreedyPatternRewriteDriver::simplifyFunction(unsigned maxIterations) {
+bool GreedyPatternRewriteDriver::simplifyFunction(int maxIterations) {
   Function *fn = builder.getFunction();
   ConstantFoldHelper helper(fn);