[bench] Add benchmark for unserialize prevector #13711

pull AkioNak wants to merge 1 commits into bitcoin:master from AkioNak:add_bench_unserialize changing 1 files +28 −0

AkioNak commented at 8:09 AM on July 19, 2018: contributor

This PR adds benchmarks for the unserialization of the prevector.

Note: Separated from #12324.
in src/bench/prevector.cpp:82 in 72b2a49f09 outdated
```
  73 | +    CDataStream s0(SER_NETWORK, 0);
  74 | +    prevector<28, T> t0;
  75 | +    t0.resize(28);
  76 | +    for (auto x = 0; x < 1001; ++x) {
  77 | +        s0 << t0;
  78 | +    }
```
AkioNak commented at 9:03 AM on July 19, 2018:

Self review: To closing to actual condition, it would be better to include the case where the size exceeds the initial capacity.

AkioNak commented at 8:58 AM on July 20, 2018:

done.
fanquake added the label Tests on Jul 19, 2018

[bench] Add benchmark for unserialize prevector

This patch adds 2 benchmarks to measure for performance of
the unserialization of prevector.

46340b3337

AkioNak force-pushed on Jul 20, 2018
kallewoof approved
kallewoof commented at 8:43 AM on July 27, 2018: member

utACK 46340b3337b7ef9fa68030e9c4b2092a25874837
MarcoFalke merged this on Jul 27, 2018
MarcoFalke closed this on Jul 27, 2018
MarcoFalke referenced this in commit f98d1e0008 on Jul 27, 2018
AkioNak deleted the branch on Aug 13, 2018
PastaPastaPasta referenced this in commit c59dcbba59 on Jul 29, 2020
gades referenced this in commit 2968f93c4f on Jul 1, 2021
MarcoFalke locked this on Sep 8, 2021

Contributors

Labels