Revert "feat(op2): remove async attribute and make FromV8/ToV8 the default behaviour (#1262)" (#1275)

This reverts commit df40fcb.

Author: littledivy

core/benches/ops/async.rs

@@ -51,10 +51,10 @@ pub fn op_make_external() -> *const c_void {
   std::ptr::null()
 }
 
-#[op2]
+#[op2(async)]
 pub async fn op_async_void() {}
 
-#[op2]
+#[op2(async)]
 pub async fn op_async_yield() {
   tokio::task::yield_now().await
 }

core/benches/ops/sync.rs

@@ -25,6 +25,8 @@ deno_core::extension!(
     op_local,
     op_local_scope,
     op_local_nofast,
+    op_global,
+    op_global_scope,
     op_scope,
     op_isolate_nofast,
     op_make_external,
@@ -109,6 +111,16 @@ pub fn op_local_scope<'s>(
 #[op2(nofast)]
 pub fn op_local_nofast(_s: v8::Local<v8::String>) {}
 
+#[op2]
+pub fn op_global(#[global] _s: v8::Global<v8::String>) {}
+
+#[op2]
+pub fn op_global_scope(
+  _scope: &mut v8::PinScope,
+  #[global] _s: v8::Global<v8::String>,
+) {
+}
+
 #[op2(fast)]
 pub fn op_scope(_scope: &mut v8::PinScope) {}
 
@@ -443,6 +455,28 @@ fn bench_op_v8_local_nofast(b: &mut Bencher) {
   );
 }
 
+/// A function that takes a v8::Global<String>
+fn bench_op_v8_global(b: &mut Bencher) {
+  bench_op(
+    b,
+    BENCH_COUNT,
+    "op_global",
+    1,
+    "op_global('this is a reasonably long string that we would like to get the length of!');",
+  );
+}
+
+/// A function that takes a v8::Global<String>
+fn bench_op_v8_global_scope(b: &mut Bencher) {
+  bench_op(
+    b,
+    BENCH_COUNT,
+    "op_global_scope",
+    1,
+    "op_global_scope('this is a reasonably long string that we would like to get the length of!');",
+  );
+}
+
 fn bench_op_bigint(b: &mut Bencher) {
   bench_op(b, BENCH_COUNT, "op_bigint", 1, "op_bigint(0n);");
 }
@@ -547,6 +581,8 @@ benchmark_group!(
   bench_op_v8_local,
   bench_op_v8_local_scope,
   bench_op_v8_local_nofast,
+  bench_op_v8_global,
+  bench_op_v8_global_scope,
   bench_op_bigint,
   bench_op_bigint_return,
   bench_op_v8_scope,