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

Consider b in pop_under_mask #40

Open
kammce opened this issue Jul 24, 2024 · 0 comments
Open

Consider b in pop_under_mask #40

kammce opened this issue Jul 24, 2024 · 0 comments
Labels
💳 technical debt A lazy, quick, or bad decision that we must fix later

Comments

@kammce
Copy link
Member

kammce commented Jul 24, 2024

‘Pop’ generally denotes removal from the stack commencing at current
vsp, with subsequent increment of vsp to beyond the removed quantities.
The sole exception to this rule is popping r13, when the writeback of
the loaded value to vsp is delayed until after the whole instruction
has completed. When multiple registers are popped by a single
instruction they are taken as lowest numbered register at lowest stack
address.

This is remark b. The code currently performs no extra work is performed if r13 is in the mask. It should follow the instructions above.

@kammce kammce added the 💳 technical debt A lazy, quick, or bad decision that we must fix later label Jul 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
💳 technical debt A lazy, quick, or bad decision that we must fix later
Projects
Status: No status
Development

No branches or pull requests

1 participant