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

[Andrzej Warzyński]

banach-space wrote:

> > > 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.
> 
> These two statements are at odds. If it's a table-gen'ed file, it's not `Linalg.h`. I imagine the Tensor side would just include the `.inc` directly?

Yes, that's possible:
* https://github.com/llvm/llvm-project/pull/127541

@chsigg , would that make the Bazel side of things easier for you?

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