[Mlir-commits] [mlir] [MLIR] Make `OneShotModuleBufferize` use `OpInterface` (PR #110322)

Andrzej Warzyński llvmlistbot at llvm.org
Sun Oct 20 20:00:37 PDT 2024


banach-space wrote:

>  you can wrap `matmul` and `main` in a payload module (i.e., `module @payload attributes { transform.target_tag = "payload" }` and use the `debug-payload-root-tag` in your `RUN` command. ` RUN: mlir-opt --transform-interpreter="debug-payload-root-tag=payload"`. You may also need to separate your `RUN` command into two `mlir-opt`s similar to this other test after erasing the transform module: 

Thanks for these hints. I noticed that you have also rewritten the pass pipeline to use `-pass-pipeline` and used this nesting `builtin.module(builtin.module(`. That's quite a workaround 😅 

Unfortunately, the steps outlined above weren't sufficient for the SME e2e tests to pass (https://github.com/llvm/llvm-project/pull/113117). I am running out of time and both Matthias and I are travelling this week, hence I am suggesting a revert:
* https://github.com/llvm/llvm-project/pull/113124

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


More information about the Mlir-commits mailing list