[PATCH] D107060: [SPEC2017] Add timeit dependencies to validator and bench binaries.

[Michael Kruse via Phabricator via llvm-commits]

Meinersbur added a comment.

Shouldn't the top-level CMakeLists.txt be responsible to build `build-timeit` before building anything? This looks like a recursive problem because `TEST_SUITE_COLLECT_COMPILE_TIME` sets

  set(CMAKE_C_COMPILE_OBJECT "${CMAKE_BINARY_DIR}/tools/timeit --summary <OBJECT>.time ${CMAKE_C_COMPILE_OBJECT}")

but that command line is also used to compile `timeit.c`. How does this even work?


Repository:
  rT test-suite

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

https://reviews.llvm.org/D107060