[libcxx-commits] [PATCH] D121647: [runtimes] Detect changes to Tests.cmake

[Petr Hosek via Phabricator via libcxx-commits]

phosek marked an inline comment as done.
phosek added inline comments.


================
Comment at: llvm/runtimes/CMakeLists.txt:225
+      # This file will be later replaced by the runtimes subbuild.
+      file(TOUCH ${LLVM_BINARY_DIR}/runtimes/Tests.cmake)
     endif()
----------------
phosek wrote:
> ldionne wrote:
> > I'm a bit confused -- doesn't this mean that we're going to include an empty `Tests.cmake` file and hence all the `LLVM_RUNTIMES_LIT_TESTSUITES` & friends below will expand to nothing?
> Yes, this is just so we can include this file unconditionally below which is only necessary during the clean build. We could also use `include(... OPTIONAL)` but in my experience that didn't result in `Tests.cmake` being properly tracked by Ninja until you re-run CMake.
Turned out this is no longer need with `BYPRODUCTS`.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D121647