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

Jeff Niu via llvm-commits llvm-commits at lists.llvm.org
Tue Mar 12 18:13:49 PDT 2024


Mogball wrote:

+1 to keeping transform specific interfaces in the TransformUtils library. The `mlir/Interfaces` directory, as Mehdi says, I think definitely reflects stronger and more first-class elements of the IR, like control-flow. I don't understand how this creates a circular dependency. What dialect depends on TransformUtils that Transform also depends on? You can always create an `InlinerInterface` library that lives under `Transform/Utils`.

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


More information about the llvm-commits mailing list