Commit 638b510
cli: add --expose-gc flag available to NODE_OPTIONS
This commits allows users to send `--expose-gc` via `NODE_OPTIONS`
environment variable.
Using `node --expose-gc` is possible but via `NODE_OPTIONS` won't
work.
```sh
NODE_OPTIONS='--expose-gc' node
node: --expose-gc is not allowed in NODE_OPTIONS
```
Signed-off-by: Juan José Arboleda <soyjuanarbol@gmail.com>
PR-URL: #53078
Reviewed-By: Tierney Cyren <hello@bnb.im>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Joyee Cheung <joyeec9h3@gmail.com>
Reviewed-By: Chengzhong Wu <legendecas@gmail.com>1 parent 4a17dda commit 638b510
3 files changed
Lines changed: 22 additions & 0 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
@@ -673,6 +673,23 @@ Make built-in language features like `eval` and `new Function` that generate | |||
| 673 | 673 | code from strings throw an exception instead. This does not affect the Node.js | |
| 674 | 674 | `node:vm` module. | |
| 675 | 675 | ||
| 676 | + ### `--expose-gc` | ||
| 677 | + | ||
| 678 | + <!-- YAML | ||
| 679 | + added: REPLACEME | ||
| 680 | + --> | ||
| 681 | + | ||
| 682 | + > Stability: 1 - Experimental. This flag is inherited from V8 and is subject to | ||
| 683 | + > change upstream. | ||
| 684 | + | ||
| 685 | + This flag will expose the gc extension from V8. | ||
| 686 | + | ||
| 687 | + ```js | ||
| 688 | + if (globalThis.gc) { | ||
| 689 | + globalThis.gc(); | ||
| 690 | + } | ||
| 691 | + ``` | ||
| 692 | + | ||
| 676 | 693 | ### `--dns-result-order=order` | |
| 677 | 694 | ||
| 678 | 695 | <!-- YAML | |
@@ -2913,6 +2930,7 @@ V8 options that are allowed are: | |||
| 2913 | 2930 | * `--abort-on-uncaught-exception` | |
| 2914 | 2931 | * `--disallow-code-generation-from-strings` | |
| 2915 | 2932 | * `--enable-etw-stack-walking` | |
| 2933 | + * `--expose-gc` | ||
| 2916 | 2934 | * `--huge-max-old-generation-size` | |
| 2917 | 2935 | * `--interpreted-frames-native-stack` | |
| 2918 | 2936 | * `--jitless` | |
@@ -3242,6 +3260,8 @@ documented here: | |||
| 3242 | 3260 | ||
| 3243 | 3261 | ### `--enable-etw-stack-walking` | |
| 3244 | 3262 | ||
| 3263 | + ### `--expose-gc` | ||
| 3264 | + | ||
| 3245 | 3265 | ### `--harmony-shadow-realm` | |
| 3246 | 3266 | ||
| 3247 | 3267 | ### `--huge-max-old-generation-size` | |
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
@@ -489,6 +489,7 @@ EnvironmentOptionsParser::EnvironmentOptionsParser() { | |||
| 489 | 489 | AddOption("--experimental-report", "", NoOp{}, kAllowedInEnvvar); | |
| 490 | 490 | AddOption( | |
| 491 | 491 | "--experimental-wasi-unstable-preview1", "", NoOp{}, kAllowedInEnvvar); | |
| 492 | + AddOption("--expose-gc", "expose gc extension", V8Option{}, kAllowedInEnvvar); | ||
| 492 | 493 | AddOption("--expose-internals", "", &EnvironmentOptions::expose_internals); | |
| 493 | 494 | AddOption("--frozen-intrinsics", | |
| 494 | 495 | "experimental frozen intrinsics support", | |
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
@@ -69,6 +69,7 @@ if (common.hasCrypto) { | |||
| 69 | 69 | // V8 options | |
| 70 | 70 | expect('--abort_on-uncaught_exception', 'B\n'); | |
| 71 | 71 | expect('--disallow-code-generation-from-strings', 'B\n'); | |
| 72 | + expect('--expose-gc', 'B\n'); | ||
| 72 | 73 | expect('--huge-max-old-generation-size', 'B\n'); | |
| 73 | 74 | expect('--jitless', 'B\n'); | |
| 74 | 75 | expect('--max-old-space-size=0', 'B\n'); | |
0 commit comments