fix: exclude paused sections from instrumentation measurement

PauseTiming/ResumeTiming now stop/start Callgrind instrumentation in
CODSPEED_ANALYSIS mode, ensuring setup/teardown code is not included
in benchmark measurements.

Author: not-matthias

google_benchmark/include/benchmark/benchmark.h

@@ -1074,9 +1074,6 @@ struct State::StateIterator {
     if (BENCHMARK_BUILTIN_EXPECT(cached_ != 0, true)) {
       return true;
     }
-#ifdef CODSPEED_ANALYSIS
-    measurement_stop();
-#endif
     parent_->FinishKeepRunning();
 
 #ifdef CODSPEED_ANALYSIS
@@ -1103,9 +1100,6 @@ inline BENCHMARK_ALWAYS_INLINE State::StateIterator State::end() {
 #endif
 
   StartKeepRunning();
-#ifdef CODSPEED_ANALYSIS
-  measurement_start();
-#endif
   return StateIterator();
 }
 

google_benchmark/src/benchmark.cc

@@ -19,7 +19,7 @@
 #include "codspeed.h"
 #include "internal_macros.h"
 
-#ifdef CODSPEED_WALLTIME
+#if defined(CODSPEED_WALLTIME) || defined(CODSPEED_ANALYSIS)
 #include "measurement.hpp"
 #endif
 
@@ -272,6 +272,9 @@ void State::PauseTiming() {
 #ifdef CODSPEED_WALLTIME
   uint64_t pause_timestamp = measurement_current_timestamp();
 #endif
+#ifdef CODSPEED_ANALYSIS
+  measurement_stop();
+#endif
 
   // Add in time accumulated so far
   BM_CHECK(started_ && !finished_ && !skipped());
@@ -310,6 +313,9 @@ void State::ResumeTiming() {
   BM_CHECK(resume_timestamp_ == 0);
   resume_timestamp_ = measurement_current_timestamp();
 #endif
+#ifdef CODSPEED_ANALYSIS
+  measurement_start();
+#endif
 }
 
 void State::SkipWithMessage(const std::string& msg) {