feat: ugly hack in pre-js file to enable multi thread on browser

InfiniBrains · Dec 13, 2022 · ceb6f1c · ceb6f1c
1 parent 1590cf1
commit ceb6f1c
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 2 deletions.
diff --git a/CMakeLists.txt b/CMakeLists.txt
@@ -55,7 +55,7 @@ add_library(${PROJECT_NAME} ${headers} ${sources})
 set_target_properties(${PROJECT_NAME} PROPERTIES CXX_STANDARD 20)
 
 if(EMSCRIPTEN)
-  set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -Oz -s WASM=2 -pthread -s USE_PTHREADS=1")
+  set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -Oz -s WASM=2 -pthread -s USE_PTHREADS=1 --extern-pre-js ${CMAKE_CURRENT_SOURCE_DIR}/cmake/pre.js")
 endif()
 
 # being a cross-platform target, we enforce standards conformance on MSVC

diff --git a/cmake/pre.js b/cmake/pre.js
@@ -0,0 +1,3 @@
+// todo: move this to a better place
+// ugly hack for crossOriginIsolated not enabled and SharedArrayBuffer not available
+if(typeof(window)=='object' && (typeof(crossOriginIsolated)=='undefined'||!crossOriginIsolated))SharedArrayBuffer = ArrayBuffer;
diff --git a/standalone/CMakeLists.txt b/standalone/CMakeLists.txt
@@ -28,7 +28,7 @@ if(EMSCRIPTEN)
   # todo: add pre js to have this if (!crossOriginIsolated) SharedArrayBuffer = ArrayBuffer;
   set(CMAKE_EXECUTABLE_SUFFIX ".html") # todo: we should not need this!
   message(STATUS "EMSCRIPTEN BUILD")
-  set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -Oz -s WASM=2 -pthread -s USE_PTHREADS=1")
+  set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -Oz -s WASM=2 -pthread -s USE_PTHREADS=1 --extern-pre-js ${CMAKE_CURRENT_SOURCE_DIR}/../cmake/pre.js")
 endif()
 
 message(STATUS "CMAKE_EXECUTABLE_SUFFIX: ${CMAKE_EXECUTABLE_SUFFIX}")