Skip to content

fix(msp430): correct memset and memcpy assembly operand order for TI …#981

Open
by965738071 wants to merge 1 commit into
ZigEmbeddedGroup:mainfrom
by965738071:fix/msp430-memset-memcpy
Open

fix(msp430): correct memset and memcpy assembly operand order for TI …#981
by965738071 wants to merge 1 commit into
ZigEmbeddedGroup:mainfrom
by965738071:fix/msp430-memset-memcpy

Conversation

@by965738071

@by965738071 by965738071 commented Jul 15, 2026

Copy link
Copy Markdown
Contributor

No description provided.

@mattnite

Copy link
Copy Markdown
Contributor

How did you validate this fix?

@by965738071

Copy link
Copy Markdown
Contributor Author

The operand order in the old code was reversed: ADD R5, R14 meant R14 += R5 (causing count corruption), while SUB R12, R14 left R12 unrecoverable. The fix changes both to conform to TI's OP src, dst → dst = dst OP src semantics.

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