-
Notifications
You must be signed in to change notification settings - Fork 5.3k
[wasm] Initial emscripten 3.1.30 support #81215
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Merged
Merged
Changes from 1 commit
Commits
Show all changes
57 commits
Select commit
Hold shift + click to select a range
ed80ed3
[wasm] Initial emscripten 3.1.30 support
radekdoulik 96d59b8
Update icu dependency
radekdoulik 6512388
Merge remote-tracking branch 'remotes/origin/main' into pr-wasm-emscr…
radekdoulik 0d522ba
Update emsdk dependency
radekdoulik f5a205e
Use new docker images
radekdoulik 7d2e8eb
Add "icudt68_dat" as JS imported function.
maraf 44ede5e
Drop build-time support for undefined icudt68_dat.
maraf ab5c12e
Fix linker options
radekdoulik 2e74270
Merge remote-tracking branch 'remotes/origin/main' into pr-wasm-emscr…
radekdoulik e275857
Fix remaining conflict
radekdoulik 54dfd24
Limit new warnings option to browser
radekdoulik c302891
Try to force reinstall of certifi
radekdoulik 9f145a3
Unset `FROZEN_CACHE` for `embuilder build MINIMAL`
radekdoulik d6f6a4e
Update emsdk deps
radekdoulik 5067a1d
Remove assert for stack address
radekdoulik 0bb6b01
Get back the stack base assert
radekdoulik 04a83eb
Set stack size to 5MB
radekdoulik eb544bd
Disable WBT tests with SkiSharp
radekdoulik 8b4a6e2
Disable more WBT tests with SkiSharp
radekdoulik 6e8bd62
Set the stack size also in native targets
radekdoulik 48464d3
Merge branch 'main' into pr-wasm-emscripten-3-1-30
radekdoulik df724b5
Update icu deps
radekdoulik 2d67a1f
Update emsdk deps
radekdoulik e6ecd14
Enable BigInt support
radekdoulik 918a47a
Enable BigInt here as well
radekdoulik cde2398
Merge branch 'main' into pr-wasm-emscripten-3-1-30
radekdoulik b7999eb
Temporarily allow undefined symbols
radekdoulik 1107185
Merge branch 'main' into pr-wasm-emscripten-3-1-30
radekdoulik 4f5b216
Try to use pip-system-certs package
radekdoulik 0b3ce3c
Fix merge damage for marking "icudt68_dat" a JS imported function + r…
maraf 053f2d3
Try another python certifi package
radekdoulik a5a8498
Merge branch 'main' into pr-wasm-emscripten-3-1-30
lewing a977682
Try install pip-system-certs for emsdk's python
radekdoulik d8b1e10
Merge branch 'main' into pr-wasm-emscripten-3-1-30
radekdoulik 1e02d7c
Try to update machine certs for www.sqlite.org
radekdoulik e8f080b
Refactor + upgrade pip
radekdoulik 031511d
Use new net8 images
radekdoulik 889d036
Add `-s INCOMING_MODULE_JS_API=print,printErr`
radekdoulik f67ea5e
Introduce EmccStackSize msbuild property
radekdoulik 37f4314
Merge branch 'main' into pr-wasm-emscripten-3-1-30
radekdoulik c179e40
Remove FIXME from merge
radekdoulik d2ee742
Fixes around updateMemoryViews
radekdoulik 84dbae9
Pass --experimental-wasm-bigint to v8
radekdoulik b94be3d
Try multiple --engine-arg options
radekdoulik b551626
Revert "Introduce EmccStackSize msbuild property"
radekdoulik 4c45801
Disable 3 filesystem related tests
radekdoulik c5f073f
Add bigint to AOT test template
radekdoulik b2a99a8
Introduce EmccStackSize msbuild property
radekdoulik fda96f7
Update emsdk deps
radekdoulik 55fddbf
Update icu deps
radekdoulik 0da18d5
Revert "Add `-s INCOMING_MODULE_JS_API=print,printErr`"
radekdoulik eb7be72
Merge branch 'main' into pr-wasm-emscripten-3-1-30
radekdoulik 3fcf10f
Merge remote-tracking branch 'remotes/origin/main' into pr-wasm-emscr…
radekdoulik 6420bf6
Update emsdk deps
radekdoulik 58b5c1d
Update icu deps
radekdoulik 2a446e1
Disable one more file related test
radekdoulik ef6443c
Merge branch 'main' into pr-wasm-emscripten-3-1-30
lewing File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Next
Next commit
[wasm] Initial emscripten 3.1.30 support
- Loading branch information
commit ed80ed394d2d2467904a2ae9be31d6bda9703cdf
There are no files selected for viewing
This file contains hidden or 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
This file contains hidden or 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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1 +1 @@ | ||
| 3.1.12 | ||
| 3.1.30 |
This file contains hidden or 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
This file contains hidden or 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
This file contains hidden or 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
This file contains hidden or 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
This file contains hidden or 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
This file contains hidden or 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
This file contains hidden or 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
This file contains hidden or 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
This file contains hidden or 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
This file contains hidden or 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
This file contains hidden or 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
This file contains hidden or 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
This file contains hidden or 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
This file contains hidden or 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
This file contains hidden or 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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -26,6 +26,7 @@ export declare interface EmscriptenModule { | |
| HEAP8: Int8Array, | ||
| HEAP16: Int16Array; | ||
| HEAP32: Int32Array; | ||
| HEAP64: BigInt64Array; | ||
| HEAPU8: Uint8Array; | ||
| HEAPU16: Uint16Array; | ||
| HEAPU32: Uint32Array; | ||
|
|
@@ -37,8 +38,8 @@ export declare interface EmscriptenModule { | |
| _free(ptr: VoidPtr): void; | ||
|
|
||
| // this should match emcc -s EXPORTED_RUNTIME_METHODS | ||
| print(message: string): void; | ||
| printErr(message: string): void; | ||
| out(message: string): void; | ||
|
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. This will probably break Blazor. |
||
| err(message: string): void; | ||
| ccall<T>(ident: string, returnType?: string | null, argTypes?: string[], args?: any[], opts?: any): T; | ||
| cwrap<T extends Function>(ident: string, returnType: string, argTypes?: string[], opts?: any): T; | ||
| cwrap<T extends Function>(ident: string, ...args: any[]): T; | ||
|
|
||
This file contains hidden or 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
This file contains hidden or 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
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.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is API breaking change.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do we consider this API stable?
We can still get the deprecated ones with
-sINCOMING_MODULE_JS_API=print,printErrif needed.There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No
Yes please, I think these 2 are widely enough used even by Uno.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
They could be removed from .d.ts but should keep working.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm not sure we want to be in the business of maintaining unstable emscripten apis that emscripten has dropped long term but we should document the change and short term compat is fine.