[Mlir-commits] [llvm] [mlir] [mlir][tensor][linalg] Move RelayoutOpInterface from linalg back to tensor (PR #127533)

[Christian Sigg]

chsigg wrote:

> > If the interface is in a header that does not require implementation details, then it's a reasonable way to share information without actually requiring one to link all object files unless using the actual dialect.
> 
> It's indeed a header file, but it's TableGen'ed, hence "build system" dependency. I suspect that's the undesired side-effect?

The undesired part is that it's only exposed in the 'top-level' dialect header `Linalg.h`. If it were exposed in a separate header (that Linalg.h is free to include), it would be much easier to handle from a bazel build perspective.

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