Skip to content

Commit cdc89d0

Browse files
committed
test: unskip v8-updates/test-linux-perf-logger
Refs: nodejs#51308 nodejs#52821
1 parent be8d64e commit cdc89d0

File tree

1 file changed

+7
-3
lines changed

1 file changed

+7
-3
lines changed

test/v8-updates/test-linux-perf-logger.js

+7-3
Original file line numberDiff line numberDiff line change
@@ -20,7 +20,11 @@
2020
//
2121
// NOTE: This test runs only on linux, as that is the only platform supported by perf, and
2222
// accordingly the only platform where `perf-basic-prof*` v8 flags are available.
23-
23+
//
24+
// MAINTAINERS' NOTE: As of early 2024, the most common failure mode for this test suite
25+
// is for v8 options to change from version to version. If this suite fails, look there first.
26+
// We use options to forcibly require certain test cases to JIT code, and the nodeFlags to do
27+
// so can change.
2428

2529
const common = require('../common');
2630
if (!common.isLinux) {
@@ -48,7 +52,7 @@ const testCases = [
4852
},
4953
{
5054
title: '--perf-basic-prof compiled',
51-
nodeFlags: ['--perf-basic-prof', '--no-turbo-inlining', '--always-turbofan'],
55+
nodeFlags: ['--perf-basic-prof', '--no-turbo-inlining', '--always-turbofan', '--minimum-invocations-before-optimization=0'],
5256
matches: [
5357
'test-regex',
5458
'~functionOne .+/linux-perf-logger.js',
@@ -66,7 +70,7 @@ const testCases = [
6670
},
6771
{
6872
title: '--perf-basic-prof-only-functions compiled',
69-
nodeFlags: ['--perf-basic-prof-only-functions', '--no-turbo-inlining', '--always-turbofan'],
73+
nodeFlags: ['--perf-basic-prof-only-functions', '--no-turbo-inlining', '--always-turbofan', '--minimum-invocations-before-optimization=0'],
7074
matches: [
7175
'~functionOne .+/linux-perf-logger.js',
7276
'~functionTwo .+/linux-perf-logger.js',

0 commit comments

Comments
 (0)