[Mlir-commits] [llvm] [mlir] [mlir] Consider mlir-linalg-ods-gen as a tablegen tool in build (PR #76843)

[Renato Golin]

rengolin wrote:

> > However, it is a necessary component of the build, so it can cause build issues with users of the installed LLVM, and so I think it should not be excluded.
> 
> I don't follow: I understand this tool as a tool internal to Linalg development, when is it needed by a user?

This is a real build order issue, yes, but I agree that this is not a _user_ tool. No one should need to use this beyond generating the ODS tables from the definitions.

What we really need here is a CMake dependency that runs that script every time it changes, so that the Linalg build can pickup the right YAML file, but IIRC, the problem is that you need the Python bindings to be enabled, which is weird.

Perhaps the best way forward is to just move everything to ODS and kill the whole OpDSL story.

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