diff options
| author | fanquake <fanquake@gmail.com> | 2022-05-17 16:18:49 +0100 |
|---|---|---|
| committer | fanquake <fanquake@gmail.com> | 2022-05-17 16:19:07 +0100 |
| commit | dd8a2df488f6fbff7e7863d5278d60f266f8d2f2 (patch) | |
| tree | f7793d8a19ac05ca476c8bd1048a83cef71a1bc8 /src/uint256.cpp | |
| parent | 1ab389b1bac1378e17b13a4c4a81c24b92745cfb (diff) | |
| parent | 4f31c21b7f384e50e0af8275e831085d1d69b386 (diff) | |
Merge bitcoin/bitcoin#25107: bench: Add `--sanity-check` flag, use it in `make check`
4f31c21b7f384e50e0af8275e831085d1d69b386 bench: Make all arguments -kebab-case (laanwj)
652b54e53220fedf2c342e428617bcbd2022964d bench: Add `--sanity-check` flag, use it in `make check` (laanwj)
Pull request description:
The benchmarks are run as part of `make check` for a crash-sanity check. The actual results are being ignored. So only run them for one iteration.
This makes the `bench_bitcoin` part take 2m00 instead of 5m20 here. Which is still too long (imo), but this needs to be solved in the `WalletLoading*` benchmarks which take that long per iteration.
Also change all `bench_bitcoin` arguments to kebab-case to be consistent with the other tools (in a separate commit).
ACKs for top commit:
jonatack:
ACK 4f31c21b7f384e50e0af8275e831085d1d69b386 on the sanity-check version per `git diff c52a71e 4f31c28` (modulo s/--sanity check/--sanity-check/ in src/bench/bench.cpp::L61)
hebasto:
ACK 4f31c21b7f384e50e0af8275e831085d1d69b386, tested on Ubuntu 22.04.
Tree-SHA512: 2661d130fd82e57c9041755190997a4af588fadddcdd05e04fd024f75da1202480e9feab5764566e8dfe7930e8ae0ec71e93f40ac373274953d274072723980d
Diffstat (limited to 'src/uint256.cpp')
0 files changed, 0 insertions, 0 deletions