From 72b6cea25d4e43b2db63eeb32417088c5bd93307 Mon Sep 17 00:00:00 2001
From: Shelley Vohr <shelley.vohr@gmail.com>
Date: Thu, 20 Feb 2020 16:58:58 -0800
Subject: [PATCH] cli: --perf-prof only works on Linux

PR-URL: https://github.com/nodejs/node/pull/31892
Reviewed-By: Matheus Marchini <mat@mmarchini.me>
Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com>
---
 doc/api/cli.md                         |  3 +++
 test/parallel/test-cli-node-options.js | 11 +++++------
 2 files changed, 8 insertions(+), 6 deletions(-)

diff --git a/doc/api/cli.md b/doc/api/cli.md
index 89d5f62d14f683..8b45a4b5bb3b99 100644
--- a/doc/api/cli.md
+++ b/doc/api/cli.md
@@ -1164,6 +1164,9 @@ V8 options that are allowed are:
 * `--stack-trace-limit`
 <!-- node-options-v8 end -->
 
+`--perf-basic-prof-only-functions`, `--perf-basic-prof`,
+`--perf-prof-unwinding-info`, and `--perf-prof` are only available on Linux.
+
 ### `NODE_PATH=path[:…]`
 <!-- YAML
 added: v0.1.32
diff --git a/test/parallel/test-cli-node-options.js b/test/parallel/test-cli-node-options.js
index 51a0d9a952e614..49f8371a1aaf08 100644
--- a/test/parallel/test-cli-node-options.js
+++ b/test/parallel/test-cli-node-options.js
@@ -48,15 +48,14 @@ expectNoWorker('--trace-event-file-pattern {pid}-${rotation}.trace_events ' +
        '--trace-event-categories node.async_hooks', 'B\n');
 expect('--unhandled-rejections=none', 'B\n');
 
-if (!common.isWindows) {
+if (common.isLinux) {
   expect('--perf-basic-prof', 'B\n');
   expect('--perf-basic-prof-only-functions', 'B\n');
-}
 
-if (common.isLinux && ['arm', 'x64'].includes(process.arch)) {
-  // PerfJitLogger is only implemented in Linux.
-  expect('--perf-prof', 'B\n');
-  expect('--perf-prof-unwinding-info', 'B\n');
+  if (['arm', 'x64'].includes(process.arch)) {
+    expect('--perf-prof', 'B\n');
+    expect('--perf-prof-unwinding-info', 'B\n');
+  }
 }
 
 if (common.hasCrypto) {