Skip to content

Commit 4e3569c

Browse files
authored
Improve and extend translation benchmarks (#1227)
* refactor and rename translation benchmarks * add lazy+unchecked translation benchmarks
1 parent 35eb605 commit 4e3569c

File tree

1 file changed

+21
-13
lines changed

1 file changed

+21
-13
lines changed

crates/wasmi/benches/benches.rs

Lines changed: 21 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -156,24 +156,24 @@ fn bench_translate_for(
156156
c: &mut Criterion,
157157
name: &str,
158158
path: &str,
159-
validation: Validation,
160159
mode: CompilationMode,
160+
validation: Validation,
161161
fuel_metering: FuelMetering,
162162
) {
163-
let validation_id = match validation {
164-
Validation::Checked => "checked",
165-
Validation::Unchecked => "unchecked",
166-
};
167163
let mode_id = match mode {
168164
CompilationMode::Eager => "eager",
169165
CompilationMode::LazyTranslation => "lazy-translation",
170166
CompilationMode::Lazy => "lazy",
171167
};
168+
let validation_id = match validation {
169+
Validation::Checked => "checked",
170+
Validation::Unchecked => "unchecked",
171+
};
172172
let fuel_id = match fuel_metering {
173-
FuelMetering::Enabled => "fuel",
174-
FuelMetering::Disabled => "default",
173+
FuelMetering::Enabled => "+metered",
174+
FuelMetering::Disabled => "",
175175
};
176-
let bench_id = format!("translate/{name}/{validation_id}/{mode_id}/{fuel_id}");
176+
let bench_id = format!("translate/{name}/{mode_id}/{validation_id}{fuel_id}");
177177
c.bench_function(&bench_id, |b| {
178178
let mut config = bench_config();
179179
if matches!(fuel_metering, FuelMetering::Enabled) {
@@ -202,40 +202,48 @@ fn bench_translate_for_all(c: &mut Criterion, name: &str, path: &str) {
202202
c,
203203
name,
204204
path,
205-
Validation::Checked,
206205
CompilationMode::Eager,
206+
Validation::Checked,
207207
FuelMetering::Disabled,
208208
);
209209
bench_translate_for(
210210
c,
211211
name,
212212
path,
213-
Validation::Checked,
214213
CompilationMode::Eager,
214+
Validation::Checked,
215215
FuelMetering::Enabled,
216216
);
217217
bench_translate_for(
218218
c,
219219
name,
220220
path,
221-
Validation::Checked,
222-
CompilationMode::LazyTranslation,
221+
CompilationMode::Eager,
222+
Validation::Unchecked,
223223
FuelMetering::Disabled,
224224
);
225225
bench_translate_for(
226226
c,
227227
name,
228228
path,
229+
CompilationMode::LazyTranslation,
229230
Validation::Checked,
231+
FuelMetering::Disabled,
232+
);
233+
bench_translate_for(
234+
c,
235+
name,
236+
path,
230237
CompilationMode::Lazy,
238+
Validation::Checked,
231239
FuelMetering::Disabled,
232240
);
233241
bench_translate_for(
234242
c,
235243
name,
236244
path,
245+
CompilationMode::Lazy,
237246
Validation::Unchecked,
238-
CompilationMode::Eager,
239247
FuelMetering::Disabled,
240248
);
241249
}

0 commit comments

Comments
 (0)