The existing logic has a workaround where configure_file is used to write a single dummy file output many times.
CMake has a feature to more directly add the dependency and avoid the dummy file (it is available in the minimum version specified).
Differential D80218
Use configure depends to trigger reconfiguration when LLVMBuild files change danielframpton on May 19 2020, 8:42 AM. Authored by
Details The existing logic has a workaround where configure_file is used to write a single dummy file output many times. CMake has a feature to more directly add the dependency and avoid the dummy file (it is available in the minimum version specified).
Diff Detail
Event TimelineComment Actions Looks good to me. I presume we didn't use this because of CMake version dependencies. Was this feature available in our current minimum CMake version? Comment Actions It was added in CMake 3.0.0, and we currently require 3.4.3. The code I am changing here dates from 2011. Comment Actions This seems to be failing on the Windows pre-merge test run. Unfortunately, I don't have access to those logs and so I can't merge this until this issue is fixed: https://github.com/google/llvm-premerge-checks/issues/187 |