chore: create new function to set executed benchmark

Author: not-matthias

core/include/measurement.hpp

@@ -52,24 +52,30 @@ __attribute__((always_inline)) inline void measurement_start() {
   CALLGRIND_START_INSTRUMENTATION;
 }
 
-__attribute__((always_inline)) inline void measurement_stop(
-    const std::string& name) {
+__attribute__((always_inline)) inline void measurement_stop() {
   CALLGRIND_STOP_INSTRUMENTATION;
+
   instrument_hooks_stop_benchmark(get_hooks());
+};
 
+__attribute__((noinline)) inline void measurement_set_executed(
+    const std::string& name) {
 #ifdef _WIN32
   auto current_pid = _getpid();
 #else
   auto current_pid = getpid();
 #endif
+
   instrument_hooks_executed_benchmark(get_hooks(), current_pid, name.c_str());
-};
+}
+
 #else
 // Stub implementations for non-instrumentation builds
 inline bool measurement_is_instrumented() { return false; }
 inline void measurement_set_metadata() {}
 inline void measurement_start() {}
-inline void measurement_stop(const std::string& name) { (void)name; }
+inline void measurement_stop() {}
+inline void measurement_set_executed(const std::string& name) { (void)name; }
 #endif
 
 #endif  // MEASUREMENT_H

core/src/codspeed.cpp

@@ -85,7 +85,7 @@ void CodSpeed::start_benchmark(const std::string &name) {
 }
 
 void CodSpeed::end_benchmark() {
-  measurement_stop(current_benchmark);
+  measurement_set_executed(current_benchmark);
   benchmarked.push_back(current_benchmark);
   std::string action_str = is_instrumented ? "Measured" : "Checked";
   std::string group_str =

google_benchmark/include/benchmark/benchmark.h

@@ -1066,6 +1066,7 @@ struct State::StateIterator {
     if (BENCHMARK_BUILTIN_EXPECT(cached_ != 0, true)) return true;
 #ifdef CODSPEED_INSTRUMENTATION
     if (parent_->codspeed_ != NULL) {
+      measurement_stop();
       parent_->codspeed_->end_benchmark();
     }
 #endif