-
Notifications
You must be signed in to change notification settings - Fork 66
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
performance issue for std::span on wasm C calling convention #217
Comments
What happens if you enable |
Nothing changes. It is an ABI issue. |
The full set of magic flags @dschuff is thinking of is |
Ah thanks for that, I had forgotten that the feature enabler and the ABI were separate flags. |
std::span is passed by memory, not by registers on wasm. which is extremely slow.
This is going to encourage people to overuse pointers. Please change the calling convention here. Also, the return value has the same issue.
The text was updated successfully, but these errors were encountered: