Skip to content

feat(new-execution): add execute functions for rv32im instructions #1590

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Merged
merged 6 commits into from
Apr 18, 2025

Conversation

shuklaayush
Copy link
Contributor

@shuklaayush shuklaayush commented Apr 17, 2025

  • introduce a new generic InsExecutorE1 trait
  • add InsExecutor::execute_e1 for rv32im instructions

@shuklaayush shuklaayush changed the title feat(new-execution): add execution functions for rv32im instructions feat(new-execution): add execute functions for rv32im instructions Apr 17, 2025
@shuklaayush shuklaayush force-pushed the feat/rv32im-executors branch from e580c49 to 0e0c3fe Compare April 17, 2025 16:33
Copy link
Contributor

@jonathanpwang jonathanpwang left a comment

Choose a reason for hiding this comment

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

we can merge after moving execute_instruction2 to separate trait

@jonathanpwang jonathanpwang merged commit 1c25221 into feat/new-execution Apr 18, 2025
8 of 27 checks passed
@jonathanpwang jonathanpwang deleted the feat/rv32im-executors branch April 18, 2025 20:39
jonathanpwang pushed a commit that referenced this pull request May 2, 2025
…1590)

- introduce a new generic `InsExecutorE1` trait 
- add `InsExecutor::execute_e1` for rv32im instructions
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants