Commit aa52873
daomingq
build: add configure option --v8-enable-short-builtin-calls
Add configure option --v8-enable-short-builtin-calls
and enable it by default on x86_64 platform.
PR-URL: #42109
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: Jiawen Geng <technicalcute@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Richard Lau <rlau@redhat.com>1 parent db7fa9f commit aa52873
2 files changed
Lines changed: 15 additions & 0 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
@@ -782,6 +782,13 @@ | |||
| 782 | 782 | help='Enable V8 transparent hugepage support. This feature is only '+ | |
| 783 | 783 | 'available on Linux platform.') | |
| 784 | 784 | ||
| 785 | + parser.add_argument('--v8-enable-short-builtin-calls', | ||
| 786 | + action='store_true', | ||
| 787 | + dest='v8_enable_short_builtin_calls', | ||
| 788 | + default=None, | ||
| 789 | + help='Enable V8 short builtin calls support. This feature is enabled '+ | ||
| 790 | + 'on x86_64 platform by default.') | ||
| 791 | + | ||
| 785 | 792 | parser.add_argument('--node-builtin-modules-path', | |
| 786 | 793 | action='store', | |
| 787 | 794 | dest='node_builtin_modules_path', | |
@@ -1464,6 +1471,8 @@ def configure_v8(o): | |||
| 1464 | 1471 | if flavor != 'linux' and options.v8_enable_hugepage: | |
| 1465 | 1472 | raise Exception('--v8-enable-hugepage is supported only on linux.') | |
| 1466 | 1473 | o['variables']['v8_enable_hugepage'] = 1 if options.v8_enable_hugepage else 0 | |
| 1474 | + if options.v8_enable_short_builtin_calls or o['variables']['target_arch'] == 'x64': | ||
| 1475 | + o['variables']['v8_enable_short_builtin_calls'] = 1 | ||
| 1467 | 1476 | ||
| 1468 | 1477 | def configure_openssl(o): | |
| 1469 | 1478 | variables = o['variables'] | |
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
@@ -132,6 +132,9 @@ | |||
| 132 | 132 | 'v8_enable_pointer_compression%': 0, | |
| 133 | 133 | 'v8_enable_31bit_smis_on_64bit_arch%': 0, | |
| 134 | 134 | ||
| 135 | + # Sets -dV8_SHORT_BUILTIN_CALLS | ||
| 136 | + 'v8_enable_short_builtin_calls%': 0, | ||
| 137 | + | ||
| 135 | 138 | # Sets -dOBJECT_PRINT. | |
| 136 | 139 | 'v8_enable_object_print%': 0, | |
| 137 | 140 | ||
@@ -294,6 +297,9 @@ | |||
| 294 | 297 | ['v8_enable_pointer_compression==1 or v8_enable_31bit_smis_on_64bit_arch==1', { | |
| 295 | 298 | 'defines': ['V8_31BIT_SMIS_ON_64BIT_ARCH',], | |
| 296 | 299 | }], | |
| 300 | + ['v8_enable_short_builtin_calls==1', { | ||
| 301 | + 'defines': ['V8_SHORT_BUILTIN_CALLS',], | ||
| 302 | + }], | ||
| 297 | 303 | ['v8_enable_zone_compression==1', { | |
| 298 | 304 | 'defines': ['V8_COMPRESS_ZONES',], | |
| 299 | 305 | }], | |
0 commit comments