Skip to content

Add br_i{32,64}[_not]_eq_{s,r}z ops - #2026

Open
Robbepop wants to merge 11 commits into
mainfrom
rf-add-br-eqz-nez-ops
Open

Add br_i{32,64}[_not]_eq_{s,r}z ops#2026
Robbepop wants to merge 11 commits into
mainfrom
rf-add-br-eqz-nez-ops

Conversation

@Robbepop

@Robbepop Robbepop commented Aug 9, 2026

Copy link
Copy Markdown
Member

Assembly: branch_i32_eq_rz

wasmi::engine::executor::handler::exec::branch_i32_eq_rz:
Lfunc_begin1308:
        .cfi_startproc
        ldrsw x8, [x1, #8]
        add x9, x1, #16
        add x8, x1, x8
        cmp w6, #0
        csel x1, x8, x9, eq
        ldr x7, [x1]
        br x7

Assembly: branch_i32_eq_ri

wasmi::engine::executor::handler::exec::branch_i32_eq_ri:
Lfunc_begin1306:
        .cfi_startproc
        ldp w8, w9, [x1, #8]
        sxtw x8, w8
        add x10, x1, #16
        add x8, x1, x8
        cmp w9, w6
        csel x1, x8, x10, eq
        ldr x7, [x1]
        br x7

@codecov

codecov Bot commented Aug 9, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 82.35294% with 6 lines in your changes missing coverage. Please review.
✅ Project coverage is 76.28%. Comparing base (c242c15) to head (a5db502).

Files with missing lines Patch % Lines
crates/wasmi/src/engine/translator/comparator.rs 76.92% 6 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2026      +/-   ##
==========================================
- Coverage   76.29%   76.28%   -0.01%     
==========================================
  Files         168      168              
  Lines       15531    15559      +28     
==========================================
+ Hits        11849    11869      +20     
- Misses       3682     3690       +8     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

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.

1 participant