[Mlir-commits] [mlir] fa58926 - [mlir][linalg] Remove redundant check on linalgOps to fix windows builder

[Stanley Winata]

Author: Stanley Winata
Date: 2022-10-10T13:19:18-07:00
New Revision: fa58926d7cff53ecd737ea9fdc5d12900c016667

URL: https://github.com/llvm/llvm-project/commit/fa58926d7cff53ecd737ea9fdc5d12900c016667
DIFF: https://github.com/llvm/llvm-project/commit/fa58926d7cff53ecd737ea9fdc5d12900c016667.diff

LOG: [mlir][linalg] Remove redundant check on linalgOps to fix windows builder

One of the assertion is causing signed/unsigned mismatch. However this
assertion seems redundant and is no longer used.

Reviewed By: mravishankar, ThomasRaoux

Differential Revision: https://reviews.llvm.org/D135612

Added: 
    

Modified: 
    mlir/lib/Dialect/Linalg/IR/LinalgOps.cpp

Removed: 
    


################################################################################
diff  --git a/mlir/lib/Dialect/Linalg/IR/LinalgOps.cpp b/mlir/lib/Dialect/Linalg/IR/LinalgOps.cpp
index eaa192358f11..5dbec1ea80d5 100644
--- a/mlir/lib/Dialect/Linalg/IR/LinalgOps.cpp
+++ b/mlir/lib/Dialect/Linalg/IR/LinalgOps.cpp
@@ -1048,9 +1048,6 @@ struct DeduplicateAndRemoveDeadOperandsAndResults
           genericOp.getMatchingIndexingMap(outputOpOperand.value());
       auto key = std::make_tuple(outputOpOperand.value()->get(), indexingMap,
                                  yieldOp->getOperand(outputOpOperand.index()));
-      assert(static_cast<std::size_t>(genericOp.getNumOutputs()) >=
-                 outputOpOperand.index() &&
-             "Output op idx greater than number of outputs.");
       if (isResultValueDead(genericOp, result)) {
         // Check if the opoperand can be dropped without affecting loop
         // bound computation. Add the operand to the list of dropped op