Skip to content

Fix arm pop reg access #2718

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

Open
wants to merge 3 commits into
base: next
Choose a base branch
from

Conversation

jiegec
Copy link
Contributor

@jiegec jiegec commented May 24, 2025

Your checklist for this pull request

  • I've documented or updated the documentation of every API function and struct this PR changes.
  • I've added tests that prove my fix is effective or that my feature works (if possible)

Detailed description

Fix the reg access type of arm thumb pop instruction.

Test plan

./suite/cstest/cstest ../tests/details/arm.yaml

Closing issues

closes #2713

jiegec added 2 commits May 24, 2025 15:23
In some environment, the iteration order does not follow the
alphabetical order, so that the patches cannot be applied cleanly. Add
extra sorting to ensure the order.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Wrong register access mode reported in thumb pop instruction
2 participants