summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorAlon Zakai <azakai@google.com>2020-09-28 16:49:35 -0700
committerGitHub <noreply@github.com>2020-09-28 16:49:35 -0700
commit2a869194c5fb7f54b3811043bfcf723e3d53c1df (patch)
treea66996c5c38b7b1a1b68a642fb611223c883d686
parent5a4db93bac7e123910ea173bb0a9c9f61bc6ee78 (diff)
downloadbinaryen-2a869194c5fb7f54b3811043bfcf723e3d53c1df.tar.gz
binaryen-2a869194c5fb7f54b3811043bfcf723e3d53c1df.tar.bz2
binaryen-2a869194c5fb7f54b3811043bfcf723e3d53c1df.zip
Restore minification in emscripten builds (#3173)
The emscripten bug has been fixed in emscripten-core/emscripten#12329
-rw-r--r--CMakeLists.txt9
1 files changed, 4 insertions, 5 deletions
diff --git a/CMakeLists.txt b/CMakeLists.txt
index 2b6a198c3..8e40de635 100644
--- a/CMakeLists.txt
+++ b/CMakeLists.txt
@@ -355,11 +355,10 @@ if(EMSCRIPTEN)
target_link_libraries(binaryen_wasm "--post-js ${CMAKE_CURRENT_SOURCE_DIR}/src/js/binaryen.js-post.js")
target_link_libraries(binaryen_wasm "--extern-pre-js ${CMAKE_CURRENT_SOURCE_DIR}/src/js/binaryen.js-extern-pre.js")
target_link_libraries(binaryen_wasm "--extern-post-js ${CMAKE_CURRENT_SOURCE_DIR}/src/js/binaryen.js-extern-post.js")
- target_link_libraries(binaryen_wasm optimized "--closure 0")
+ target_link_libraries(binaryen_wasm optimized "--closure 1")
target_link_libraries(binaryen_wasm optimized "--closure-args \"--language_in=ECMASCRIPT6 --language_out=ECMASCRIPT6\"")
target_link_libraries(binaryen_wasm optimized "-flto")
- # TODO: put profiling only on debug, and restore closure. in binaryen_js too.
- target_link_libraries(binaryen_wasm "--profiling")
+ target_link_libraries(binaryen_wasm debug "--profiling")
set_property(TARGET binaryen_wasm PROPERTY CXX_STANDARD ${CXX_STANDARD})
set_property(TARGET binaryen_wasm PROPERTY CXX_STANDARD_REQUIRED ON)
install(TARGETS binaryen_wasm DESTINATION ${CMAKE_INSTALL_BINDIR})
@@ -380,10 +379,10 @@ if(EMSCRIPTEN)
target_link_libraries(binaryen_js "--post-js ${CMAKE_CURRENT_SOURCE_DIR}/src/js/binaryen.js-post.js")
target_link_libraries(binaryen_js "--extern-pre-js ${CMAKE_CURRENT_SOURCE_DIR}/src/js/binaryen.js-extern-pre.js")
target_link_libraries(binaryen_js "--extern-post-js ${CMAKE_CURRENT_SOURCE_DIR}/src/js/binaryen.js-extern-post.js")
- target_link_libraries(binaryen_js optimized "--closure 0")
+ target_link_libraries(binaryen_js optimized "--closure 1")
target_link_libraries(binaryen_js optimized "--closure-args \"--language_in=ECMASCRIPT6 --language_out=ECMASCRIPT6\"")
target_link_libraries(binaryen_js optimized "-flto")
- target_link_libraries(binaryen_js "--profiling")
+ target_link_libraries(binaryen_js debug "--profiling")
target_link_libraries(binaryen_js debug "-s ASSERTIONS")
set_property(TARGET binaryen_js PROPERTY CXX_STANDARD ${CXX_STANDARD})
set_property(TARGET binaryen_js PROPERTY CXX_STANDARD_REQUIRED ON)