Skip to content

Commit 1496be4

Browse files
committed
update binggan
1 parent b911b71 commit 1496be4

File tree

2 files changed

+23
-22
lines changed

2 files changed

+23
-22
lines changed

Cargo.toml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -23,7 +23,7 @@ more-asserts = "0.3.1"
2323
snap = "1.1.0"
2424
serde_json = "1.0.91"
2525
proptest = "1.0.0"
26-
binggan = { version = "0.12.0"}
26+
binggan = { version = "0.14.0"}
2727
jemallocator = "0.5.4"
2828

2929
[dev-dependencies.lz-fear]

benches/binggan_bench.rs

Lines changed: 22 additions & 21 deletions
Original file line numberDiff line numberDiff line change
@@ -51,7 +51,7 @@ fn main() {
5151
fn frame_decompress(data_sets: &[(String, Vec<u8>)]) {
5252
let mut runner = BenchRunner::with_name("frame_decompress");
5353
runner
54-
.add_plugin(PerfCounterPlugin::default())
54+
//.add_plugin(PerfCounterPlugin::default())
5555
.add_plugin(PeakMemAllocPlugin::new(&GLOBAL));
5656
for (name, data_set) in data_sets {
5757
let compressed_independent = lz4_cpp_frame_compress(data_set, true).unwrap();
@@ -62,24 +62,24 @@ fn frame_decompress(data_sets: &[(String, Vec<u8>)]) {
6262
group.set_input_size(data_set.len());
6363

6464
group.register_with_input("lz4 flex independent", &compressed_independent, move |i| {
65-
black_box(lz4_flex_frame_decompress(i).unwrap());
66-
Some(())
65+
let out = black_box(lz4_flex_frame_decompress(i).unwrap());
66+
out.len()
6767
});
6868
group.register_with_input("lz4 c90 independent", &compressed_independent, move |i| {
69-
black_box(lz4_cpp_frame_decompress(i).unwrap());
70-
Some(())
69+
let out = black_box(lz4_cpp_frame_decompress(i).unwrap());
70+
out.len()
7171
});
7272
group.register_with_input("lz4 flex linked", &compressed_linked, move |i| {
73-
black_box(lz4_flex_frame_decompress(i).unwrap());
74-
Some(())
73+
let out = black_box(lz4_flex_frame_decompress(i).unwrap());
74+
out.len()
7575
});
7676
group.register_with_input("lz4 c90 linked", &compressed_linked, move |i| {
77-
black_box(lz4_cpp_frame_decompress(i).unwrap());
78-
Some(())
77+
let out = black_box(lz4_cpp_frame_decompress(i).unwrap());
78+
out.len()
7979
});
8080
group.register_with_input("snap", &comp_snap, move |i| {
81-
black_box(decompress_snap_frame(i));
82-
Some(())
81+
let out = black_box(decompress_snap_frame(i));
82+
out.len()
8383
});
8484

8585
group.run();
@@ -97,26 +97,26 @@ fn frame_compress(mut runner: InputGroup<Vec<u8>, usize>) {
9797
frame_info.block_size = lz4_flex::frame::BlockSize::Max256KB;
9898
frame_info.block_mode = lz4_flex::frame::BlockMode::Independent;
9999
let out = black_box(lz4_flex_frame_compress_with(frame_info, i).unwrap());
100-
Some(out.len())
100+
out.len()
101101
});
102102
runner.register("lz4 c90 indep", move |i| {
103103
let out = black_box(lz4_cpp_frame_compress(i, true).unwrap());
104-
Some(out.len())
104+
out.len()
105105
});
106106
runner.register("lz4 flex linked", move |i| {
107107
let mut frame_info = lz4_flex::frame::FrameInfo::new();
108108
frame_info.block_size = lz4_flex::frame::BlockSize::Max256KB;
109109
frame_info.block_mode = lz4_flex::frame::BlockMode::Linked;
110110
let out = black_box(lz4_flex_frame_compress_with(frame_info, i).unwrap());
111-
Some(out.len())
111+
out.len()
112112
});
113113
runner.register("lz4 c90 linked", move |i| {
114114
let out = black_box(lz4_cpp_frame_compress(i, false).unwrap());
115-
Some(out.len())
115+
out.len()
116116
});
117117
runner.register("snap", move |i| {
118118
let out = compress_snap_frame(i);
119-
Some(out.len())
119+
out.len()
120120
});
121121

122122
runner.run();
@@ -130,15 +130,15 @@ fn block_compress(mut runner: InputGroup<Vec<u8>, usize>) {
130130
runner.throughput(|data| data.len());
131131
runner.register("lz4 flex", move |i| {
132132
let out = black_box(lz4_flex::compress(i));
133-
Some(out.len())
133+
out.len()
134134
});
135135
runner.register("lz4 c90", move |i| {
136136
let out = black_box(lz4_cpp_block_compress(i).unwrap());
137-
Some(out.len())
137+
out.len()
138138
});
139139
runner.register("snap", move |i| {
140140
let out = black_box(compress_snap(i));
141-
Some(out.len())
141+
out.len()
142142
});
143143

144144
runner.run();
@@ -148,6 +148,7 @@ fn block_decompress() {
148148
let mut runner = BenchRunner::with_name("block_decompress");
149149
// Set the peak mem allocator. This will enable peak memory reporting.
150150
runner.add_plugin(PeakMemAllocPlugin::new(&GLOBAL));
151+
runner.add_plugin(CacheTrasher::default());
151152
for data_uncomp in ALL {
152153
let comp_lz4 = lz4_cpp_block_compress(data_uncomp).unwrap();
153154
let bundle = (comp_lz4, data_uncomp.len());
@@ -159,11 +160,11 @@ fn block_decompress() {
159160

160161
group.register_with_input("lz4 flex", &bundle, move |i| {
161162
let size = black_box(lz4_flex::decompress(&i.0, i.1).unwrap());
162-
Some(size.len())
163+
size.len()
163164
});
164165
group.register_with_input("lz4 c90", &bundle, move |i| {
165166
let size = black_box(lz4_cpp_block_decompress(&i.0, i.1).unwrap());
166-
Some(size.len())
167+
size.len()
167168
});
168169

169170
group.run();

0 commit comments

Comments
 (0)