[Mlir-commits] [llvm] [mlir] [mlir] Move InlinerInterface from Transforms to Interfaces. (PR #84878)

[Christian Sigg]

chsigg wrote:

I see.

Another way to solve the circular dependency is to create a separate bazel target for just InliningUtils. After that, TransformUtils can be merged into Transforms without creating circular dependencies. Sorry for being bazel-centric here. I'm assuming that with CMake the layering is not as strict, but the layering is still not ideal. What is the relationship of MLIRTransform and MLIRTransformUtils?

Would it be better to move the InlinerInterface to a separate file and CMake library that still lives under Transforms/Utils?

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