From 5666807328c0cf0f6ac5d6f9079d33edb5b42d88 Mon Sep 17 00:00:00 2001 From: Behdad Esfahbod Date: Sat, 27 May 2023 12:37:36 -0600 Subject: [PATCH] [benchmark-map] Improve Insert benchmark Previously it was enlarging the map depending on whatever number of runs the benchmark-runner decided to run the loop. That wasn't very useful... --- perf/benchmark-map.cc | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/perf/benchmark-map.cc b/perf/benchmark-map.cc index f278a0c94..112f7a24d 100644 --- a/perf/benchmark-map.cc +++ b/perf/benchmark-map.cc @@ -30,13 +30,12 @@ static void BM_MapInsert(benchmark::State& state) { RandomMap(map_size, original); assert(hb_map_get_population(original) == map_size); - auto needle = map_size / 2; - auto v = 0; + auto needle = 0u; for (auto _ : state) { // TODO(garretrieger): create a copy of the original map. // Needs a hb_map_copy(..) in public api. - hb_map_set (original, needle++, v++); + hb_map_set (original, needle++ % map_size, 1); } hb_map_destroy(original);