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

Modeler extension #52

Open
wants to merge 28 commits into
base: main
Choose a base branch
from

Conversation

CAS-ual-TY
Copy link
Contributor

This is the second PR that Fabian and me worked on.

I must note: Most of this was uncommited for some time now and when orderly committing everything we realised that there were conflicts now as some new features we added to the modeler were added a couple days ago. The merge was hopefully done properly but review it twice just to be sure.

Emulator additions:

  • lwu
  • slti
  • xor
  • srlw
  • sraw
  • divuw
  • remuw

Modeler attitions:

  • Reordered these instructions translations to follow the order of instruction.rs and emulator.rs
  • Improved addiw to use AND instead of SLL+SRL
  • Improved slliw to use AND instead of SLL+SRL
  • Improved srliw to use AND instead of SLL+SRL
  • Improved sraiw to use AND instead of SLL+SRL
  • Improved addw to use AND instead of SLL+SRL
  • Improved subw to use AND instead of SLL+SRL
  • Add lwu
  • Mark slti as unimplemented
  • Mark slt as unimplemented
  • Mark xor as unimplemented
  • Mark rem as unimplemented
  • Mark remw as unimplemented
  • Mark remuw as unimplemented

I hope there are not too many issues in here. Just let us know and we will fix it. Feel free to squash on merge.

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.

None yet

1 participant