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

Reuse allocations for br_table translations #440

Merged
merged 1 commit into from Sep 14, 2022

Conversation

Robbepop
Copy link
Collaborator

@Robbepop Robbepop commented Sep 14, 2022

This causes less strain on the heap allocator for Wasm binaries with many br_table instructions.

@codecov-commenter
Copy link

Codecov Report

Merging #440 (0ab2a1a) into master (11b35ae) will increase coverage by 0.04%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master     #440      +/-   ##
==========================================
+ Coverage   79.33%   79.38%   +0.04%     
==========================================
  Files          72       72              
  Lines        6198     6198              
==========================================
+ Hits         4917     4920       +3     
+ Misses       1281     1278       -3     
Impacted Files Coverage Δ
wasmi_v1/src/engine/func_builder/mod.rs 92.44% <100.00%> (+0.36%) ⬆️

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@Robbepop Robbepop merged commit 8fd6464 into master Sep 14, 2022
@Robbepop Robbepop deleted the rf-cache-allocs-for-br-table-translations branch September 14, 2022 09:09
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.

None yet

2 participants