[all-commits] [llvm/llvm-project] 5f4f92: [mlir][transform][gpu][python] Add MapForallToBloc...

Ingo Müller via All-commits all-commits at lists.llvm.org
Thu Jul 20 07:21:04 PDT 2023


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 5f4f9220f9d2f0163b330e4a2e2a60ea8a917378
      https://github.com/llvm/llvm-project/commit/5f4f9220f9d2f0163b330e4a2e2a60ea8a917378
  Author: Ingo Müller <ingomueller at google.com>
  Date:   2023-07-20 (Thu, 20 Jul 2023)

  Changed paths:
    M mlir/python/CMakeLists.txt
    A mlir/python/mlir/dialects/_gpu_transform_ops_ext.py
    A mlir/test/python/dialects/transform_gpu_ext.py

  Log Message:
  -----------
  [mlir][transform][gpu][python] Add MapForallToBlocks mix-in.

This patch adds a mix-in class for MapForallToBlocks with overloaded
constructors. This makes it optional to provide the return type of the
op, which is defaulte to `AnyOpType`.

Reviewed By: ftynse

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


  Commit: f62cb13fb285bfce1135be73afb9d2f1c731012b
      https://github.com/llvm/llvm-project/commit/f62cb13fb285bfce1135be73afb9d2f1c731012b
  Author: Ingo Müller <ingomueller at google.com>
  Date:   2023-07-20 (Thu, 20 Jul 2023)

  Changed paths:
    M mlir/include/mlir/Dialect/Transform/IR/TransformOps.td

  Log Message:
  -----------
  [mlir][linalg][transform] Rename ApplyPatternsOp.{region => patterns}.

This gives the region a more meaningful name. The topic came up in a
discussion on https://reviews.llvm.org/D155435, where the name `region`
would have led to a situation where a convenience accessor called
`region` (after the ODS name) would have returned a Block.

Reviewed By: ftynse

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


  Commit: 4f30746ca006f91daa6e84d126c11545ecaf195e
      https://github.com/llvm/llvm-project/commit/4f30746ca006f91daa6e84d126c11545ecaf195e
  Author: Ingo Müller <ingomueller at google.com>
  Date:   2023-07-20 (Thu, 20 Jul 2023)

  Changed paths:
    M mlir/python/mlir/dialects/_transform_ops_ext.py
    M mlir/test/python/dialects/transform.py

  Log Message:
  -----------
  [mlir][transform][python] Add extended ApplyPatternsOp.

This patch adds a mixin for ApplyPatternsOp to _transform_ops_ext.py
with syntactic sugar for construction such ops. Curiously, the op did
not have any constructors yet, probably because its tablegen definition
said to skip the default builders. The new constructor is thus quite
straightforward. The commit also adds a refined `region` property which
returns the first block of the single region.

Reviewed By: ftynse

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


Compare: https://github.com/llvm/llvm-project/compare/53c936a67c15...4f30746ca006


More information about the All-commits mailing list