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.

Sign up for GitHub

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

Jump to bottom

ARMv7: incorrect disassembly of VCVT #37

Closed
tzoz opened this issue Nov 22, 2020 · 6 comments
Closed

ARMv7: incorrect disassembly of VCVT #37

tzoz opened this issue Nov 22, 2020 · 6 comments
Labels
bug Something isn't working

Comments

@tzoz
Copy link

tzoz commented Nov 22, 2020 •
edited
Loading

Describe the bug
The sequence of bytes "E00AB7EE" (little-endian) should decode to

"vcvt.f64.f32 d0, s1"

But the ARM32 decoder will decode to

"vcvt.f64.f32 s0, s1"

It seems that the helper "getRegAL" is incorrect for VCVT.

To Reproduce

Try to decode "E00AB7EE" for ARMv7.

Expected behavior

The sequence of bytes should decode to

"vcvt.f64.f32 d0, s1".
@tzoz tzoz added the bug Something isn't working label Nov 22, 2020
@sangkilc
Copy link
Member

Thanks for creating the issue. We will address this issue in the next version.

@stale
Copy link

stale bot commented Jan 23, 2021

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the wontfix This will not be worked on label Jan 23, 2021
@sangkilc
Copy link
Member

This issue has been fixed in the internal repo. GitHub repo will be updated soon.

@stale stale bot removed the wontfix This will not be worked on label Jan 23, 2021
@stale
Copy link

stale bot commented Jun 2, 2021

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the wontfix This will not be worked on label Jun 2, 2021
@sangkilc sangkilc removed the wontfix This will not be worked on label Jun 2, 2021
@stale
Copy link

stale bot commented Aug 3, 2021

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the wontfix This will not be worked on label Aug 3, 2021
@sangkilc sangkilc removed the wontfix This will not be worked on label Aug 3, 2021
@sangkilc
Copy link
Member

This has been fixed in the new release.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@sangkilc @tzoz