Merged blaugold's WASM branch, plus fixes #2069
Draft
+529
−98
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
(cherry picked from commit 9e3f4d7) # Conflicts: # C/tests/CMakeLists.txt # C/tests/c4DatabaseTest.cc # LiteCore/Support/Error.cc # LiteCore/Support/FilePath.cc # LiteCore/Support/LogDecoder.cc # LiteCore/Support/Logging.cc # LiteCore/Support/MultiLogDecoder.hh # LiteCore/Support/StringUtil.cc # LiteCore/Support/StringUtil.hh # LiteCore/Support/ThreadUtil.cc # LiteCore/tests/DataFileTest.cc # LiteCore/tests/SQLiteFunctionsTest.cc # REST/tests/RESTListenerTest.cc
CppTests:
test cases: 499 | 496 passed | 3 failed
assertions: 682799 | 682794 passed | 5 failed
failed tests:
>>> Query Functions
>>> RealClockSource
>>> Continuous Fast Push
C4Tests:
All tests passed (791941 assertions in 155 test cases)
|
Code Coverage Results:
|
|
FYI, in a (In a For comparison, in a Mac build C4Tests is 4.7MB, or about 2.3MB per architecture; CppTests is 21MB, or about 10.5MB per architecture [which is surprisingly big...] |
- Defined the compile/link flags as variables and reuse them - Enabled LTO - Forced -Os, since build type MinSizeRel doesn't seem to do anything - Changed build type to Release, since the debug symbols add ~50MB to the .wasm files
|
To make the WASM smaller I
This reduced C4Tests to 3.1MB (1.1MB gzipped), and CppTests to 5.2MB (1.7MB gzipped). Nice! |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Cherry-picked @blaugold 's branch that adds WASM support via emscripten.
Made some changes to get it to build again.
This depends on the emscripten support in Fleece, currently couchbase/fleece#232
I chose to disable all of the socket and listener code -- BuiltInWebSocket, etc. It doesn't make sense to have code that uses custom TCP sockets in a library targeted at web browsers. Instead the hypothetical JS platform code would use the C4Socket API to bridge to the JS WebSocket class, the same way the other (non-C) platforms do.