[llvm] [DLCov 1/5] Add CMake option to enable enhanced line number coverage tracking (PR #107278)

NAKAMURA Takumi via llvm-commits llvm-commits at lists.llvm.org
Sat Jun 14 06:11:30 PDT 2025


================
@@ -196,6 +196,16 @@ else()
   message(FATAL_ERROR "Unknown value for LLVM_ABI_BREAKING_CHECKS: \"${LLVM_ABI_BREAKING_CHECKS}\"!")
 endif()
 
+string(TOUPPER "${LLVM_ENABLE_DEBUGLOC_COVERAGE_TRACKING}" uppercase_LLVM_ENABLE_DEBUGLOC_COVERAGE_TRACKING)
+
+if( uppercase_LLVM_ENABLE_DEBUGLOC_COVERAGE_TRACKING STREQUAL "COVERAGE" )
+  set( ENABLE_DEBUGLOC_COVERAGE_TRACKING 1 )
----------------
chapuni wrote:

IMO it is not a good practice to override a cached variable with a non-cached variable that has different semantics.

https://github.com/llvm/llvm-project/pull/107278


More information about the llvm-commits mailing list