[Mlir-commits] [mlir] [mlir][transform] Fix handling of transitive include in interpreter. (PR #67560)
Oleksandr Alex Zinenko
llvmlistbot at llvm.org
Thu Sep 28 00:30:18 PDT 2023
ftynse wrote:
Approach B makes sense to me. Generally, anything that keeps the interpreter simpler and more focused is likely a good idea. I suppose it should be sufficient to just move the symbols from other files,normally, it is allowed to have multiple declarations as long as there is a single definition + handle name conflicts between _private_ symbols that may have repeated names in different modules. Then we can ran symbol-dce separately to remove unused symbols.
https://github.com/llvm/llvm-project/pull/67560
More information about the Mlir-commits
mailing list