[all-commits] [llvm/llvm-project] 2f1558: [mlir][transform] Plumb a simplified form of Affin...

[Nicolas Vasilache via All-commits]

  Branch: refs/heads/users/nico/revisit-pad-drop-padding-dimensions
  Home:   https://github.com/llvm/llvm-project
  Commit: 2f1558ae8c1c90a6091dbc821fd5438f5136b8ae
      https://github.com/llvm/llvm-project/commit/2f1558ae8c1c90a6091dbc821fd5438f5136b8ae
  Author: Nicolas Vasilache <nico.vasilache at amd.com>
  Date:   2025-06-22 (Sun, 22 Jun 2025)

  Changed paths:
    M mlir/include/mlir/Dialect/Affine/IR/AffineOps.h
    M mlir/include/mlir/Interfaces/ValueBoundsOpInterface.h
    M mlir/lib/Dialect/Affine/IR/AffineOps.cpp
    M mlir/lib/Dialect/Linalg/Transforms/PadTilingInterface.cpp
    M mlir/lib/Interfaces/ValueBoundsOpInterface.cpp
    M mlir/test/Dialect/Linalg/transform-op-pad-tiling-interface-multiple-of.mlir

  Log Message:
  -----------
  [mlir][transform] Plumb a simplified form of AffineMin folding into transform.pad-tiling-interface

This revision introduces a simple variant of AffineMin folding in makeComposedFoldedAffineApply
and makes use of it in transform.pad-tiling-interface.
Since this version explicitly call ValueBoundsInterface, it may be too expensive and is
only activate behind a flag.
It results in better foldings when mixing tiling and padding, including with dynamic shapes.

This should be further composed with #145068 to provide full simplification and address
the remaining TODO in the test.


  Commit: 512be3adf3bda6baeb41da93a94fd235c1eb39b9
      https://github.com/llvm/llvm-project/commit/512be3adf3bda6baeb41da93a94fd235c1eb39b9
  Author: Nicolas Vasilache <nico.vasilache at amd.com>
  Date:   2025-06-22 (Sun, 22 Jun 2025)

  Changed paths:
    M mlir/include/mlir/Dialect/Linalg/TransformOps/LinalgTransformOps.td
    M mlir/lib/Dialect/Linalg/TransformOps/LinalgTransformOps.cpp
    M mlir/lib/Dialect/Linalg/Transforms/PadTilingInterface.cpp
    M mlir/test/Dialect/Linalg/transform-op-pad-tiling-interface-multiple-of.mlir
    M mlir/test/Dialect/Linalg/transform-op-pad-tiling-interface.mlir

  Log Message:
  -----------
  [mlir][transform] Drop redundant padding_dimensions spec from pad_tiling_interface

This revision aligns padding specification in pad_tiling_interface to that of tiling specification.
Dimensions that should be skipped are specified by "padding by 0".
Trailing dimensions that are ignored are automatically completed to "pad to 0".


Compare: https://github.com/llvm/llvm-project/compare/2f1558ae8c1c%5E...512be3adf3bd

To unsubscribe from these emails, change your notification settings at https://github.com/llvm/llvm-project/settings/notifications