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

Fix aarch64 cpsr #855

Merged
merged 3 commits into from
Jun 28, 2022
Merged

Fix aarch64 cpsr #855

merged 3 commits into from
Jun 28, 2022

Conversation

hugsy
Copy link
Owner

@hugsy hugsy commented Jun 28, 2022

Description/Motivation/Screenshots

Everything we do on the architecture AARCH64 is partially broken (or at least not working as it should) due to the way the execution mode is determined from the CPSR.

This PR fixes that by applying what the ARMv8 says, differentiating AARCH64, AARCH32 and THUMB32 execution modes.

How Has This Been Tested?

Architecture Yes/No Comments
ARM
AARCH64
Others N/A
make test

Checklist

  • My PR was done against the dev branch, not main.
  • My code follows the code style of this project.
  • My change includes a change to the documentation, if required.
  • If my change adds new code, adequate tests have been added.
  • I have read and agree to the CONTRIBUTING document.

@hugsy hugsy added the bug label Jun 28, 2022
@hugsy hugsy added this to the Release: next milestone Jun 28, 2022
@hugsy hugsy merged commit 290a984 into dev Jun 28, 2022
@hugsy hugsy deleted the fix_aarch64_cpsr branch June 28, 2022 15:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant