[Mlir-commits] [mlir] [mlir][linalg] Create a dedicated target for `LinalgRelayoutInterface` (PR #128485)

[Mehdi Amini]

================

----------------
joker-eph wrote:

> Mechanically, sure. Logically, the dependency still exists.

right, but moving the interface as you suggest wouldn’t change that either.

> I wanted to raise a more general question here whether we need a better place for interfaces with cross-dialect usage

In a case where "logically" we want to teach Tensor dialect about a property exposed by the Linalg dialect, I'm not sure there would be anything to do that wouldn't just be fundamentally "mechanical".

>  Or perhaps the offending folder that creates this dependency should've been moved instead of all these build system tricks.

Right: our problem is that canonicalization and folders aren't flexible enough from this point of view.
A canonicalization for the tensor dialect can be injected by Linalg, but not a folder.


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