Commit 8caa47cf by Derek Mauro Committed by Copybara-Service

bits_benchmark: Fix typo to benchmark the intended function

PiperOrigin-RevId: 496788919
Change-Id: I8867f8b884b81aa2f7b6d08dabd90c75965d1939
parent 1f8da02d
...@@ -24,7 +24,7 @@ namespace absl { ...@@ -24,7 +24,7 @@ namespace absl {
namespace { namespace {
template <typename T> template <typename T>
static void BM_bitwidth(benchmark::State& state) { static void BM_bit_width(benchmark::State& state) {
const auto count = static_cast<size_t>(state.range(0)); const auto count = static_cast<size_t>(state.range(0));
absl::BitGen rng; absl::BitGen rng;
...@@ -36,17 +36,17 @@ static void BM_bitwidth(benchmark::State& state) { ...@@ -36,17 +36,17 @@ static void BM_bitwidth(benchmark::State& state) {
while (state.KeepRunningBatch(static_cast<int64_t>(count))) { while (state.KeepRunningBatch(static_cast<int64_t>(count))) {
for (size_t i = 0; i < count; ++i) { for (size_t i = 0; i < count; ++i) {
benchmark::DoNotOptimize(values[i]); benchmark::DoNotOptimize(absl::bit_width(values[i]));
} }
} }
} }
BENCHMARK_TEMPLATE(BM_bitwidth, uint8_t)->Range(1, 1 << 20); BENCHMARK_TEMPLATE(BM_bit_width, uint8_t)->Range(1, 1 << 20);
BENCHMARK_TEMPLATE(BM_bitwidth, uint16_t)->Range(1, 1 << 20); BENCHMARK_TEMPLATE(BM_bit_width, uint16_t)->Range(1, 1 << 20);
BENCHMARK_TEMPLATE(BM_bitwidth, uint32_t)->Range(1, 1 << 20); BENCHMARK_TEMPLATE(BM_bit_width, uint32_t)->Range(1, 1 << 20);
BENCHMARK_TEMPLATE(BM_bitwidth, uint64_t)->Range(1, 1 << 20); BENCHMARK_TEMPLATE(BM_bit_width, uint64_t)->Range(1, 1 << 20);
template <typename T> template <typename T>
static void BM_bitwidth_nonzero(benchmark::State& state) { static void BM_bit_width_nonzero(benchmark::State& state) {
const auto count = static_cast<size_t>(state.range(0)); const auto count = static_cast<size_t>(state.range(0));
absl::BitGen rng; absl::BitGen rng;
...@@ -60,14 +60,14 @@ static void BM_bitwidth_nonzero(benchmark::State& state) { ...@@ -60,14 +60,14 @@ static void BM_bitwidth_nonzero(benchmark::State& state) {
for (size_t i = 0; i < count; ++i) { for (size_t i = 0; i < count; ++i) {
const T value = values[i]; const T value = values[i];
ABSL_ASSUME(value > 0); ABSL_ASSUME(value > 0);
benchmark::DoNotOptimize(value); benchmark::DoNotOptimize(absl::bit_width(value));
} }
} }
} }
BENCHMARK_TEMPLATE(BM_bitwidth_nonzero, uint8_t)->Range(1, 1 << 20); BENCHMARK_TEMPLATE(BM_bit_width_nonzero, uint8_t)->Range(1, 1 << 20);
BENCHMARK_TEMPLATE(BM_bitwidth_nonzero, uint16_t)->Range(1, 1 << 20); BENCHMARK_TEMPLATE(BM_bit_width_nonzero, uint16_t)->Range(1, 1 << 20);
BENCHMARK_TEMPLATE(BM_bitwidth_nonzero, uint32_t)->Range(1, 1 << 20); BENCHMARK_TEMPLATE(BM_bit_width_nonzero, uint32_t)->Range(1, 1 << 20);
BENCHMARK_TEMPLATE(BM_bitwidth_nonzero, uint64_t)->Range(1, 1 << 20); BENCHMARK_TEMPLATE(BM_bit_width_nonzero, uint64_t)->Range(1, 1 << 20);
} // namespace } // namespace
} // namespace absl } // namespace absl
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment