[PATCH] D52713: Move llvm util dependencies from clang-tools-extra to add_lit_target.
    Michał Górny via Phabricator via llvm-commits 
    llvm-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 llvm-commits
mailing list