Fixes #480: Compiling example executable for each supported C++ version

* No longer requiring the compiler to support C++14 * Now compiling multiple executables (from the same source), one for each language standard version past C++11 until C++23
eyalroz · Mar 25, 2023 · 165cadd · 165cadd
1 parent 4bc8ba2
commit 165cadd
Show file tree

Hide file tree

Showing 2 changed files with 20 additions and 4 deletions.
diff --git a/examples/CMakeLists.txt b/examples/CMakeLists.txt
@@ -117,10 +117,17 @@ target_link_libraries(vectorAdd_profiled cuda-api-wrappers::nvtx)
 add_executable(manipulate_current_device other/manipulate_current_device.cu)
 add_executable(inclusion_in_two_translation_units other/inclusion_in_two_translation_units/main.cpp other/inclusion_in_two_translation_units/second_tu.cpp)
 target_link_libraries(inclusion_in_two_translation_units cuda-api-wrappers::rtc cuda-api-wrappers::nvtx)
-add_executable(cpp_14 other/cpp_14/main.cpp)
-target_link_libraries(cpp_14 cuda-api-wrappers::rtc cuda-api-wrappers::nvtx)
-set_property(TARGET cpp_14 PROPERTY CXX_STANDARD 14)
-set_property(TARGET cpp_14 PROPERTY CXX_STANDARD_REQUIRED ON)
+foreach(std_version "14" "17" "20" "23")
+	if("cxx_std_${std_version}" IN_LIST CMAKE_CXX_COMPILE_FEATURES)
+		set(tgt "cpp_${std_version}")
+		add_executable(${tgt} other/new_cpp_standard/main.cpp)
+		target_link_libraries(${tgt} cuda-api-wrappers::rtc cuda-api-wrappers::nvtx)
+		set_target_properties(${tgt} PROPERTIES
+			CXX_STANDARD ${std_version}
+			CXX_STANDARD_REQUIRED ON
+			CXX_EXTENSIONS OFF)
+	endif()
+endforeach()
 add_executable(runtime_api_header_collection other/runtime_api_header_collection.cpp)
 add_executable(jitify other/jitify/jitify.cpp)
 set(jitify_headers_target_dir ${CMAKE_CURRENT_BINARY_DIR}/${CMAKE_RUNTIME_OUTPUT_DIRECTORY}/example_headers)

diff --git a/examples/other/cpp_14/main.cpp → examples/other/new_cpp_standard/main.cpp b/examples/other/cpp_14/main.cpp → examples/other/new_cpp_standard/main.cpp
@@ -1,3 +1,12 @@
+/**
+ * @file
+ *
+ * @brief This is a simple program, which uses very little of the CUDA 
+ * C++ API wrappers (but more than nothing); it is intended for checking
+ * whether the wrappers can be compiled using different values for the
+ * C++ language standard, beyond the library's minimum supported 
+ * standard version.
+ */
 #include <cuda/api.hpp>
 #include <cuda/nvtx.hpp>
 #include <cuda/rtc.hpp>