[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