Disassembling 6502 code is giving me "RTI A" instead of "RTI" for code $40, which appears invalid?

Arakula / dasmfw

The DisASseMbler FrameWork

GNU General Public License v2.0

13 stars 4 forks source link

Disassembling 6502 code is giving me "RTI A" instead of "RTI" for code $40, which appears invalid? #5

Closed phillipeaton closed 2 years ago

phillipeaton commented 2 years ago

I'm a total newbie with 6502 code, but this appears wrong as far as I can tell?

Thanks!

Arakula commented 2 years ago

You seem to be the first one ever using the 6502 disassembler :-) That one's not exactly fully mature yet. I'll look into it.

phillipeaton commented 2 years ago

I knew that when I started, you commented somewhere as such. The program gave me such good results for 6809, it's worth being a 6502 tester! (My 6809 project is still active, will publish results eventually.)

Arakula commented 2 years ago

Hmmm, looks like a table error ...

  _rti  ,_acc,   ...     /* 40..43 */

where it should rather be

  _rti  ,_imp,   ...     /* 40..43 */

in some places (one for each supported 65xx family member instruction set). If you're feeling lucky, you might try that simple fix. Should occur in 3 lines in Dasm6500.cpp (lines 115, 1064, 1288 in my local copy).

phillipeaton commented 2 years ago

Thanks for the fix.

Last time you made available a patched binary, but this time I have VC2019 installed and have now been able successfully recompile dasmfw and the fix appears to work!

I assume you'll put it into the next release.

Arakula commented 2 years ago

Yes, of course. I'll presumably upload a "Phillip Eaton release" later today.