From 1fc605a8aea247b2d5776c22b3e187d1b190fd99 Mon Sep 17 00:00:00 2001 From: Akio Nakamura Date: Thu, 5 Jul 2018 20:04:48 +0900 Subject: [PATCH] fix bench/prevector.cpp 1. PrevectorClear() 2nd call of clear() should to operate t1 instead of t0. This patch changes t0 to t1. 2. PREVECTOR_TEST() PREVECTOR_TEST macro should to call both PrevectorXX(state) and PrevectorXX(state) by specific "name" which given by parameter instead of calling PrevectorResize<>() regardless of "name". This patch changes "PrevectorResize<" of this macro to "Prevector ## name<". --- src/bench/prevector.cpp | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/src/bench/prevector.cpp b/src/bench/prevector.cpp index 3cfad1b2c..09c702084 100644 --- a/src/bench/prevector.cpp +++ b/src/bench/prevector.cpp @@ -42,7 +42,7 @@ static void PrevectorClear(benchmark::State& state) t0.resize(28); t0.clear(); t1.resize(29); - t0.clear(); + t1.clear(); } } } @@ -64,11 +64,11 @@ static void PrevectorResize(benchmark::State& state) #define PREVECTOR_TEST(name, nontrivops, trivops) \ static void Prevector ## name ## Nontrivial(benchmark::State& state) { \ - PrevectorResize(state); \ + Prevector ## name(state); \ } \ BENCHMARK(Prevector ## name ## Nontrivial, nontrivops); \ static void Prevector ## name ## Trivial(benchmark::State& state) { \ - PrevectorResize(state); \ + Prevector ## name(state); \ } \ BENCHMARK(Prevector ## name ## Trivial, trivops);