Skip to content

Remove unused WAVM opcodes#686

Merged
rachel-bousfield merged 2 commits intowasm-testsuitefrom
rm-unused-opcodes
Jun 17, 2022
Merged

Remove unused WAVM opcodes#686
rachel-bousfield merged 2 commits intowasm-testsuitefrom
rm-unused-opcodes

Conversation

@PlasmaPower
Copy link
Contributor

Targets #586

Removes the block opcodes, the branch opcodes, the stack boundary opcodes, and the stack boundary type. Also changes the ArbitrumJump and ArbitraryJumpIf opcode numbers to be adjacent.

@cla-bot cla-bot bot added the cla-signed label Jun 10, 2022
@PlasmaPower PlasmaPower added the arbitrator Relates to the Arbitrator proving component label Jun 10, 2022
@codecov
Copy link

codecov bot commented Jun 10, 2022

Codecov Report

Merging #686 (c5fb345) into wasm-testsuite (d934588) will increase coverage by 0.12%.
The diff coverage is 36.36%.

@@                Coverage Diff                 @@
##           wasm-testsuite     #686      +/-   ##
==================================================
+ Coverage           53.03%   53.15%   +0.12%     
==================================================
  Files                 223      223              
  Lines               25921    25849      -72     
  Branches              486      475      -11     
==================================================
- Hits                13747    13741       -6     
+ Misses              10764    10696      -68     
- Partials             1410     1412       +2     

Copy link
Contributor

@rachel-bousfield rachel-bousfield left a comment

Choose a reason for hiding this comment

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

Looks great!

@rachel-bousfield rachel-bousfield merged commit 13b907d into wasm-testsuite Jun 17, 2022
@rachel-bousfield rachel-bousfield deleted the rm-unused-opcodes branch June 17, 2022 01:12
tsahee pushed a commit that referenced this pull request Jul 29, 2025
Co-authored-by: Pepper Lebeck-Jobe <pepper@offchainlabs.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

arbitrator Relates to the Arbitrator proving component cla-signed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants