1berry/nodejs
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

build: replace runtime flag with compiler option

Browse Source 
V8 changed the typed array threshold option from a runtime flag to a
compile-time option.

PR-URL: https://github.com/nodejs/node/pull/16271
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
Reviewed-By: Myles Borins <myles.borins@gmail.com>
This commit is contained in:
Peter Marshall 2017-09-14 12:51:54 +02:00 committed by Michaël Zasso
parent a1ed29b1c6
commit 2c75b52af8
2 changed files with 4 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
configure vendored
View File

@ -1027,6 +1027,10 @@ def configure_v8(o):
o['variables']['node_use_bundled_v8'] = b(not options.without_bundled_v8) o['variables']['node_use_bundled_v8'] = b(not options.without_bundled_v8)
o['variables']['force_dynamic_crt'] = 1 if options.shared else 0 o['variables']['force_dynamic_crt'] = 1 if options.shared else 0
o['variables']['node_enable_d8'] = b(options.enable_d8) o['variables']['node_enable_d8'] = b(options.enable_d8)
# Unconditionally force typed arrays to allocate outside the v8 heap. This
# is to prevent memory pointers from being moved around that are returned by
# Buffer::Data().
o['variables']['v8_typed_array_max_size_in_heap'] = 0
if options.enable_d8: if options.enable_d8:
o['variables']['test_isolation_mode'] = 'noop' # Needed by d8.gyp. o['variables']['test_isolation_mode'] = 'noop' # Needed by d8.gyp.
if options.without_bundled_v8 and options.enable_d8: if options.without_bundled_v8 and options.enable_d8:

6
src/node.cc
View File

@ -4232,12 +4232,6 @@ void Init(int* argc,
} }
#endif #endif
// Unconditionally force typed arrays to allocate outside the v8 heap. This
// is to prevent memory pointers from being moved around that are returned by
// Buffer::Data().
const char no_typed_array_heap[] = "--typed_array_max_size_in_heap=0";
V8::SetFlagsFromString(no_typed_array_heap, sizeof(no_typed_array_heap) - 1);
// Needed for access to V8 intrinsics. Disabled again during bootstrapping, // Needed for access to V8 intrinsics. Disabled again during bootstrapping,
// see lib/internal/bootstrap_node.js. // see lib/internal/bootstrap_node.js.
const char allow_natives_syntax[] = "--allow_natives_syntax"; const char allow_natives_syntax[] = "--allow_natives_syntax";