chore: dont start/stop measurements in warmup

We would also measure the warmup with memory profiling, which is not what we want. This was fine with simulation, because we cleared the results on the second run.

Author: not-matthias

google_benchmark/include/benchmark/benchmark.h

@@ -1075,12 +1075,14 @@ struct State::StateIterator {
       return true;
     }
 #ifdef CODSPEED_ANALYSIS
+  if (!parent_->is_warmup_) {
     measurement_stop();
+  }
 #endif
     parent_->FinishKeepRunning();
 
 #ifdef CODSPEED_ANALYSIS
-    if (parent_->codspeed_ != NULL) {
+    if (parent_->codspeed_ != NULL && !parent_->is_warmup_) {
       parent_->codspeed_->end_benchmark();
     }
 #endif
@@ -1097,14 +1099,16 @@ inline BENCHMARK_ALWAYS_INLINE State::StateIterator State::begin() {
 }
 inline BENCHMARK_ALWAYS_INLINE State::StateIterator State::end() {
 #ifdef CODSPEED_ANALYSIS
-  if (this->codspeed_ != NULL) {
+  if (this->codspeed_ != NULL && !is_warmup_) {
     this->codspeed_->start_benchmark(name_);
   }
 #endif
 
   StartKeepRunning();
 #ifdef CODSPEED_ANALYSIS
-  measurement_start();
+  if (!is_warmup_) {
+    measurement_start();
+  }
 #endif
   return StateIterator();
 }