Skip to content

fix: use EvmError from zk_ee instead of zksync_os_evm_errors in evm_interpreter#525

Merged
antoniolocascio merged 1 commit intodevfrom
alocascio-evm-errors
Feb 23, 2026
Merged

fix: use EvmError from zk_ee instead of zksync_os_evm_errors in evm_interpreter#525
antoniolocascio merged 1 commit intodevfrom
alocascio-evm-errors

Conversation

@antoniolocascio
Copy link
Contributor

What ❔

Avoids making changes to the interface errors alter the proving binary.

Why ❔

Is this a breaking change?

  • Yes
  • No

Checklist

  • PR title corresponds to the body of PR (we generate changelog entries from PRs).
  • Tests for the changes have been added / updated.
  • Documentation comments have been added / updated.
  • Code has been formatted.

@antoniolocascio antoniolocascio changed the title fix: ese EvmError from zk_ee instead of zksync_os_evm_errors in evm_interpreter fix: use EvmError from zk_ee instead of zksync_os_evm_errors in evm_interpreter Feb 20, 2026
@github-actions
Copy link
Contributor

Benchmark report

Benchmark Symbol Base Eff Head Eff (%) Base Raw Head Raw (%) Base Blake Head Blake (%) Base Bigint Head Bigint (%)
block_19299001 process_block 315,709,790 315,709,790 (+0.00%) 273,044,390 273,044,390 (+0.00%) 410,630 410,630 (+0.00%) 9,023,830 9,023,830 (+0.00%)
block_22244135 process_block 197,594,936 197,594,936 (+0.00%) 170,679,144 170,679,144 (+0.00%) 172,040 172,040 (+0.00%) 6,040,788 6,040,788 (+0.00%)
precompiles bn254_ecadd 53,268 53,268 (+0.00%) 47,816 47,816 (+0.00%) 0 0 (+0.00%) 1,363 1,363 (+0.00%)
precompiles bn254_ecmul 728,781 728,781 (+0.00%) 564,593 564,593 (+0.00%) 0 0 (+0.00%) 41,047 41,047 (+0.00%)
precompiles bn254_pairing 72,336,733 72,336,733 (+0.00%) 57,808,589 57,808,589 (+0.00%) 0 0 (+0.00%) 3,632,036 3,632,036 (+0.00%)
precompiles ecrecover 475,819 479,345 (+0.74%) 308,603 310,701 (+0.68%) 0 0 (+0.00%) 41,804 42,161 (+0.85%)
precompiles id 927 927 (+0.00%) 927 927 (+0.00%) 0 0 (+0.00%) 0 0 (+0.00%)
precompiles keccak 137,579 137,579 (+0.00%) 137,579 137,579 (+0.00%) 0 0 (+0.00%) 0 0 (+0.00%)
precompiles modexp 31,267,898 31,267,898 (+0.00%) 20,610,078 20,610,078 (+0.00%) 0 0 (+0.00%) 2,664,455 2,664,455 (+0.00%)
precompiles p256_verify 748,861 748,861 (+0.00%) 470,169 470,169 (+0.00%) 0 0 (+0.00%) 69,673 69,673 (+0.00%)
precompiles point_evaluation 52,918,413 52,918,413 (+0.00%) 40,570,377 40,570,377 (+0.00%) 0 0 (+0.00%) 3,087,009 3,087,009 (+0.00%)
precompiles process_block 147,539,915 147,546,500 (+0.00%) 118,075,291 118,079,412 (+0.00%) 5,140 5,140 (+0.00%) 7,345,596 7,346,212 (+0.01%)
precompiles process_transaction 73,480,232 73,484,657 (+0.01%) 58,786,776 58,789,317 (+0.00%) 160 160 (+0.00%) 3,672,724 3,673,195 (+0.01%)
precompiles ripemd 8,013 8,013 (+0.00%) 8,013 8,013 (+0.00%) 0 0 (+0.00%) 0 0 (+0.00%)
precompiles run_tx_loop 146,889,285 146,895,849 (+0.00%) 117,504,021 117,508,121 (+0.00%) 180 180 (+0.00%) 7,345,596 7,346,212 (+0.01%)
precompiles sha256 13,168 13,168 (+0.00%) 13,168 13,168 (+0.00%) 0 0 (+0.00%) 0 0 (+0.00%)
precompiles system_init 46,565 46,565 (+0.00%) 46,565 46,565 (+0.00%) 0 0 (+0.00%) 0 0 (+0.00%)
precompiles verify_and_apply_batch 146,508 146,508 (+0.00%) 110,188 110,188 (+0.00%) 2,270 2,270 (+0.00%) 0 0 (+0.00%)

Copy link
Contributor

@0xVolosnikov 0xVolosnikov left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Probably will require some small changes in the server as well

@antoniolocascio antoniolocascio merged commit b722dbb into dev Feb 23, 2026
17 checks passed
@antoniolocascio antoniolocascio deleted the alocascio-evm-errors branch February 23, 2026 07:11
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.

2 participants