[all-commits] [llvm/llvm-project] 9ccf01: [mlir][transform] Support for multiple top-level t...

martin-luecke via All-commits all-commits at lists.llvm.org
Fri Oct 20 02:42:34 PDT 2023


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 9ccf01fbf711f4f9b0aa82d5732d778ea169fa88
      https://github.com/llvm/llvm-project/commit/9ccf01fbf711f4f9b0aa82d5732d778ea169fa88
  Author: martin-luecke <martin.luecke at ed.ac.uk>
  Date:   2023-10-20 (Fri, 20 Oct 2023)

  Changed paths:
    M mlir/include/mlir/Dialect/Transform/IR/TransformInterfaces.h
    M mlir/lib/Dialect/Transform/Transforms/TransformInterpreterPassBase.cpp
    A mlir/test/Dialect/Transform/test-interpreter-multiple-top-level-ops.mlir
    M mlir/test/lib/Dialect/Transform/TestTransformDialectInterpreter.cpp

  Log Message:
  -----------
  [mlir][transform] Support for multiple top-level transform ops (#69615)

This adds a flag to the `TransformDialectInterpreter` that relaxes the
requirement for only a single top-level transform op.
This is useful for supporting transforms that take transform IR as
payload.

This also aligns the function `findTopLevelTransform`
[here](https://github.com/llvm/llvm-project/commit/7b0f4c9db55c355bffddf94d7710f40ee2c1e9db#diff-551f92bb609487ccf981daf9571f0f1b1703ab2330560a388a5f0d133e520be4L59)
with its documentation:
In the presence of multiple top-level transform ops it now correctly
returns the first of them after reporting the error instead of returning
a `nullptr`.




More information about the All-commits mailing list