[Mlir-commits] [mlir] d5946fd - [mlir][linalg][transform][python] Simplify mix-in of PadOp.

Ingo Müller llvmlistbot at llvm.org
Mon Sep 4 04:30:55 PDT 2023


Author: Ingo Müller
Date: 2023-09-04T11:30:50Z
New Revision: d5946fd3ede076131b811f7133dca9144ac660a2

URL: https://github.com/llvm/llvm-project/commit/d5946fd3ede076131b811f7133dca9144ac660a2
DIFF: https://github.com/llvm/llvm-project/commit/d5946fd3ede076131b811f7133dca9144ac660a2.diff

LOG: [mlir][linalg][transform][python] Simplify mix-in of PadOp.

This patch removes some manual conversion of mixed Python/attribute
arguments to `I64ArrayAttr`s, which turned out to be unnecessary.
Interestingly, this change does not depend on the additional attribute
builders added in the (currently pending)
https://reviews.llvm.org/D159403 patch.

Reviewed By: ftynse

Differential Revision: https://reviews.llvm.org/D159419

Added: 
    

Modified: 
    mlir/python/mlir/dialects/_structured_transform_ops_ext.py

Removed: 
    


################################################################################
diff  --git a/mlir/python/mlir/dialects/_structured_transform_ops_ext.py b/mlir/python/mlir/dialects/_structured_transform_ops_ext.py
index 2d0eeb772862cd..792629457552b6 100644
--- a/mlir/python/mlir/dialects/_structured_transform_ops_ext.py
+++ b/mlir/python/mlir/dialects/_structured_transform_ops_ext.py
@@ -508,9 +508,6 @@ def __init__(
         loc=None,
         ip=None,
     ):
-        padding_dimensions = _get_int_array_attr(padding_dimensions)
-        pad_to_multiple_of = _get_int_array_attr(pad_to_multiple_of)
-        pack_paddings = _get_int_array_attr(pack_paddings)
         transpose_paddings = _get_int_array_array_attr(transpose_paddings)
 
         pdl_operation_type = pdl.OperationType.get()


        


More information about the Mlir-commits mailing list