Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Investigate trunk support for mipsy web #196

Closed
2 tasks done
spanishpear opened this issue May 14, 2022 · 2 comments · Fixed by #197
Closed
2 tasks done

Investigate trunk support for mipsy web #196

spanishpear opened this issue May 14, 2022 · 2 comments · Fixed by #197
Assignees
Labels
A-mipsy_web Area: mipsy web

Comments

@spanishpear
Copy link
Collaborator

spanishpear commented May 14, 2022

  • make mipsy_web use trunk
  • investigate performance tradeoffs

context: trunk-rs/trunk#46 has now been merged, so we can use trunk isntead of wasm-pack for htings :)

@spanishpear spanishpear added the A-mipsy_web Area: mipsy web label May 14, 2022
@spanishpear spanishpear self-assigned this May 14, 2022
@spanishpear
Copy link
Collaborator Author

spanishpear commented May 14, 2022

wasm_pack (w/ release)

.rw-rw-r--   34k shrey 14 May 17:22  wasm.js
.rw-rw-r--  1.8M shrey 14 May 17:31  wasm_bg.wasm

@spanishpear
Copy link
Collaborator Author

spanishpear commented May 14, 2022

trunk (w/ release)

.rw-rw-r--   33k shrey 14 May 21:10  app-eddec691fbe340ac.js
.rw-rw-r--  748k shrey 14 May 21:11  app-eddec691fbe340ac_bg.wasm
.rw-rw-r--   11k shrey 14 May 21:10  worker.js
.rw-rw-r--  978k shrey 14 May 21:11  worker_bg.wasm

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
A-mipsy_web Area: mipsy web
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant