dlbeer / mspdebug

Debugging tool for MSP430 MCUs
GNU General Public License v2.0
184 stars 80 forks source link

Fix CPUX pushm/popm for address words #122

Open harriswms opened 1 year ago

harriswms commented 1 year ago

This patch fixes the "step_pushm_popm" function for the CPUX simulator, which originally 1) decoded PUSHM.A and POPM.A instructions incorrectly and 2) treated all PUSHM/POPM instructions as 16-bit word accesses, which caused programs using 20-bit address words to crash. After patch, simulation of my test binary matched operation on a physical MSP430FR5994.