[Mlir-commits] [mlir] [mlir] Make MLIR Python sources idempotent (PR #97167)
Stella Laurenzo
llvmlistbot at llvm.org
Mon Jul 1 14:29:00 PDT 2024
stellaraccident wrote:
> Looks good to me. I was initially confused why we were creating a list of paths to use as `DEPENDS` later rather than using `add_dependencies` as we go, but I now see in the CMake documentation that `add_dependencies` is only for top-level targets to depend on other top-level targets, so a list is in fact needed and this PR as-is makes sense.
Yes, cmake is very fiddly for mixed file/target dependencies.
https://github.com/llvm/llvm-project/pull/97167
More information about the Mlir-commits
mailing list