[emscripten] Remove -sWASM_BIGINT flag

This flag has been enabled by default in emscripten for a while now
so I don't think we need this here anymore.

Also remove the `-Wno-experimental` flag which is no longer needed for
`-sMEMORY64`.

Also remove `-sMODULARIZE` which is implied by `-sEXPORT_ES6`.

Author: sbc100

CMakeLists.txt

@@ -297,21 +297,6 @@ else() # MSVC
 endif()
 
 if(EMSCRIPTEN)
-  # Note: to debug with DWARF you will usually want to enable BIGINT support, as
-  # that helps avoid running Binaryen on the wasm after link. Binaryen's DWARF
-  # rewriting has known limitations, so avoiding it during link is recommended
-  # where possible (like local debugging).
-  #
-  # Note that this is debug info for Binaryen itself, that is, when you are
-  # debugging Binaryen source code. This flag has no impact on what Binaryen
-  # does when run on wasm files.
-  option(ENABLE_BIGINT "Enable wasm BigInt support" OFF)
-  if(ENABLE_BIGINT)
-    add_link_flag("-sWASM_BIGINT")
-   else()
-    add_link_flag("-sWASM_BIGINT=0")
-  endif()
-
   if("${CMAKE_BUILD_TYPE}" MATCHES "Release")
     # Extra check that cmake has set -O3 in its release flags.
     # This is really as an assertion that cmake is behaving as we expect.
@@ -346,7 +331,6 @@ if(EMSCRIPTEN)
     add_link_flag("-sENVIRONMENT=web,worker")
     add_link_flag("-sINVOKE_RUN=0")
     add_link_flag("-sEXPORTED_RUNTIME_METHODS=run,callMain,FS")
-    add_link_flag("-sMODULARIZE")
     add_link_flag("-sEXPORT_ES6")
     add_link_flag("-sFILESYSTEM")
     add_link_flag("-sFORCE_FILESYSTEM")
@@ -359,7 +343,7 @@ if(EMSCRIPTEN)
     add_nondebug_compile_flag("-flto")
   endif()
   if(EMSCRIPTEN_ENABLE_WASM64)
-    add_compile_flag("-sMEMORY64 -Wno-experimental")
+    add_compile_flag("-sMEMORY64")
     add_link_flag("-sMEMORY64")
   endif()
 endif()

scripts/test/shared.py

@@ -195,7 +195,7 @@ def is_exe(fpath):
             which('gcc') or which('clang'))
 NATIVEXX = (os.environ.get('CXX') or which('mingw32-g++') or
             which('g++') or which('clang++'))
-NODEJS = os.environ.get('NODE') or which('node') or which('nodejs')
+NODEJS = os.environ.get('NODE') or os.environ.get('EMSDK_NODE') or which('node') or which('nodejs')
 MOZJS = which('mozjs') or which('spidermonkey')
 
 V8 = os.environ.get('V8') or which('v8') or which('d8')
@@ -264,9 +264,8 @@ def has_shell_timeout():
 
 try:
     if NODEJS is not None:
-        subprocess.check_call([NODEJS, '--version'],
-                              stdout=subprocess.PIPE,
-                              stderr=subprocess.PIPE)
+        version = subprocess.check_output([NODEJS, '--version'])
+        print(f'Using node ({NODEJS}) version: {version.strip()}')
 except (OSError, subprocess.CalledProcessError):
     NODEJS = None
 if NODEJS is None: