summaryrefslogtreecommitdiff
path: root/bench_util/benches/op_baseline.rs
diff options
context:
space:
mode:
Diffstat (limited to 'bench_util/benches/op_baseline.rs')
-rw-r--r--bench_util/benches/op_baseline.rs31
1 files changed, 13 insertions, 18 deletions
diff --git a/bench_util/benches/op_baseline.rs b/bench_util/benches/op_baseline.rs
index 4b3bc0203..c2481d0dd 100644
--- a/bench_util/benches/op_baseline.rs
+++ b/bench_util/benches/op_baseline.rs
@@ -6,7 +6,7 @@ use deno_bench_util::bench_or_profile;
use deno_bench_util::bencher::benchmark_group;
use deno_bench_util::bencher::Bencher;
-use deno_core::op;
+use deno_core::op2;
use deno_core::Extension;
deno_core::extension!(
@@ -22,25 +22,25 @@ fn setup() -> Vec<Extension> {
vec![bench_setup::init_ops()]
}
-#[op]
+#[op2(fast)]
fn op_nop() {}
-// TODO(bartlomieju): reenable, currently this op generates a fast function,
-// which is wrong, because i64 is not a compatible type for fast call.
-// #[op]
-// fn op_pi_json() -> i64 {
-// 314159
-// }
+#[op2(fast)]
+#[number]
+fn op_pi_json() -> i64 {
+ 314159
+}
// this is a function since async closures aren't stable
-#[op]
+#[op2(async)]
+#[number]
async fn op_pi_async() -> i64 {
314159
}
-// fn bench_op_pi_json(b: &mut Bencher) {
-// bench_js_sync(b, r#"Deno.core.ops.op_pi_json();"#, setup);
-// }
+fn bench_op_pi_json(b: &mut Bencher) {
+ bench_js_sync(b, r#"Deno.core.ops.op_pi_json();"#, setup);
+}
fn bench_op_nop(b: &mut Bencher) {
bench_js_sync(b, r#"Deno.core.ops.op_nop();"#, setup);
@@ -50,11 +50,6 @@ fn bench_op_async(b: &mut Bencher) {
bench_js_async(b, r#"Deno.core.opAsync("op_pi_async");"#, setup);
}
-benchmark_group!(
- benches,
- // bench_op_pi_json,
- bench_op_nop,
- bench_op_async,
-);
+benchmark_group!(benches, bench_op_pi_json, bench_op_nop, bench_op_async,);
bench_or_profile!(benches);