[PATCH] D52713: Move llvm util dependencies from clang-tools-extra to add_lit_target.

Michał Górny via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Nov 21 04:26:13 PST 2018


mgorny added a comment.

I would like to request reverting this. It's established standard within LLVM to explicitly specify the dependencies the particular test suite relies on.

This change causes the `-DLLVM_OCAML_OUT_OF_TREE=1` to start wrongly depending on additional test utility targets, causing building an unnecessary second copy of LLVMSupport.


Repository:
  rL LLVM

https://reviews.llvm.org/D52713





More information about the cfe-commits mailing list