Skip to content

Add support for some new WASI targets:#99

Merged
sunfishcode merged 1 commit intobytecodealliance:mainfrom
alexcrichton:wasi-targets
Feb 21, 2024
Merged

Add support for some new WASI targets:#99
sunfishcode merged 1 commit intobytecodealliance:mainfrom
alexcrichton:wasi-targets

Conversation

@alexcrichton
Copy link
Copy Markdown
Member

@alexcrichton alexcrichton commented Jan 30, 2024

  • wasm32-wasip1
  • wasm32-wasip2
  • wasm32-wasip1-threads

@bjorn3
Copy link
Copy Markdown
Contributor

bjorn3 commented Feb 21, 2024

Gentle ping. rust-lang/miri#722 (comment) needs this to compile cg_clif for wasm32-wasi-preview1-threads.

@sunfishcode
Copy link
Copy Markdown
Member

Following this discussion, we should change these names to wasm32-wasip1 and wasm32-wasip2.

* `wasm32-wasip1`
* `wasm32-wasip2`
* `wasm32-wasip1-threads`
@alexcrichton
Copy link
Copy Markdown
Member Author

Oops yes, updated now!

@sunfishcode sunfishcode merged commit c1e9b35 into bytecodealliance:main Feb 21, 2024
@sunfishcode
Copy link
Copy Markdown
Member

This is now released in target-lexicon 0.12.14.

"wasm32-unknown-emscripten",
"wasm32-unknown-unknown",
"wasm32-wasi",
"wasm32-wasip1-threads",
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.

wasm32-wasi-preview1-threads hasn't been renamed to wasm32-wasip1-threads yet and there is no open PR for this either.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Indeed! I was a bit forward thinking adding this in, so I suspect it won't be used for a bit.

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.

I specifically need support for the exact target triple for wasi with threads for rust-lang/miri#722 (comment) to compile successfully. target-lexicon refuses to compile for any target which it doesn't know how to parse even if said target will never be used at runtime as compilation target.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Once the wasip{1,2} targets land in rustc I'll post a follow-up to rename preview1-threads to p1-threads, but in the meantime it seems reasonable to add the target here too

@alexcrichton alexcrichton deleted the wasi-targets branch February 22, 2024 15:30
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.

3 participants