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

Add wasm-opt-for-rust-maintenance-5 #847

Merged
merged 1 commit into from
May 11, 2023

Conversation

brson
Copy link
Contributor

@brson brson commented May 1, 2023

Milestone Delivery Checklist

Link to the application pull request: w3f/Grants-Program#1305

Only 2 hours of work this month.

Our patches to upstream Binaryen to enable unicode on windows are not going to be accepted as-is - the Binaryen authors have begun writing their own patches for this purpose.

We have received a request on the issue tracker to add support for in-memory operation. It's a feature that makes sense, but which is not supported by Binaryen's underlying file-oriented APIs, so implementing it requires more upstream work. I'm inclined to hack on it, and there's lots of budget since there's nothing else on the table for this project at the moment. If somebody disagrees, let me know.

No new Binaryen releases this month, but it's about time for a new one soon.

Internet Computer integrated the wasm-opt crate into one of their projects, and Holochain seems to have used it for some purpose. That's in addition to the 2 Polkadot projects and Stellar already using it. GitHub thinks 555 repos have wasm-opt-rs dependencies (mostly substrate forks). So this modest project has had a pretty nice impact.

@semuelle semuelle self-assigned this May 9, 2023
Copy link
Member

@semuelle semuelle left a comment

Choose a reason for hiding this comment

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

Thanks for the delivery and the updates, @brson. It's great to hear the project being used in different places. And feel free to work on the in-memory support.

I have forwarded your invoice for processing.

@semuelle semuelle merged commit 603ab63 into w3f:master May 11, 2023
@RouvenP
Copy link

RouvenP commented May 17, 2023

hi @brson we just transferred the payment

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