[Mlir-commits] [mlir] [mlir][linalg] Enable fuse consumer (PR #85528)

Oleksandr Alex Zinenko llvmlistbot at llvm.org
Thu Apr 18 04:12:59 PDT 2024


================
@@ -96,6 +96,25 @@ def TilingInterface : OpInterface<"TilingInterface"> {
           return failure();
         }]
       >,
+      InterfaceMethod<
+        /*desc=*/[{
+          Method to return the position of iteration domain tile computed by the
+          tiled operation.
+        }],
+        /*retType=*/"LogicalResult",
+        /*methodName=*/"getIterationDomainTileFromOperandTile",
+        /*args=*/(ins
+          "OpBuilder &":$b,
+          "unsigned":$operandNumber,
+          "ArrayRef<OpFoldResult> ":$offsets,
+          "ArrayRef<OpFoldResult> ":$sizes,
+          "SmallVector<OpFoldResult> &":$iterDomainOffsets,
----------------
ftynse wrote:

I see that this is cargo-culted from the method above, but we should be using `SmallVectorImpl` to avoid hardcoding the implicit number of stack elements. `SmallVector` is in fact `SmallVector<6>` in this case.

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


More information about the Mlir-commits mailing list