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

[Debugger] Load/Store instructions don't indicate writeback flag #1915

Closed
lunasorcery opened this issue Oct 24, 2020 · 2 comments
Closed

[Debugger] Load/Store instructions don't indicate writeback flag #1915

lunasorcery opened this issue Oct 24, 2020 · 2 comments

Comments

@lunasorcery
Copy link

@lunasorcery lunasorcery commented Oct 24, 2020

Observed at 4aed125.

The disassembler appears not to note the writeback flag when disassembling ldr/str instructions.

For example:

ldrh r8, [r7, r2]!  ; opcode e1b780b2
ldrh r8, [r7, r2]   ; opcode e19780b2

mGBA will display both instructions as ldrh r8, [r7, r2], missing the trailing !, but will correctly perform the writeback when executing.

This appears to affect all forms - all of ldr, ldrh, ldrb, str, strh, strb, with both register offsets and immediate offsets.

@endrift
Copy link
Member

@endrift endrift commented Oct 24, 2020

I'm pretty sure this used to work. It may be a recent regression. Checking now.

@endrift endrift closed this in a2cd8f6 Oct 24, 2020
endrift added a commit that referenced this issue Oct 24, 2020
@endrift
Copy link
Member

@endrift endrift commented Oct 24, 2020

This was indeed a regression, but not a recent one. It broke in 0.2.0.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
2 participants
You can’t perform that action at this time.