347 only build bench_kernels and bench_kernels_omp for targets that depend on it (fix coverage tests in internal CI)

After merge #355 the internal CI coverage jobs on the main `develop` branch fail due to errors related to the `bench_kernels` and `bench_kernels_omp` library targets. This is odd since those two libraries are only required by three specific performance tests - fma, dot, and reduce - while the coverage jobs rely only on unit tests: thus #355 appears to somehow have laid bare an older issue with the CMake infra, which are fixed in this MR.

Additionally, this MR renames the sequential `fma`, `dot`, and `reduce` performance test binaries to the more-descriptive `fma-sequential`, `dot-sequential`, and `reduce-sequential`, respectively.

Author: anyzelman

tests/performance/CMakeLists.txt

@@ -28,8 +28,14 @@ add_library( bench_kernels_omp OBJECT bench_kernels.c bench_kernels.h )
 target_compile_definitions( bench_kernels_omp PRIVATE BENCH_KERNELS_OPENMP )
 target_link_libraries( bench_kernels PRIVATE test_performance_flags )
 target_link_libraries( bench_kernels_omp PRIVATE test_performance_flags OpenMP::OpenMP_C )
+set_target_properties( bench_kernels PROPERTIES
+	EXCLUDE_FROM_ALL TRUE
+)
+set_target_properties( bench_kernels_omp PROPERTIES
+	EXCLUDE_FROM_ALL TRUE
+)
 
-add_grb_executables( fma fma.cpp $<TARGET_OBJECTS:bench_kernels>
+add_grb_executables( fma-sequential fma.cpp $<TARGET_OBJECTS:bench_kernels>
 	BACKENDS reference NO_BACKEND_NAME
 	ADDITIONAL_LINK_LIBRARIES "rt"
 )
@@ -44,7 +50,7 @@ add_grb_executables( fma-nonblocking fma.cpp $<TARGET_OBJECTS:bench_kernels_omp>
 	ADDITIONAL_LINK_LIBRARIES OpenMP::OpenMP_CXX "rt"
 )
 
-add_grb_executables( reduce reduce.cpp $<TARGET_OBJECTS:bench_kernels>
+add_grb_executables( reduce-sequential reduce.cpp $<TARGET_OBJECTS:bench_kernels>
 	BACKENDS reference NO_BACKEND_NAME
 )
 
@@ -53,7 +59,7 @@ add_grb_executables( reduce-openmp reduce.cpp $<TARGET_OBJECTS:bench_kernels_omp
 	ADDITIONAL_LINK_LIBRARIES OpenMP::OpenMP_CXX
 )
 
-add_grb_executables( dot dot.cpp $<TARGET_OBJECTS:bench_kernels>
+add_grb_executables( dot-sequential dot.cpp $<TARGET_OBJECTS:bench_kernels>
 	BACKENDS reference NO_BACKEND_NAME
 	ADDITIONAL_LINK_LIBRARIES backend_headers_nodefs
 )

tests/performance/performancetests.sh

@@ -150,26 +150,26 @@ if [[ -z $DATASETTORUN && ( -z "$EXPTYPE" || "$EXPTYPE" == "KERNEL" ) ]]; then
 	echo ">>>      [ ]           [x]       Testing semiring axpy versus hardcoded axpy over"
 	echo "                                 100 000 000 doubles"
 	echo " "
-	${TEST_BIN_DIR}/fma 100000000 0 &> ${TEST_OUT_DIR}/fma
-	head -1 ${TEST_OUT_DIR}/fma
-	tail -2 ${TEST_OUT_DIR}/fma
-	egrep 'label|Overall timings|0,' ${TEST_OUT_DIR}/fma | grep -v Outer >> ${TEST_OUT_DIR}/benchmarks
+	${TEST_BIN_DIR}/fma-sequential 100000000 0 &> ${TEST_OUT_DIR}/fma-sequential
+	head -1 ${TEST_OUT_DIR}/fma-sequential
+	tail -2 ${TEST_OUT_DIR}/fma-sequential
+	egrep 'label|Overall timings|0,' ${TEST_OUT_DIR}/fma-sequential | grep -v Outer >> ${TEST_OUT_DIR}/benchmarks
 
 	echo ">>>      [ ]           [x]       Testing monoid reduce versus hardcoded reduce over"
 	echo "                                 100 000 000 doubles"
 	echo " "
-	${TEST_BIN_DIR}/reduce 100000000 0 &> ${TEST_OUT_DIR}/reduce
-	head -1 ${TEST_OUT_DIR}/reduce
-	tail -2 ${TEST_OUT_DIR}/reduce
-	egrep 'label|Overall timings|0,' ${TEST_OUT_DIR}/reduce | grep -v Outer >> ${TEST_OUT_DIR}/benchmarks
+	${TEST_BIN_DIR}/reduce-sequential 100000000 0 &> ${TEST_OUT_DIR}/reduce-sequential
+	head -1 ${TEST_OUT_DIR}/reduce-sequential
+	tail -2 ${TEST_OUT_DIR}/reduce-sequential
+	egrep 'label|Overall timings|0,' ${TEST_OUT_DIR}/reduce-sequential | grep -v Outer >> ${TEST_OUT_DIR}/benchmarks
 
 	echo ">>>      [ ]           [x]       Testing semiring dot product versus its hardcoded variant"
 	echo "                                 over 100 000 000 doubles"
 	echo " "
-	${TEST_BIN_DIR}/dot 100000000 0 &> ${TEST_OUT_DIR}/dot
-	head -1 ${TEST_OUT_DIR}/dot
-	tail -2 ${TEST_OUT_DIR}/dot
-	egrep 'label|Overall timings|0,' ${TEST_OUT_DIR}/dot | grep -v Outer >> ${TEST_OUT_DIR}/benchmarks
+	${TEST_BIN_DIR}/dot-sequential 100000000 0 &> ${TEST_OUT_DIR}/dot-sequential
+	head -1 ${TEST_OUT_DIR}/dot-sequential
+	tail -2 ${TEST_OUT_DIR}/dot-sequential
+	egrep 'label|Overall timings|0,' ${TEST_OUT_DIR}/dot-sequential | grep -v Outer >> ${TEST_OUT_DIR}/benchmarks
 
 	echo ">>>      [ ]           [x]       Testing semiring axpy versus hardcoded axpy over"
 	echo "                                 100 000 000 doubles, using the OpenMP reference backend"