-
Notifications
You must be signed in to change notification settings - Fork 8
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
Add missing dummy reads/writes #107
Comments
This was referenced Jan 26, 2020
I think this is resolved? @johnor? |
Almost. There are some instructions that are not using create_addressing_steps that might need to be double checked though. Like Pha, Pla, Rts, Rti. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
A lot of the 6502 instructions write even when they have no reason to. We've skipped this in places. We should set up an integration test checking dummy reads/writes and add them as they can cause side-effects.
The text was updated successfully, but these errors were encountered: