Add concat to benchmark

nodejs · Apr 19, 2016 · 328ae5f · 328ae5f
1 parent d068679
commit 328ae5f
Showing 1 changed file with 24 additions and 3 deletions.
diff --git a/benchmark/misc/console.js b/benchmark/misc/console.js
@@ -9,13 +9,20 @@ const v8 = require('v8');
 v8.setFlagsFromString('--allow_natives_syntax');
 
 var bench = common.createBenchmark(main, {
-  method: ['restAndSpread', 'argumentsAndApply', 'restAndApply'],
+  method: ['restAndSpread',
+           'argumentsAndApply',
+           'restAndApply',
+           'restAndConcat'],
   concat: [1, 0],
   n: [1000000]
 });
 
 const nullStream = createNullStream();
 
+function usingRestAndConcat(...args) {
+  nullStream.write('this is ' + args[0] + ' of ' + args[1] + '\n');
+}
+
 function usingRestAndSpreadTS(...args) {
   nullStream.write(`${util.format(...args)}\n`);
 }
@@ -46,6 +53,16 @@ function optimize(method, ...args) {
   method(...args);
 }
 
+function runUsingRestAndConcat(n) {
+  optimize(usingRestAndConcat, 'a', 1);
+
+  var i = 0;
+  bench.start();
+  for (; i < n; i++)
+    usingRestAndConcat('a', 1);
+  bench.end(n);
+}
+
 function runUsingRestAndSpread(n, concat) {
 
   const method = concat ? usingRestAndSpreadC : usingRestAndSpreadTS;
@@ -89,11 +106,15 @@ function main(conf) {
       runUsingRestAndSpread(n, conf.concat);
       break;
     case 'restAndApply':
-      runUsingRestAndApply(n);
+      runUsingRestAndApply(n, conf.concat);
       break;
     case 'argumentsAndApply':
-      runUsingArgumentsAndApply(n);
+      runUsingArgumentsAndApply(n, conf.concat);
       break;
+    case 'restAndConcat':
+      if (conf.concat)
+        runUsingRestAndConcat(n);
+    break;
     default:
       throw new Error('Unexpected method');
   }