[Mlir-commits] [mlir] [MLIR] Add continuous tiling to TileUsingForOp (PR #82792)

Oleksandr Alex Zinenko llvmlistbot at llvm.org
Thu May 30 01:56:58 PDT 2024


================
@@ -107,6 +107,138 @@ static void emitIsPositiveIndexAssertion(ImplicitLocOpBuilder &b,
       b.getStringAttr("expected strictly positive tile size and divisor"));
 }
 
+FailureOr<StaticContinuousTileSizeSpecification>
+mlir::linalg::computeStaticContinuousTileSizes(TilingInterface op,
+                                               unsigned dimension,
+                                               unsigned targetSize) {
+
+  LinalgOp linalgOp = dyn_cast<LinalgOp>(op.getOperation());
----------------
ftynse wrote:

There's no check that `linalgOp` is non-null here. Given that the code below assumes `op` is a Linalg op, I'd just take a `LinalgOp op` in this function and let the caller do the casting. 

https://github.com/llvm/llvm-project/pull/82792


More information about the Mlir-commits mailing list