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.

Sign up for GitHub

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

Jump to bottom

Compile to webassembly for speed #32

Open
axic opened this issue Jul 29, 2018 · 5 comments · May be fixed by #29
Open

Compile to webassembly for speed #32

axic opened this issue Jul 29, 2018 · 5 comments · May be fixed by #29
Labels
help wanted

Comments

@axic
Copy link
Member

axic commented Jul 29, 2018

No description provided.

@axic axic linked a pull request Jul 29, 2018 that will close this issue
Open
@HenryNguyen5
Copy link

HenryNguyen5 commented Mar 30, 2020 •
edited
Loading

@axic would asm.js still need to be a fallback? Or can it be completely dropped in favour of wasm?

@axic
Copy link
Member Author

axic commented Mar 30, 2020

I think it is still needed as a fallback.

@axic
Copy link
Member Author

axic commented Mar 30, 2020

Also please note #29 has an attempt at this, but it is not finished. Feel free to pick it up.

@HenryNguyen5
Copy link

How would you feel about using wasm-bindgen? It would eliminate the need to maintain any boilerplate code. The generated wasm artifact can then be converted to an asm.js like representation via wasm2js, seen here.

@axic
Copy link
Member Author

axic commented Mar 30, 2020

It should be fine, when this work started wasm-bindgen wasn't around or wasn't mature enough.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
help wanted
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

[WIP] Support wasm target ethereumjs/rustbn.js
2 participants
@axic @HenryNguyen5