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

winch: Use cranelift as the trampoline compiler #8098

Closed
saulecabrera opened this issue Mar 12, 2024 · 3 comments · Fixed by #8109
Closed

winch: Use cranelift as the trampoline compiler #8098

saulecabrera opened this issue Mar 12, 2024 · 3 comments · Fixed by #8109
Assignees
Labels
winch Winch issues or pull requests

Comments

@saulecabrera
Copy link
Member

saulecabrera commented Mar 12, 2024

#8082 introduces Cranelift as the trampoline compiler for components in Winch. This allows reducing code duplication and allowing for better maintainability.

In order to have uniform trampoline compilation across the board, we want to migrate the rest of Winch's trampolines to use Cranelift.

@saulecabrera saulecabrera added the winch Winch issues or pull requests label Mar 12, 2024
Copy link

Subscribe to Label Action

cc @saulecabrera

This issue or pull request has been labeled: "winch"

Thus the following users have been cc'd because of the following labels:

  • saulecabrera: winch

To subscribe or unsubscribe from this label, edit the .github/subscribe-to-label.json configuration file.

Learn more.

@elliottt
Copy link
Member

elliottt commented Mar 14, 2024

Fixed in #8082

@elliottt elliottt reopened this Mar 14, 2024
@elliottt
Copy link
Member

I confused this with #8087 when closing old issues.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
winch Winch issues or pull requests
Projects
Status: Done
Development

Successfully merging a pull request may close this issue.

2 participants