[Mlir-commits] [mlir] [MLIR][Transform] FuseOp: accept transform params, add use_forall argument (PR #161883)

[Rolf Morel]

================
@@ -157,9 +158,10 @@ def __init__(
         self,
         target: Union[Operation, Value, OpView],
         *,
-        tile_sizes: Optional[Union[DynamicIndexList, ArrayAttr]] = None,
-        tile_interchange: OptionalIntList = None,
+        tile_sizes: Optional[MixedValues] = None,
+        tile_interchange: Optional[MixedValues] = None,
         apply_cleanup: Optional[bool] = False,
+        use_forall: Optional[bool] = False,
----------------
rolfmorel wrote:

I see it was already the case for `apply_cleanup` though is there a benefit to marking these flags as optional if the default is already `False` (i.e., why not have the type be just `bool`)?

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