Skip to content

[browser] emscripten 6.0.2 follow up#130614

Merged
pavelsavara merged 7 commits into
dotnet:mainfrom
pavelsavara:emscripten6_prep
Jul 15, 2026
Merged

[browser] emscripten 6.0.2 follow up#130614
pavelsavara merged 7 commits into
dotnet:mainfrom
pavelsavara:emscripten6_prep

Conversation

@pavelsavara

@pavelsavara pavelsavara commented Jul 13, 2026

Copy link
Copy Markdown
Member

Bump to emsdk 6.0.2 (with dotnet patches) landed in #130631

  • Skip adding O_NOCTTY in SystemNative_Open when targeting WASM.
  • Add __stack_pointer as an explicit JS-library dependency
  • rename isSharedArrayBuffer to arrayBufferNeedsCopy and also respect buffer.resizable

@pavelsavara pavelsavara added this to the 11.0.0 milestone Jul 13, 2026
@pavelsavara pavelsavara self-assigned this Jul 13, 2026
Copilot AI review requested due to automatic review settings July 13, 2026 10:45
@pavelsavara pavelsavara added arch-wasm WebAssembly architecture area-Host os-browser Browser variant of arch-wasm labels Jul 13, 2026
@dotnet-policy-service

Copy link
Copy Markdown
Contributor

Tagging subscribers to 'arch-wasm': @lewing, @pavelsavara
See info in area-owners.md if you want to be subscribed.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Prepares the WASM (browser) toolchain/runtime integration for newer Emscripten behavior by (1) avoiding an unsupported open flag under WASM and (2) ensuring emscripten_stack_get_current is preserved/exported for JS/native interop.

Changes:

  • Skip adding O_NOCTTY in SystemNative_Open when targeting WASM.
  • Add emscripten_stack_get_current as an explicit JS-library dependency for browser utils and corerun WASM host code.
  • Export _emscripten_stack_get_current from the Emscripten link step.

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 1 comment.

File Description
src/native/libs/System.Native/pal_io.c Avoids O_NOCTTY on WASM targets to prevent unsupported flag rejection.
src/native/libs/System.Native.Browser/libSystem.Native.Browser.Utils.footer.js Adds emscripten_stack_get_current to the browser utils dependency list.
src/coreclr/hosts/corerun/wasm/libCorerun.js Adds emscripten_stack_get_current to corerun’s WASM dependency list.
eng/native.wasm.targets Exports _emscripten_stack_get_current during browser WASM linking.

Comment thread src/native/libs/System.Native/pal_io.c
Copilot AI review requested due to automatic review settings July 13, 2026 11:05

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 16 out of 16 changed files in this pull request and generated 3 comments.

Comment thread src/native/libs/System.Native/pal_io.c
Comment thread src/native/libs/System.Native.Browser/utils/memory.ts Outdated
Comment thread src/native/libs/System.Native.Browser/native/crypto.ts
Copilot AI review requested due to automatic review settings July 13, 2026 11:45

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 16 out of 16 changed files in this pull request and generated 3 comments.

Comment thread src/native/libs/System.Native/pal_io.c
Comment thread src/native/libs/System.Native.Browser/utils/memory.ts
Comment thread src/native/libs/System.Native.Browser/native/crypto.ts
Copilot AI review requested due to automatic review settings July 13, 2026 11:52

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 15 out of 15 changed files in this pull request and generated 2 comments.

Comment thread src/native/libs/System.Native.Browser/native/crypto.ts
Comment thread src/coreclr/hosts/corerun/wasm/libCorerun.js
Copilot AI review requested due to automatic review settings July 15, 2026 08:21

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 15 out of 15 changed files in this pull request and generated 4 comments.

Comment thread src/native/libs/System.Native.Browser/utils/memory.ts
Comment thread src/mono/browser/runtime/strings.ts
Comment thread src/mono/browser/runtime/crypto.ts
Comment thread src/native/libs/System.Native.Browser/native/crypto.ts
@pavelsavara
pavelsavara marked this pull request as ready for review July 15, 2026 12:45
@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines:
Successfully started running 4 pipeline(s).
11 pipeline(s) were filtered out due to trigger conditions.
There may be pipelines that require an authorized user to comment /azp run to run.

@pavelsavara pavelsavara changed the title [browser] emscripten6 prep [browser] emscripten 6.0.x follow up Jul 15, 2026
@pavelsavara pavelsavara changed the title [browser] emscripten 6.0.x follow up [browser] emscripten 6.0.2 follow up Jul 15, 2026
@pavelsavara

Copy link
Copy Markdown
Member Author

/ba-g unrelated failures

@pavelsavara
pavelsavara merged commit 5f7b765 into dotnet:main Jul 15, 2026
177 of 181 checks passed
@pavelsavara
pavelsavara deleted the emscripten6_prep branch July 15, 2026 22:07
@dotnet-milestone-bot dotnet-milestone-bot Bot modified the milestones: 11.0.0, 11.0-preview7 Jul 16, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

arch-wasm WebAssembly architecture area-Host os-browser Browser variant of arch-wasm

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants