From: Stanley Winata <stanley@nod-labs.com>
Date: Mon, 10 Oct 2022 20:12:41 +0000 (-0700)
Subject: [mlir][linalg] Remove redundant check on linalgOps to fix windows builder
X-Git-Tag: upstream/17.0.6~31042
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=fa58926d7cff53ecd737ea9fdc5d12900c016667;p=platform%2Fupstream%2Fllvm.git

[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
---

diff --git a/mlir/lib/Dialect/Linalg/IR/LinalgOps.cpp b/mlir/lib/Dialect/Linalg/IR/LinalgOps.cpp
index eaa1923..5dbec1e 100644
--- a/mlir/lib/Dialect/Linalg/IR/LinalgOps.cpp
+++ b/mlir/lib/Dialect/Linalg/IR/LinalgOps.cpp
@@ -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