HomePhabricator

Context isn't freed in the ECDH benchmark

Description

Context isn't freed in the ECDH benchmark

Summary:

  • free the ctx at the end of bench_ecdh
  • Pass num of iters to benchmarks as variable, and define envvar
  • Add running benchmarks regularly and under valgrind in travis

This is a backport of libsecp256k1 PR722

Test Plan:

ninja all check check-secp256k1 bench-secp256k1

Reviewers: #bitcoin_abc, Fabien

Reviewed By: #bitcoin_abc, Fabien

Subscribers: Fabien

Differential Revision: https://reviews.bitcoinabc.org/D6364

Details

Provenance
Elichai Turkel <elichai.turkel@gmail.com>Authored on Mar 4 2020, 12:14
deadalnixCommitted on Jun 4 2020, 16:17
deadalnixPushed on Jun 4 2020, 16:17
Reviewer
Restricted Project
Differential Revision
D6364: Context isn't freed in the ECDH benchmark
Parents
rABC0f1539259c1f: Use jemalloc as a default
Branches
Unknown
Tags
Unknown