Skip to content
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

cranelift-control: support no_std compilation #8995

Merged

Conversation

marceline-cramer
Copy link
Contributor

Follows discussion in #1158. Ratchets cranelift-control's no_std support in CI.

Also ratchets cranelift-control's no_std support in CI.
@marceline-cramer marceline-cramer requested review from a team as code owners July 23, 2024 02:18
@marceline-cramer marceline-cramer requested review from fitzgen and removed request for a team July 23, 2024 02:18
@github-actions github-actions bot added the cranelift Issues related to the Cranelift code generator label Jul 23, 2024
Copy link
Member

@alexcrichton alexcrichton left a comment

Choose a reason for hiding this comment

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

Thanks!

Comment on lines 434 to 438
- run: |
cargo check -p wasmtime --no-default-features --features runtime,gc,component-model
cargo check -p cranelift-control --no-default-features
env:
CARGO_BUILD_TARGET: x86_64-unknown-none
Copy link
Member

Choose a reason for hiding this comment

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

Mind splitting these into separate run jobs so it's more obvious which one failed when it fails?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Sure thing, done!

@alexcrichton alexcrichton added this pull request to the merge queue Jul 24, 2024
Merged via the queue into bytecodealliance:main with commit 55af5ac Jul 24, 2024
37 checks passed
@marceline-cramer marceline-cramer deleted the cranelift-control-no-std branch July 24, 2024 21:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cranelift Issues related to the Cranelift code generator
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants