Merge bitcoin/bitcoin#24784: refactor: deduplicate integer serialization in RollingBloom benchmark

fff91418ffa4911f7262e824418af664b25d4026 refactor: Remove deduplication of data in rollingbloom bench (phyBrackets)

Pull request description:

  Fixed up #24088.

ACKs for top commit:
  vincenzopalazzo:
    ACK fff91418ff

Tree-SHA512: 9fef617bceb74a1aec4f4a1e7c4732c4764af3e8ac2fc02b84ce370e8b97431957ca17ee8f44fb96765f7304f8d7e5bfb951440db98ba40f240612f2232d215e
This commit is contained in:
MarcoFalke 2022-04-07 11:53:16 +02:00 committed by pasta
parent e3a0fce5c9
commit 61bae780b6
No known key found for this signature in database
GPG Key ID: E2F3D7916E722D38

View File

@ -5,6 +5,9 @@
#include <bench/bench.h> #include <bench/bench.h>
#include <common/bloom.h> #include <common/bloom.h>
#include <crypto/common.h>
#include <vector>
static void RollingBloom(benchmark::Bench& bench) static void RollingBloom(benchmark::Bench& bench)
{ {
@ -13,16 +16,10 @@ static void RollingBloom(benchmark::Bench& bench)
uint32_t count = 0; uint32_t count = 0;
bench.run([&] { bench.run([&] {
count++; count++;
data[0] = count & 0xFF; WriteLE32(data.data(), count);
data[1] = (count >> 8) & 0xFF;
data[2] = (count >> 16) & 0xFF;
data[3] = (count >> 24) & 0xFF;
filter.insert(data); filter.insert(data);
data[0] = (count >> 24) & 0xFF; WriteBE32(data.data(), count);
data[1] = (count >> 16) & 0xFF;
data[2] = (count >> 8) & 0xFF;
data[3] = count & 0xFF;
filter.contains(data); filter.contains(data);
}); });
} }