[PATCH] D83032: [utils] New script `check_ninja_deps.py`

[Nico Weber via Phabricator via llvm-commits]

thakis accepted this revision.
thakis added inline comments.
This revision is now accepted and ready to land.


================
Comment at: llvm/utils/check_ninja_deps.py:178
+            # stale data from targets that existed only in past builds in the
+            # same directory.
+            if (dep in targets and currtarget in deps and
----------------
This also means you have to make sure that latter deps for a file overwrite earlier deps for a file, iirc (the old entry might be stale)


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D83032/new/

https://reviews.llvm.org/D83032