Skip to content

Commit 0ac06c1

Browse files
laanwjPastaPastaPasta
authored andcommitted
Merge bitcoin#19214: Auto-detect SHA256 implementation in benchmarks
addf18d Call SHA256AutoDetect in benchmark setup (Pieter Wuille) Pull request description: It seems `SHA256AutoDetect()` was not being called in benchmarks, making the numbers only reflect the naive implementation. Fix this by calling it in bench_bitcoin's setup. ACKs for top commit: fjahr: tested ACK addf18d pstratem: ACK addf18d laanwj: ACK addf18d Tree-SHA512: 3ba4b068145942df1429bf5913e3f685511e6ebeae2c1a3f9b8ac0144f6db1c7df456f88f480a2129f3e1602e3bf6a39530bb96e2c74c03ddb19324cec6799c7
1 parent edf1789 commit 0ac06c1

File tree

1 file changed

+1
-0
lines changed

1 file changed

+1
-0
lines changed

src/bench/bench_bitcoin.cpp

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -41,6 +41,7 @@ int main(int argc, char** argv)
4141
{
4242
ArgsManager argsman;
4343
SetupBenchArgs(argsman);
44+
SHA256AutoDetect();
4445
std::string error;
4546
if (!argsman.ParseParameters(argc, argv, error)) {
4647
tfm::format(std::cerr, "Error parsing command line arguments: %s\n", error);

0 commit comments

Comments
 (0)