[PATCH] D37421: [XRay] [test-suite] Add LNT support to retref-bench benchmarks.

Dean Michael Berris via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun Sep 17 23:47:42 PDT 2017


dberris added a comment.

Let's wait for @MatzeB's feedback on this.



================
Comment at: LNTBased/MicroBenchmarks/XRay/CMakeLists.txt:7
+if("${ARCH}" STREQUAL "x86_64" AND "${COMPILER_HAS_FXRAY_INSTRUMENT}")
+  add_subdirectory(benchmark-1.2.0)
+
----------------
This one you probably want to move into either the MicroBenchmarks/CMakeLists.txt file so it gets added if any of the microbenchmarks are built. Then from here you can just add a dependency to the executable besides adding the target_link_libraries.

This should address the multiple build problem you were describing (i.e. for each of the different micro benchmark runs).


https://reviews.llvm.org/D37421





More information about the llvm-commits mailing list