fixup! fix: exclude paused sections from instrumentation measurement

Author: not-matthias

core/include/measurement.hpp

@@ -56,7 +56,18 @@ inline void measurement_set_metadata() {
   instrument_hooks_write_environment(g_hooks, getpid());
 }
 
+#ifdef CODSPEED_ANALYSIS
+// inline (C++17) so every translation unit shares one definition; a static
+// would give each TU its own copy and silently desync the toggle parity.
+inline bool measurement_collecting = true;
+#endif
+
 ALWAYS_INLINE void measurement_start() {
+#ifdef CODSPEED_ANALYSIS
+  // START_INSTRUMENTATION leaves callgrind collecting (collect-atstart=yes),
+  // so the parity flag must start each benchmark in the ON state.
+  measurement_collecting = true;
+#endif
   instrument_hooks_start_benchmark_inline(g_hooks);
 }
 
@@ -89,8 +100,6 @@ ALWAYS_INLINE void measurement_add_benchmark_timestamps(uint64_t start,
 }
 
 #ifdef CODSPEED_ANALYSIS
-static bool measurement_collecting = true;
-
 ALWAYS_INLINE void measurement_pause_timing() {
   if (measurement_collecting) {
     callgrind_toggle_collect();