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

lib6502 fixes #93

Merged
merged 9 commits into from
Dec 21, 2023
Merged

lib6502 fixes #93

merged 9 commits into from
Dec 21, 2023

Conversation

ivop
Copy link
Contributor

@ivop ivop commented Dec 21, 2023

Fixes for lib6502.

Mostly ported from PiTubeDirect project and adjusted for your version.

Tested with Klaus Dormann test suite. Passes both 65C02_extended_opcodes_test.bin and 6502_decimal_test.bin.

I could add the test harness to the repo if you want?

Regards,
Ivo

@davidgiven
Copy link
Owner

Good grief. I'd no idea it was that bad. I wonder whether a different emulator core would be preferable; I find lib6502 almost impossible to read.

@davidgiven davidgiven merged commit a858dfb into davidgiven:master Dec 21, 2023
@ivop
Copy link
Contributor Author

ivop commented Dec 21, 2023

For a recent project I did (https://github.com/ivop/random6502) I used the emulator by Ben Zotto (https://github.com/bzotto/MCS6502). Another interesting emulation approach is https://github.com/floooh/chips/blob/master/chips/m6502.h but most of that code is machine generated and the accuracy is probably overkill for our usage.

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.

2 participants