lib: make queueMicrotask faster

No longer create an additional scope within queueMicrotask in order to improve performance. PR-URL: #27032 Reviewed-By: Gus Caplan <[email protected]> Reviewed-By: Ruben Bridgewater <[email protected]> Reviewed-By: James M Snell <[email protected]>

Author: apapirovski

benchmark/process/queue-microtask-breadth.js

@@ -0,0 +1,21 @@
+'use strict';
+
+constcommon=require('../common.js');
+constbench=common.createBenchmark(main,{
+n: [4e5]
+});
+
+functionmain({ n }){
+varj=0;
+
+functioncb(){
+j++;
+if(j===n)
+bench.end(n);
+}
+
+bench.start();
+for(vari=0;i<n;i++){
+queueMicrotask(cb);
+}
+}

benchmark/process/queue-microtask-depth.js

@@ -0,0 +1,17 @@
+'use strict';
+constcommon=require('../common.js');
+constbench=common.createBenchmark(main,{
+n: [12e5]
+});
+
+functionmain({ n }){
+letcounter=n;
+bench.start();
+queueMicrotask(onNextTick);
+functiononNextTick(){
+if(--counter)
+queueMicrotask(onNextTick);
+else
+bench.end(n);
+}
+}

lib/internal/process/task_queues.js

@@ -37,6 +37,8 @@ const{
 }=require('internal/errors').codes;
 constFixedQueue=require('internal/fixed_queue');
 
+constFunctionBind=Function.call.bind(Function.prototype.bind);
+
 // *Must* match Environment::TickInfo::Fields in src/env.h.
 constkHasTickScheduled=0;
 
@@ -149,28 +151,32 @@ function createMicrotaskResource(){
 });
 }
 
+functionrunMicrotask(){
+this.runInAsyncScope(()=>{
+constcallback=this.callback;
+try{
+callback();
+}catch(error){
+// TODO(devsnek) remove this if
+// https://bugs.chromium.org/p/v8/issues/detail?id=8326
+// is resolved such that V8 triggers the fatal exception
+// handler for microtasks
+triggerFatalException(error);
+}finally{
+this.emitDestroy();
+}
+});
+}
+
 functionqueueMicrotask(callback){
 if(typeofcallback!=='function'){
 thrownewERR_INVALID_ARG_TYPE('callback','function',callback);
 }
 
 constasyncResource=createMicrotaskResource();
+asyncResource.callback=callback;
 
-enqueueMicrotask(()=>{
-asyncResource.runInAsyncScope(()=>{
-try{
-callback();
-}catch(error){
-// TODO(devsnek) remove this if
-// https://bugs.chromium.org/p/v8/issues/detail?id=8326
-// is resolved such that V8 triggers the fatal exception
-// handler for microtasks
-triggerFatalException(error);
-}finally{
-asyncResource.emitDestroy();
-}
-});
-});
+enqueueMicrotask(FunctionBind(runMicrotask,asyncResource));
 }
 
 module.exports={