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

refactor(wasm): consider replacing lol_alloc with dlmalloc-rs #4781

Open
Erigara opened this issue Jun 27, 2024 · 0 comments
Open

refactor(wasm): consider replacing lol_alloc with dlmalloc-rs #4781

Erigara opened this issue Jun 27, 2024 · 0 comments
Labels
question Further information is requested Refactor Improvement to overall code quality

Comments

@Erigara
Copy link
Contributor

Erigara commented Jun 27, 2024

We should consider replacing lol_alloc with dlmalloc-rs.

I would expect some benefits:

  • dlmalloc-rs is used by rust std itself as default allocator for wasm target, so we can expect better support and less bugs
  • might have better performance

Possible downsides:

  • might increase size of the executor and smart-contracts
@Erigara Erigara added question Further information is requested Refactor Improvement to overall code quality labels Jun 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
question Further information is requested Refactor Improvement to overall code quality
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@Erigara