From fa58926d7cff53ecd737ea9fdc5d12900c016667 Mon Sep 17 00:00:00 2001 From: Stanley Winata Date: Mon, 10 Oct 2022 13:12:41 -0700 Subject: [PATCH] [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 --- mlir/lib/Dialect/Linalg/IR/LinalgOps.cpp | 3 --- 1 file changed, 3 deletions(-) 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(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 -- 2.7.4