[Mlir-commits] [llvm] [mlir] Fix CMake dependencies on mlir-linalg-ods-yaml-gen (PR #111973)

[Martin Storsjö]

mstorsjo wrote:

> I'm guessing it does not. I could change the variable to not be a cache variable. @mstorsjo Is there a reason why get_host_tool_path() sets the executable and target variables as cache? It seems to me to be build system internals.

Earlier, if users wanted to use a prebuilt e.g. `llvm-tblgen` (e.g. when cross compiling), they'd set `-DLLVM_TBLGEN=/path/to/binary` - so I think that's why those variables traditionally were cache variables. Since introducing `LLVM_NATIVE_TOOL_DIR` there's less reason to set those individually, but I guess we haven't officially deprecated/removed the old way of doing that.

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