fix: remove memory allocations

not-matthias · not-matthias · commit 6a9131904a55 · 2025-10-03T18:41:47.000+02:00
For some reason it slows down the overall execution of the benchmarks a lot. The time rose from 3 to 20 minutes.
diff --git a/examples/google_benchmark_cmake/helper.hpp b/examples/google_benchmark_cmake/helper.hpp
@@ -19,9 +19,6 @@ NOINLINE static uint64_t recursive_fib(int n) {
 }
 
 NOINLINE static uint64_t expensive_operation() {
-  // Large memory allocation
-  std::vector<uint64_t> data(1024 * 1024, 42);  // 8 MiB allocation
-
   // Expensive recursive computation that will dominate flamegraph
   uint64_t fib_result = recursive_fib(30);
 

Original file line number	Diff line number	Diff line change
`@@ -19,9 +19,6 @@ NOINLINE static uint64_t recursive_fib(int n) {`
`19`	`19`	`}`
`20`	`20`
`21`	`21`	`NOINLINE static uint64_t expensive_operation() {`
`22`		`- // Large memory allocation`
`23`		`- std::vector<uint64_t> data(1024 * 1024, 42); // 8 MiB allocation`
`24`		`-`
`25`	`22`	`// Expensive recursive computation that will dominate flamegraph`
`26`	`23`	`uint64_t fib_result = recursive_fib(30);`
`27`	`24`