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

dev: use jumpdest logic from src in interpreter #660

Open
Eikix opened this issue Feb 5, 2025 · 1 comment · May be fixed by #762
Open

dev: use jumpdest logic from src in interpreter #660

Eikix opened this issue Feb 5, 2025 · 1 comment · May be fixed by #762
Assignees

Comments

@Eikix
Copy link
Member

Eikix commented Feb 5, 2025

why

use a more optimized jump dest computation

how

in interpreter.cairo, replace from ethereum.cancun.vm.runtime import get_valid_jump_destinations with implementation in src

@github-project-automation github-project-automation bot moved this to Backlog in Keth Feb 5, 2025
@obatirou obatirou moved this from Backlog to In progress in Keth Feb 5, 2025
@obatirou obatirou self-assigned this Feb 5, 2025
@obatirou obatirou moved this from In progress to Backlog in Keth Feb 5, 2025
@obatirou obatirou moved this from Backlog to In progress in Keth Feb 5, 2025
@obatirou obatirou moved this from In progress to Backlog in Keth Feb 5, 2025
@obatirou obatirou removed their assignment Feb 6, 2025
@Eikix Eikix moved this from Backlog to Todo in Keth Feb 10, 2025
@Eikix Eikix added this to the EELS migration milestone Feb 10, 2025
@TobieTom
Copy link

Hola. I'm Tobias and I'm new to the space. I've already contributed to some projects and would like to add this to my pool of experience plus I recently learned rust and cairo

@obatirou obatirou moved this from Todo to In progress in Keth Feb 11, 2025
@obatirou obatirou self-assigned this Feb 11, 2025
@obatirou obatirou linked a pull request Feb 11, 2025 that will close this issue
@obatirou obatirou moved this from In progress to In review in Keth Feb 11, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: In review
Development

Successfully merging a pull request may close this issue.

3 participants