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

ARM64: Invalid decoding of constant for predicated instructions fadd, fsub, fsubr, fmul, fmax, fmaxnm, fmin, fminnm #2268

Closed
TIHan opened this issue Feb 9, 2024 · 0 comments · Fixed by #2298
Labels
AArch64 Arch outdated-module Issue due to an outdated arch module
Milestone

Comments

@TIHan
Copy link

TIHan commented Feb 9, 2024

ARM64 Example hex: 00805865

When using cstool, the output is:
fadd z0.h, p0/m, z0.h, #0.0

when it should be:
fadd z0.h, p0/m, z0.h, #0.5

There are more instructions (listed below) that have the incorrect interpretation of the i1 field. The decoder doesn't produce valid constants for those instructions.

FADD ., /M, .,
FMAX ., /M, .,
FMAXNM ., /M, .,
FMIN ., /M, .,
FMINNM ., /M, .,
FMUL ., /M, .,
FSUB ., /M, .,
FSUBR ., /M, .,

@Rot127 Rot127 added AArch64 Arch outdated-module Issue due to an outdated arch module labels Mar 19, 2024
@Rot127 Rot127 added this to the v6 milestone Mar 19, 2024
@Rot127 Rot127 moved this to Todo in Capstone V6 Plan Mar 20, 2024
@Rot127 Rot127 mentioned this issue Mar 25, 2024
6 tasks
@github-project-automation github-project-automation bot moved this from Todo to Done in Capstone V6 Plan Jul 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
AArch64 Arch outdated-module Issue due to an outdated arch module
Projects
Status: Done
Development

Successfully merging a pull request may close this issue.

2 participants