Skip to content

Commit 84481f9

Browse files
bzozjasnell
authored andcommitted
benchmark: add --expose_internals switch
Adds --expose_internals switch to benchmark runner. This makes misc/freelist.js benchmark run properly Reviewed-By: James M Snell <[email protected]> Reviewed-By: Andreas Madsen <[email protected]> PR-URL: #8547
1 parent e97fe46 commit 84481f9

File tree

2 files changed

+6
-2
lines changed

2 files changed

+6
-2
lines changed

benchmark/common.js

+2-1
Original file line numberDiff line numberDiff line change
@@ -137,7 +137,8 @@ Benchmark.prototype._run = function() {
137137
}
138138

139139
const child = child_process.fork(require.main.filename, childArgs, {
140-
env: childEnv
140+
env: childEnv,
141+
execArgv: ['--expose_internals'].concat(process.execArgv)
141142
});
142143
child.on('message', sendResult);
143144
child.on('close', function(code) {

benchmark/misc/freelist.js

+4-1
Original file line numberDiff line numberDiff line change
@@ -1,13 +1,16 @@
11
'use strict';
22

33
var common = require('../common.js');
4-
var FreeList = require('internal/freelist').FreeList;
54

65
var bench = common.createBenchmark(main, {
76
n: [100000]
87
});
98

109
function main(conf) {
10+
// Using internal/freelist requires node to be run with --expose_internals
11+
// switch. common.js will do that when calling main(), so we require
12+
// this module here
13+
const FreeList = require('internal/freelist').FreeList;
1114
var n = conf.n;
1215
var poolSize = 1000;
1316
var list = new FreeList('test', poolSize, Object);

0 commit comments

Comments
 (0)