Disable coverage if unsupported and only define extra targets when ne…

…eded.

CMakeLists.txt

@@ -307,6 +307,7 @@ if(AVIF_ENABLE_COVERAGE)
     else()
         # TODO: Add support for other compilers
         message(WARNING "libavif: Ignoring request for coverage (AVIF_ENABLE_COVERAGE); only clang is currently supported.")
+        set(AVIF_ENABLE_COVERAGE OFF)
     endif()
 endif()
 

tests/CMakeLists.txt

@@ -12,12 +12,14 @@ set(COVERAGE_TARGETS)
 # Macro to register a test for coverage. The first argument is the target name.
 # Other arguments, like data path, can be added.
 macro(register_test_for_coverage TEST_NAME)
-    add_custom_target(
-        ${TEST_NAME}_coverage
-        COMMAND ${CMAKE_COMMAND} -E env "LLVM_PROFILE_FILE=${CMAKE_CURRENT_BINARY_DIR}/${TEST_NAME}.profraw"
-                $<TARGET_FILE:${TEST_NAME}> ${ARGN}
-    )
-    list(APPEND COVERAGE_TARGETS ${TEST_NAME})
+    if(AVIF_ENABLE_COVERAGE)
+        add_custom_target(
+            ${TEST_NAME}_coverage
+            COMMAND ${CMAKE_COMMAND} -E env "LLVM_PROFILE_FILE=${CMAKE_CURRENT_BINARY_DIR}/${TEST_NAME}.profraw"
+                    $<TARGET_FILE:${TEST_NAME}> ${ARGN}
+        )
+        list(APPEND COVERAGE_TARGETS ${TEST_NAME})
+    endif()
 endmacro()
 
 add_executable(aviftest aviftest.c)