From 8bf7c8da6668221162c0d9550b677a8a00034b87 Mon Sep 17 00:00:00 2001 From: Brian Smith Date: Thu, 20 Oct 2022 16:51:53 -0700 Subject: [PATCH] Benchmarks: Consume the result in `black_box`. --- benches/mod.rs | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/benches/mod.rs b/benches/mod.rs index aaeea2f7..04516e2d 100644 --- a/benches/mod.rs +++ b/benches/mod.rs @@ -13,7 +13,7 @@ fn bench_getrandom(b: &mut test::Bencher) { b.iter(|| { let mut buf = [0u8; N]; getrandom::getrandom(&mut buf[..]).unwrap(); - test::black_box(&buf); + test::black_box(buf); }); } @@ -26,8 +26,9 @@ fn bench_getrandom_uninit(b: &mut test::Bencher) { // TODO: When the feature `maybe_uninit_as_bytes` is available, use: // since that is the shape we expect most callers to have. let mut buf: MaybeUninit<[u8; N]> = MaybeUninit::uninit(); - let buf = getrandom::getrandom_uninit(buf.as_bytes_mut()).unwrap(); - test::black_box(buf); + let _ = getrandom::getrandom_uninit(buf.as_bytes_mut()).unwrap(); + let buf: [u8; N] = unsafe { buf.assume_init() }; + test::black_box(buf) }); }