[PATCH] D112012: [benchmarks] Move libcxx's fork of google/benchmark and llvm/utils' under third-party

[Mircea Trofin via Phabricator via llvm-commits]

mtrofin marked 2 inline comments as done.
mtrofin added inline comments.


================
Comment at: llvm/CMakeLists.txt:1001
 
+add_llvm_external_project(third-party)
+
----------------
ldionne wrote:
> I don't understand why this is needed.
not needed


================
Comment at: llvm/cmake/modules/AddLLVM.cmake:1538
   set_property(TARGET ${benchmark_name} PROPERTY FOLDER "Utils")
+  add_dependencies(${benchmark_name} benchmark)
   target_link_libraries(${benchmark_name} PRIVATE benchmark)
----------------
ldionne wrote:
> I don't understand why this is needed.
I thought it was needed to make sure the library is built, but I suppose the line below addresses that.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D112012