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

[Andrzej Warzyński]

banach-space wrote:

> > #127544 works for bazel and only has some `#include` changes without touching cmake.
> 
> 
> 
> Let's go with your change, then. 

+1

> @banach-space I think we can close this one without merging, as it already worked with BAZEL for `LinalgInterfaces.h`. 

+1

> You can also keep this as a clean up, not necessarily related to @chsigg PR, but I agree we shouldn't include `.inc` directly like that, as you already do on the interface one.

Yeah, I am a bit concerned about the dependency that we are working around here. I don't have a good solution yet, but this is a bit fragile and I'd like to improve it at some point.

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