GPU CMake: Use FILE GENERATE to generate Default Parameter Headers to track changes and rerun if necessary

Author: davidrohr

GPU/GPUTracking/CMakeLists.txt

@@ -112,6 +112,8 @@ if(GPUCA_OVERRIDE_PARAMETER_FILE)
 else()
     set(GPU_PARAM_JSON ${CMAKE_CURRENT_SOURCE_DIR}/Definitions/Parameters/GPUParameters.csv)
 endif()
+set_property(DIRECTORY APPEND PROPERTY CMAKE_CONFIGURE_DEPENDS "${GPU_PARAM_JSON}")
+
 get_filename_component(GPU_PARAM_JSON_EXT ${GPU_PARAM_JSON} EXT)
 string(TOLOWER "${GPU_PARAM_JSON_EXT}" GPU_PARAM_JSON_EXT)
 if(GPU_PARAM_JSON_EXT STREQUAL .csv)

GPU/GPUTracking/cmake/gpu_param_header_generator.cmake

@@ -80,17 +80,15 @@ function(generate_gpu_param_header GPU_ARCH OUT_HEADER)
         set(TARGET_ARCH ${GPU_ARCH})
     endif()
     file(READ "${GPU_PARAM_JSON}" JSON_CONTENT)
-    set(TMP_HEADER "${OUT_HEADER}.tmp")
-    file(WRITE "${TMP_HEADER}" "#ifndef GPUDEFPARAMETERSDEFAULTS_H\n#define GPUDEFPARAMETERSDEFAULTS_H\n\n")
-    file(APPEND "${TMP_HEADER}" "// This file is auto-generated from gpu_params.json. Do not edit directly.\n")
+    set(TMP_HEADER "#ifndef GPUDEFPARAMETERSDEFAULTS_H\n#define GPUDEFPARAMETERSDEFAULTS_H\n\n")
+    string(APPEND TMP_HEADER "// This file is auto-generated from gpu_params.json. Do not edit directly.\n")
     string(REPLACE "," ";" ARCH_LIST "${TARGET_ARCH}")
-    file(APPEND "${TMP_HEADER}" "// Architectures: ${TARGET_ARCH}\n\n")
-    file(APPEND "${TMP_HEADER}" "#if defined(GPUCA_GPUCODE) && !defined(GPUCA_GPUCODE_GENRTC) && !defined(GPUCA_GPUCODE_NO_LAUNCH_BOUNDS) // Avoid including for RTC generation besides normal include protection.\n\n")
+    string(APPEND TMP_HEADER "// Architectures: ${TARGET_ARCH}\n\n")
+    string(APPEND TMP_HEADER "#if defined(GPUCA_GPUCODE) && !defined(GPUCA_GPUCODE_GENRTC) && !defined(GPUCA_GPUCODE_NO_LAUNCH_BOUNDS) // Avoid including for RTC generation besides normal include protection.\n\n")
 
     # Types
     set(TYPES CORE LB PAR)
     # Per architecture definitions
-    set(_first TRUE)
     generate_macros("${JSON_CONTENT}" TMP_OUTPUT "${TYPES}" "${ARCH_LIST};default;default_cpu" "JSON_ARCHITECTURES")
     list(FIND ARCH_LIST "ALL" do_all_architectures)
     if(ARGC GREATER 2)
@@ -99,31 +97,25 @@ function(generate_gpu_param_header GPU_ARCH OUT_HEADER)
     if(do_all_architectures GREATER -1)
         set(ARCH_LIST ${JSON_ARCHITECTURES})
     endif()
+    string(APPEND TMP_HEADER "#if 0\n")
     foreach(ARCH IN LISTS ARCH_LIST)
-        if(_first)
-            file(APPEND "${TMP_HEADER}" "#if defined(GPUCA_GPUTYPE_${ARCH})\n\n")
-            set(_first FALSE)
-        else()
-            file(APPEND "${TMP_HEADER}" "#elif defined(GPUCA_GPUTYPE_${ARCH})\n\n")
-        endif()
-        file(APPEND "${TMP_HEADER}" ${TMP_OUTPUT_${ARCH}})
+        string(APPEND TMP_HEADER "\n#elif defined(GPUCA_GPUTYPE_${ARCH})\n")
+        string(APPEND TMP_HEADER ${TMP_OUTPUT_${ARCH}})
     endforeach()
-    if(NOT _first)
-        file(APPEND "${TMP_HEADER}" "#else\n#error GPU TYPE NOT SET\n#endif\n")
-    endif()
+    string(APPEND TMP_HEADER "#else\n#error GPU TYPE NOT SET\n#endif\n")
 
     # Default parameters
-    file(APPEND "${TMP_HEADER}" "\n// Default parameters if not defined for the target architecture\n\n")
-    file(APPEND "${TMP_HEADER}" ${TMP_OUTPUT_default})
-    file(APPEND "${TMP_HEADER}" "#endif // defined(GPUCA_GPUCODE) && !defined(GPUCA_GPUCODE_GENRTC) && !defined(GPUCA_GPUCODE_NO_LAUNCH_BOUNDS)\n\n")
+    string(APPEND TMP_HEADER "\n// Default parameters if not defined for the target architecture\n\n")
+    string(APPEND TMP_HEADER ${TMP_OUTPUT_default})
+    string(APPEND TMP_HEADER "#endif // defined(GPUCA_GPUCODE) && !defined(GPUCA_GPUCODE_GENRTC) && !defined(GPUCA_GPUCODE_NO_LAUNCH_BOUNDS)\n\n")
 
     # CPU fallback
-    file(APPEND "${TMP_HEADER}" "#ifndef GPUCA_GPUCODE_GENRTC // Defaults for non-LB parameters also for CPU fallback\n\n")
-    file(APPEND "${TMP_HEADER}" ${TMP_OUTPUT_default_cpu})
-    file(APPEND "${TMP_HEADER}" "\n#endif // GPUCA_GPUCODE_GENRTC\n")
+    string(APPEND TMP_HEADER "#ifndef GPUCA_GPUCODE_GENRTC // Defaults for non-LB parameters also for CPU fallback\n\n")
+    string(APPEND TMP_HEADER ${TMP_OUTPUT_default_cpu})
+    string(APPEND TMP_HEADER "\n#endif // GPUCA_GPUCODE_GENRTC\n")
 
-    file(APPEND "${TMP_HEADER}" "\n#endif // GPUDEFPARAMETERSDEFAULTS_H\n")
-    file(RENAME "${TMP_HEADER}" "${OUT_HEADER}")
+    string(APPEND TMP_HEADER "\n#endif // GPUDEFPARAMETERSDEFAULTS_H\n")
+    file(GENERATE OUTPUT "${OUT_HEADER}" CONTENT "${TMP_HEADER}")
     message(STATUS "Generated ${OUT_HEADER}")
     add_custom_target(GPU_PARAM_HEADER_${GPU_ARCH}_ALL ALL DEPENDS ${OUT_HEADER} ${CMAKE_CURRENT_SOURCE_DIR}/cmake/gpu_param_header_generator.cmake ${GPU_PARAM_JSON})
 endfunction()