Skip to content

Propose goal: production-ready cranelift backend #334

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

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

folkertdev
Copy link
Contributor

@folkertdev folkertdev commented Jul 15, 2025

This project aims to get the rust cranelift codegen backend to a production-ready state. Specifically, with this work completed, we'd be confident to recommend it as the default for local development, e.g. with cargo test or cargo run.

Rendered

Copy link
Member

@Kobzol Kobzol left a comment

Choose a reason for hiding this comment

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

Very excited about this! :)

| :--------------- | -------------------- |
| Point of contact | @folkertdev |
| Teams | T-compiler, T-perf |
| Task owners | @bjorn3, @folkertdev |
Copy link
Member

Choose a reason for hiding this comment

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

These are automatically generated from the table at the bottom

Suggested change
| Task owners | @bjorn3, @folkertdev |
| Teams | <!-- TEAMS WITH ASKS --> |
| Task owners | <!-- TASK OWNERS --> |

| Standard reviews | ![Team][] [compiler] | Casual improvements to `rustc_codegen_ssa` and other parts of the backend |
| Dedicated reviewer | ![Team][] [compiler] | Larger changes to `rustc_codegen_ssa`. While not strictly required, we think having a dedicated reviewer will speed up our progress. |
| Deploy to production | ![Team][] [wg-compiler-performance] | track and show `rustc_codegen_cranelift` performance |
| Do the work | bjorn3, folkertdev | |
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
| Do the work | bjorn3, folkertdev | |
| Do the work | @bjorn3, @folkertdev | |


| Task | Owner(s) or team(s) | Notes |
| -------------------- | ------------------------------------------------ | ------------------------------------------------------------------------------------------------------------------------------------ |
| Acquire funding | Trifecta Tech Foundation @trifectatechfoundation | We will only be able to work on this project if it is funded |
Copy link
Member

Choose a reason for hiding this comment

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

This will help rendering of the Foundation name in the header

Suggested change
| Acquire funding | Trifecta Tech Foundation @trifectatechfoundation | We will only be able to work on this project if it is funded |
| Acquire funding | [Trifecta Tech Foundation] | We will only be able to work on this project if it is funded |

combined with this elsewhere, after the table or wherever

[Trifecta Tech Foundation]: https://github.com/trifectatechfoundation

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