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.

Sign up for GitHub

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

Jump to bottom

Hardware CPU: Save/Restore R8 to R12 in shadow registers during INT #172

Closed
sy2002 opened this issue Oct 19, 2020 · 4 comments
Closed

Hardware CPU: Save/Restore R8 to R12 in shadow registers during INT #172

sy2002 opened this issue Oct 19, 2020 · 4 comments
Assignees
@sy2002
Labels
V1.7

Comments

@sy2002
Copy link
Owner

sy2002 commented Oct 19, 2020 •
edited
Loading

Enhance CPU to save/restore R8 to R12 in shadow registers during interrupts (hardware and software)

Context:
#77 (comment)
@sy2002 sy2002 added the V1.7 label Oct 19, 2020
@sy2002 sy2002 self-assigned this Oct 19, 2020
@sy2002 sy2002 mentioned this issue Oct 19, 2020
Open
@MJoergen
Copy link
Collaborator

I'm updating the maze2d game to include a timer interrupt based clock. It works in the emulator, and it would be great to have it working in hardware too. Just to let you know I hope you implement this soon :-D

@sy2002
Copy link
Owner Author

sy2002 commented Oct 19, 2020

I plan to do it today evening :-D

@MJoergen MJoergen mentioned this issue Oct 25, 2020
Open
MJoergen added a commit that referenced this issue Oct 27, 2020
@MJoergen
Hardware CPU: Save/Restore R8 to R12 in shadow registers during INT
140120d 
See Issue #172.
@MJoergen
Copy link
Collaborator

I've implemented this now. I've verified no regression with the cpu_test.asm program, and I've extended the int_test.asm program to test the new feature.

The implementation is somewhat verbose, and can perhaps be refactored.

@sy2002
Copy link
Owner Author

sy2002 commented Oct 28, 2020

Thank you - due to my end of year crazyness it is unlikely that I contribute anything meaningful "before Christmas" :-)
And indeed: This does need refactoring - but this is not urgent, so I close this issue.

@sy2002 sy2002 closed this as completed Oct 28, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@sy2002 sy2002

Labels
V1.7
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@MJoergen @sy2002