fix: wrong benchmark setup

swc-project · Aug 23, 2020 · 7bd298d · 7bd298d
1 parent aa888e7
commit 7bd298d
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 6 deletions.
diff --git a/bench/index.js b/bench/index.js
@@ -1,5 +1,6 @@
 /* eslint-disable @typescript-eslint/no-floating-promises */
 const fs = require('fs')
+const os = require('os')
 
 const { transformSync: transformSyncNapi, transform: transformNapi } = require('@swc-node/core')
 const { transformSync, transform } = require('@swc/core')
@@ -8,7 +9,6 @@ const chalk = require('chalk')
 const { transformSync: transformSyncEsbuild, startService } = require('esbuild')
 const ts = require('typescript')
 
-const os = require('os')
 const cpuCount = os.cpus().length
 
 const syncSuite = new Suite('Transform rxjs/AjaxObservable.ts benchmark')
@@ -95,7 +95,7 @@ async function runAsync() {
       name: '@swc-node/core',
       fn: (deferred) => {
         Promise.all(
-          new Array(cpuCount).map(() => {
+          Array.from({ length: cpuCount }).map(() => {
             return transformNapi(SOURCE_CODE, SOURCE_PATH, {
               target: 'es2016',
               module: 'commonjs',
@@ -118,7 +118,7 @@ async function runAsync() {
       name: '@swc/core',
       fn: (deferred) => {
         Promise.all(
-          new Array(cpuCount).map(() => {
+          Array.from({ length: cpuCount }).map(() => {
             return transform(SOURCE_CODE, {
               filename: SOURCE_PATH,
               jsc: {
@@ -148,15 +148,15 @@ async function runAsync() {
       name: 'esbuild',
       fn: (deferred) => {
         Promise.all(
-          new Array(cpuCount).map(() => {
+          Array.from({ length: cpuCount }).map(() =>
             service.transform(SOURCE_CODE, {
               sourcefile: SOURCE_PATH,
               loader: 'ts',
               sourcemap: true,
               minify: false,
               target: 'es2016',
-            })
-          }),
+            }),
+          ),
         ).then(() => {
           deferred.resolve()
         })
@@ -166,6 +166,7 @@ async function runAsync() {
       queued: true,
     })
     .on('cycle', function (event) {
+      event.target.hz = event.target.hz * cpuCount
       console.info(String(event.target))
     })
     .on('complete', function () {

diff --git a/packages/core/index.js b/packages/core/index.js
@@ -37,6 +37,7 @@ module.exports = {
         type: opts.module || 'commonjs',
       },
       sourceMaps: typeof opts.sourcemap === 'undefined' ? true : opts.sourcemap,
+      swcrc: false,
     }
     return bindings.transformSync(source, path, Buffer.from(JSON.stringify(swcOptions)))
   },
@@ -61,6 +62,7 @@ module.exports = {
       module: {
         type: opts.module || 'commonjs',
       },
+      swcrc: false,
       sourceMaps: typeof opts.sourcemap === 'undefined' ? true : opts.sourcemap,
     }
     return bindings.transform(source, path, Buffer.from(JSON.stringify(swcOptions)))