[libcxx-commits] [PATCH] D127325: Restore missing runtimes-test-depends target that causes build failures when LLVM_INCLUDE_TESTS is ON

[Louis Dionne via Phabricator via libcxx-commits]

ldionne added inline comments.


================
Comment at: runtimes/CMakeLists.txt:210
 if(LLVM_INCLUDE_TESTS)
+  add_custom_target(runtimes-test-depends ${LLVM_RUNTIMES_LIT_DEPENDS})
   # Add a global check rule now that all subdirectories have been traversed
----------------
I can't find any mention of `LLVM_RUNTIMES_LIT_DEPENDS` in the monorepo anymore, so this will always be an empty target. It may "fix" the build in that it won't complain about a missing target anymore, but it doesn't look correct to me. @phosek ?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D127325