AntoinePrv commented on code in PR #49756:
URL: https://github.com/apache/arrow/pull/49756#discussion_r3129441011
##########
cpp/src/arrow/util/bpacking_benchmark.cc:
##########
@@ -107,10 +107,10 @@ void BM_Unpack(benchmark::State& state, bool aligned,
UnpackFunc<Int> unpack, bo
// will not emit runs larger than 512 (though other implementation might), so
we biased
// the benchmarks towards a rather small scale.
static const auto kNumValuesRange = benchmark::CreateRange(32, 512, 2);
-constexpr std::initializer_list<int64_t> kBitWidths8 = {1, 2, 8};
-constexpr std::initializer_list<int64_t> kBitWidths16 = {1, 2, 8, 13};
-constexpr std::initializer_list<int64_t> kBitWidths32 = {1, 2, 8, 20};
-constexpr std::initializer_list<int64_t> kBitWidths64 = {1, 2, 8, 20, 47};
+static const auto kBitWidths8 = benchmark::CreateDenseRange(0, 8, 1);
+static const auto kBitWidths16 = benchmark::CreateDenseRange(0, 16, 1);
+static const auto kBitWidths32 = benchmark::CreateDenseRange(0, 32, 1);
+static const auto kBitWidths64 = benchmark::CreateDenseRange(0, 64, 1);
Review Comment:
Sorry, did not mean to push this commit, this was for local benchmarking.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
