build: Add support for WASI in CMake files #31672
Conversation
cmake/modules/AddSwift.cmake Outdated
There was a problem hiding this comment.
I don't think it makes sense to build the toolchain for WASI. Why does the toolchain need to be built for WASI?
There was a problem hiding this comment.
It makes a huge amount of sense to build the toolchain for WASI (not that anyone has done so today, AFAIK).
For one thing, it would allow you to run the compiler in your browser. Very cool, and a great learning tool. Another use-case would be to bootstrap building the compiler as projects like Swift-driver make progress and more Swift is used in the compiler itself.
MaxDesiatov changed the title cmake/modules/AddSwift.cmake Outdated
There was a problem hiding this comment.
This shouldn't happen right? Because this is for the toolchain.
There was a problem hiding this comment.
It shouldn't, and didn't happen in my practice building it, was written more like an assertion here. I can remove it if you'd like.
There was a problem hiding this comment.
Please do, I think that minimizing the CMake logic is preferable.
There was a problem hiding this comment.
That's cleaned up now.
| Hmm, how are there no changes to the |
| Changes to |
MaxDesiatov force-pushed the maxd/wasi-sdk branch from 1d8e51f to 7620796 Compare | @swift-ci please test |
3 participants
Add basic support for WebAssembly/WASI builds in CMake files. Important to note that WebAssembly/WASI doesn't fully support dynamic linking yet, so it is disabled for now.
Related to SR-9307.
This is a part of #29465, but includes only the CMake bits that are self-contained.
(cc @compnerd @kateinoigakukun)