[Mlir-commits] [mlir] [mlir][transform] Add support for transform.param pad multiples in `PadOp` (PR #90755)

llvmlistbot at llvm.org llvmlistbot at llvm.org
Thu May 2 08:16:32 PDT 2024


================
@@ -1021,8 +1023,7 @@ def PadOp : Op<Transform_Dialect, "structured.pad",
                       TransformHandleTypeInterface:$pad,
                       TransformHandleTypeInterface:$copy);
 
-  let assemblyFormat =
-    "$target attr-dict `:` functional-type(operands, results)";
+  let hasCustomAssemblyFormat = 1;
----------------
srcarroll wrote:

so now `vectorize`, `tile_using_for` and `pad_op` have the same syntax, where the dynamic types are in the functional-type. the `tile_using_forall` op still has the types inlined in the dynamic list. so i guess we should decide, for consistency, which one to go with. and i'll go ahead and make a PR to update all of them.

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


More information about the Mlir-commits mailing list