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

implement Aarch64 UMOV instruction semantics #1331

Conversation

psi-func
Copy link
Contributor

add more VAS types to correctly handle different cases which part of vector register should move to GPR register
add umov semantics in aarch64
instruction reference here

@psi-func
Copy link
Contributor Author

Capstone v4.0.1 missing important feature with parsing extended VAS types
build fails in CI

@gulldan
Copy link

gulldan commented May 28, 2024

also need it ASAP

@JonathanSalwan
Copy link
Owner

Lol, I think this is the first ever MR that involves so many reactions ^^.

@JonathanSalwan JonathanSalwan changed the base branch from master to dev-v1.0 May 28, 2024 16:57
@JonathanSalwan JonathanSalwan merged commit f8a4192 into JonathanSalwan:dev-v1.0 May 28, 2024
10 of 28 checks passed
@JonathanSalwan JonathanSalwan added this to the v1.0 milestone May 28, 2024
JonathanSalwan pushed a commit that referenced this pull request May 28, 2024
@psi-func
Copy link
Contributor Author

psi-func commented May 29, 2024

Found problem in this code via bug in capstone
Tryed to add tests in python
"\x43\x3c\x1c\x0e", "umov w3, v2.S[3]"
Parsed by capstone as MOV instruction, because of this buildSemantics is performed incorrectly
Left issue here
They advise using the next branch

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.

None yet

3 participants