Skip to content

Emscripten compilation and execution in browsers#787

Draft
j8asic wants to merge 13 commits intoshader-slang:mainfrom
j8asic:main
Draft

Emscripten compilation and execution in browsers#787
j8asic wants to merge 13 commits intoshader-slang:mainfrom
j8asic:main

Conversation

@j8asic
Copy link

@j8asic j8asic commented Feb 6, 2026

image

This is a draft PR where SGL compiles using Emscripten.

It relies on RHI PR to be merged: shader-slang/slang-rhi#609 and Slang's PR shader-slang/slang#9818 to produce WASM libraries to be automatically used for RHI/SGL compilation

@j8asic j8asic requested a review from a team as a code owner February 6, 2026 14:47
@CLAassistant
Copy link

CLAassistant commented Feb 6, 2026

CLA assistant check
All committers have signed the CLA.

@j8asic j8asic marked this pull request as draft February 6, 2026 18:42
Copy link
Contributor

@ccummingsNV ccummingsNV left a comment

Choose a reason for hiding this comment

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

This is exciting stuff and looks mostly sane to me, but I'd need @skallweitNV to take a look as he understands the build process better than me.

I don't think we can really ship this without it being part of the testing suite either. Currently almost all tests run on all platforms. How do we want to approach that.

@skallweitNV
Copy link
Contributor

@j8asic thanks for the contribution. This is really cool to see! Let's first merge the slang-rhi PR and then get to this one after that.

@coderabbitai
Copy link

coderabbitai bot commented Feb 24, 2026

Important

Review skipped

Draft detected.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

Comment @coderabbitai help to get the list of available commands and usage tips.

@j8asic
Copy link
Author

j8asic commented Feb 24, 2026

thanks @ccummingsNV @skallweitNV; bumping to the latest slang version and merging the slang-rhi PR, this is almost ready for review.
one final question from my side: what does thread.h do, or better to say, what do you want it to do in the context of browsers?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants