[mlir][linalg] Remove redundant check on linalgOps to fix windows builder
authorStanley Winata <stanley@nod-labs.com>
Mon, 10 Oct 2022 20:12:41 +0000 (13:12 -0700)
committerStanley Winata <stanley@nod-labs.com>
Mon, 10 Oct 2022 20:19:18 +0000 (13:19 -0700)
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

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

index eaa1923..5dbec1e 100644 (file)
@@ -1048,9 +1048,6 @@ private:
           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