Skip to content

Commit ad8be95

Browse files
build: set v8_enable_webassembly=false when lite mode is enabled
We should set v8_enable_webassembly=false when lite mode is enabled, since v8_enable_lite_mode implies v8_jitless, and wasm currently doesn't work with jitless. This is automatically handled in gn, but seems to be not the case in gyp. Enabling lite mode without setting v8_enable_webassembly=false leads to "Warning: disabling flag --expose_wasm due to conflicting flags" at runtime. PR-URL: nodejs#48248 Reviewed-By: Ben Noordhuis <[email protected]> Reviewed-By: Michaël Zasso <[email protected]> Reviewed-By: Richard Lau <[email protected]> Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: Yagiz Nizipli <[email protected]>
1 parent 30243e5 commit ad8be95

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

configure.py

+1-1
Original file line numberDiff line numberDiff line change
@@ -1471,7 +1471,7 @@ def configure_library(lib, output, pkgname=None):
14711471

14721472

14731473
def configure_v8(o):
1474-
o['variables']['v8_enable_webassembly'] = 1
1474+
o['variables']['v8_enable_webassembly'] = 0 if options.v8_lite_mode else 1
14751475
o['variables']['v8_enable_javascript_promise_hooks'] = 1
14761476
o['variables']['v8_enable_lite_mode'] = 1 if options.v8_lite_mode else 0
14771477
o['variables']['v8_enable_gdbjit'] = 1 if options.gdb else 0

0 commit comments

Comments
 (0)