[Mlir-commits] [mlir] [mlir][transform] Fix handling of transitive include in interpreter. (PR #67560)

Ingo Müller llvmlistbot at llvm.org
Thu Sep 28 00:55:42 PDT 2023


ingomueller-net wrote:

> Implementation wise, we can consider having a "mergeModules" function that takes a list of modules and moves everything into the first one. It can then be exercised from the pass that loads transform library files, but is also reusable.

Yeah, good point! Being able to merge several modules is on my TODO list. That would also be required if we wanted to write a generic merger that could replace most of [SPIR-V's `ModuleCombiner`](https://github.com/llvm/llvm-project/blob/d5c8b23b/mlir/include/mlir/Dialect/SPIRV/Linking/ModuleCombiner.h#L70). But I'll leaver that to that after the remaining pending changes...

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


More information about the Mlir-commits mailing list