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

[Renato Golin]

rengolin wrote:

> What's your interpretation of that paragraph? Is there a better mechanism to disable that pattern for Linalg Ops?

Previously, the tensor dialect transforms (on pack and unpack) depended on a lot more than an interface, as it lowered those ops to linalg ops and therefore needed to bring also implementation and everything else with it.

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.

So the question is: why is this showing on @chsigg build? I'm assuming you're not using linalg, but still got linker errors, even though the interface should not have done that. So, if you can show us the error, we can fix the dependency in the right way.

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