[Mlir-commits] [mlir] [MLIR][IRDL][CMake] CMake fixes for cross-compilation (PR #145672)

[llvmlistbot at llvm.org]

arthurqiu wrote:

> I have reproduced the problem I was referring to with this patch applied. But the good news is that I also tried having the target depend on both `$MLIR_IRDL_TO_CPP_EXE` and `$MLIR_IRDL_TO_CPP_TARGET` and it seems to not reproduce the problem (because the EXE dependency still retriggers the build). If I understand your problem correctly, it should also solve yours. Please let me know if that works.

It works for me. And this explains why we need both:

https://github.com/llvm/llvm-project/blob/7e3e2e1b8c6ff21e68782a56164139cca334fcf3/mlir/cmake/modules/AddMLIR.cmake#L92-L100

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