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

PSTATE renaming #87

Open
BugraEryilmaz opened this issue Nov 26, 2024 · 1 comment
Open

PSTATE renaming #87

BugraEryilmaz opened this issue Nov 26, 2024 · 1 comment
Assignees

Comments

@BugraEryilmaz
Copy link
Contributor

Description
PSTATE is a critical register that keeps NZCV flags, which can change the program's execution. The flags must be renamed because they are often updated, and each instruction should get the correct flags. This functionality exists for most instructions, but ADC and SUBC instructions do not rename PSTATE.

Steps to Reproduce
Run data caching image with 2 cores
After some time there will be an ADC or SUBC instruction that triggers the validation error
Expected Behavior
ADC and SUBC should use the renamed carry bits and there should be no validation error.

Actual Behavior
They do not use the renamed carry bits and give a validation error.

@BugraEryilmaz BugraEryilmaz self-assigned this Nov 26, 2024
@BugraEryilmaz
Copy link
Contributor Author

Fixed: 335ec8a

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

No branches or pull requests

1 participant