Skip to content

Commit

Permalink
[mlir][linalg] Remove redundant check on linalgOps to fix windows bui…
Browse files Browse the repository at this point in the history
…lder

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
  • Loading branch information
raikonenfnu committed Oct 10, 2022
1 parent 277c382 commit fa58926
Showing 1 changed file with 0 additions and 3 deletions.
3 changes: 0 additions & 3 deletions mlir/lib/Dialect/Linalg/IR/LinalgOps.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -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
Expand Down

0 comments on commit fa58926

Please sign in to comment.