From ff68bee44900e176c877ca608742abf13b902f8a Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Alexander=20Thei=C3=9Fen?= Date: Mon, 31 Jan 2022 13:14:12 +0100 Subject: [PATCH] We should use bench_with_input so that a blackbox is used --- benches/benches.rs | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/benches/benches.rs b/benches/benches.rs index be87e7d..09e04e6 100644 --- a/benches/benches.rs +++ b/benches/benches.rs @@ -27,20 +27,20 @@ where let entry = entry.unwrap(); let bytes = read(&entry.path()).unwrap(); group.throughput(Throughput::Bytes(bytes.len().try_into().unwrap())); - group.bench_function(entry.file_name().to_str().unwrap(), |bench| { - bench.iter(|| f(deserialize_buffer(&bytes).unwrap())) + group.bench_with_input(entry.file_name().to_str().unwrap(), &bytes, |bench, input| { + bench.iter(|| f(deserialize_buffer(input).unwrap())) }); } } -pub fn gas_metering(c: &mut Criterion) { +fn gas_metering(c: &mut Criterion) { let mut group = c.benchmark_group("Gas Metering"); any_fixture(&mut group, |module| { gas_metering::inject(module, &gas_metering::ConstantCostRules::default(), "env").unwrap(); }); } -pub fn stack_height_limiter(c: &mut Criterion) { +fn stack_height_limiter(c: &mut Criterion) { let mut group = c.benchmark_group("Stack Height Limiter"); any_fixture(&mut group, |module| { inject_stack_limiter(module, 128).unwrap();