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

[Christian Sigg]

chsigg wrote:

> 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?

This would be quite uncommon, we don't include `.inc` files across dialect boundaries because they are not self-contained (they usually need other includes before them or even some defines).

But I still missed something, the LinalgInterfaces.h.inc is actually exposed in LinalgInterfaces.h, not the top-level Linalg.h. I'm sorry about that, I got confused between RelayoutOpsInterface and RelayoutOps.

I will close this PR and instead restructure the bazel BUILD files to split out a separate LinalgInterfaces target. I think this should be doable.

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