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

core/vm: merge opJumpdest to opJump/opJumpi #475

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Francesco4203
Copy link
Contributor

@Francesco4203 Francesco4203 commented Jun 18, 2024

Make opJumpdest to be executed right after opJump/opJumpi:

  • Reduce time of unnecessary checking for opJumpdest properties
  • Reduce overhead of fail branch predicting from opJump/opJumpi to opJumpdest (since it is always opJumpdest after)

@Francesco4203 Francesco4203 force-pushed the optimize-jump-opcode branch 3 times, most recently from c34a764 to b343202 Compare June 21, 2024 07:09
core/vm/evm_test.go Outdated Show resolved Hide resolved
@Francesco4203 Francesco4203 force-pushed the optimize-jump-opcode branch 2 times, most recently from 66ec8c7 to e57dee4 Compare July 8, 2024 11:05
@Francesco4203 Francesco4203 force-pushed the optimize-jump-opcode branch from e57dee4 to ce98793 Compare July 8, 2024 11:17
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