[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