coverage: Do not call __gcov_dump() when using source-based coverage

It's useless and will do nothing.

diff --git a/pdns/coverage.cc b/pdns/coverage.cc
index 3cbfa40424eb084e1d03a05687584b122288e9f6..c996207ec8d0486dd98a444e244468658d1ec8e6 100644 (file)
--- a/pdns/coverage.cc
+++ b/pdns/coverage.cc
@@ -25,11 +25,12 @@
 #ifdef COVERAGE
 extern "C"
 {
-  // NOLINTNEXTLINE(bugprone-reserved-identifier): not ours
-  void __gcov_dump(void);
 #ifdef CLANG_COVERAGE
   // NOLINTNEXTLINE(bugprone-reserved-identifier): not ours
   int __llvm_profile_write_file(void);
+#else /* CLANG_COVERAGE */
+  // NOLINTNEXTLINE(bugprone-reserved-identifier): not ours
+  void __gcov_dump(void);
 #endif /* CLANG_COVERAGE */
 }
 #endif /* COVERAGE */
@@ -39,9 +40,10 @@ namespace pdns::coverage
 void dumpCoverageData()
 {
 #ifdef COVERAGE
-  __gcov_dump();
 #ifdef CLANG_COVERAGE
   __llvm_profile_write_file();
+#else /* CLANG_COVERAGE */
+  __gcov_dump();
 #endif /* CLANG_COVERAGE */
 #endif /* COVERAGE */
 }