#5: moved print_configuration to header only file and added its test

kokkos · Feb 14, 2023 · b60e991 · b60e991
1 parent cc11c6d
commit b60e991
Show file tree

Hide file tree

Showing 5 changed files with 2 additions and 112 deletions.
diff --git a/common/CMakeLists.txt b/common/CMakeLists.txt
@@ -7,5 +7,3 @@ LIST(APPEND KK_INCLUDE_DIRS ${CMAKE_CURRENT_SOURCE_DIR}/common/unit_test)
 # Adding unit-tests
 KOKKOSKERNELS_INCLUDE_DIRECTORIES(${CMAKE_CURRENT_BINARY_DIR}/common)
 KOKKOSKERNELS_INCLUDE_DIRECTORIES(REQUIRED_DURING_INSTALLATION_TESTING ${CMAKE_CURRENT_SOURCE_DIR}/common)
-
-LIST(APPEND SOURCES ${CMAKE_CURRENT_SOURCE_DIR}/common/src/kokkoskernels_print_configuration.cpp)
diff --git a/common/src/kokkoskernels_print_configuration.cpp b/common/src/kokkoskernels_print_configuration.cpp
diff --git a/common/src/kokkoskernels_print_configuration.hpp b/common/src/kokkoskernels_print_configuration.hpp
diff --git a/common/unit_test/Test_Common.hpp b/common/unit_test/Test_Common.hpp
@@ -23,5 +23,6 @@
 #include <Test_Common_IOUtils.hpp>
 #include <Test_Common_Error.hpp>
 #include <Test_Common_Version.hpp>
+#include <Test_Common_PrintConfiguration.hpp>
 
 #endif  // TEST_COMMON_HPP
diff --git a/perf_test/Benchmark_Context.hpp b/perf_test/Benchmark_Context.hpp
@@ -42,7 +42,7 @@ std::string remove_unwanted_characters(std::string str) {
   return str.substr(from, to + 1);
 }
 
-/// \brief Extract all key:value pairs from kokkos configuration and add it to
+/// \brief Extract all key:value pairs from kernels configuration and add it to
 /// the benchmark context
 void add_kernels_configuration(bool verbose) {
   std::ostringstream msg;