wasmi-labs / wasmi

WebAssembly (Wasm) interpreter.
https://wasmi-labs.github.io/wasmi/
Apache License 2.0
1.56k stars 281 forks source link

New reg span types & replace `RegSpanIter` with `BoundedRegSpan` #1180

Closed Robbepop closed 2 weeks ago

Robbepop commented 2 weeks ago

This also simplifies integration of https://github.com/wasmi-labs/wasmi/pull/1152.

codecov[bot] commented 2 weeks ago

Codecov Report

Attention: Patch coverage is 75.38462% with 64 lines in your changes missing coverage. Please review.

Project coverage is 79.63%. Comparing base (b584599) to head (b2a5779). Report is 1 commits behind head on main.

Files with missing lines Patch % Lines
crates/wasmi/src/engine/bytecode/span.rs 59.43% 43 Missing :warning:
crates/wasmi/src/engine/bytecode/visit_regs.rs 22.22% 7 Missing :warning:
crates/wasmi/src/engine/bytecode/error.rs 0.00% 6 Missing :warning:
crates/wasmi/src/engine/executor/instrs/copy.rs 33.33% 2 Missing :warning:
crates/wasmi/src/engine/translator/mod.rs 81.81% 2 Missing :warning:
crates/wasmi/src/error.rs 0.00% 2 Missing :warning:
crates/wasmi/src/engine/bytecode/construct.rs 83.33% 1 Missing :warning:
crates/wasmi/src/engine/executor/instrs/branch.rs 0.00% 1 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #1180 +/- ## ========================================== - Coverage 79.77% 79.63% -0.15% ========================================== Files 296 298 +2 Lines 25243 25292 +49 ========================================== + Hits 20138 20141 +3 - Misses 5105 5151 +46 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.