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

x86 sse - can't disassemble MULSD, MULSS commands #368

Closed
XVilka opened this issue Nov 16, 2013 · 3 comments
Closed

x86 sse - can't disassemble MULSD, MULSS commands #368

XVilka opened this issue Nov 16, 2013 · 3 comments
Milestone

Comments

@XVilka
Copy link
Contributor

XVilka commented Nov 16, 2013

rasm2 -d -a x86 -b 64 66f3f20f59ff

invalid
should be 'MULSD xmm7, xmm7'

rasm2 -d -a x86 -b 64 66f2f30f59ff

invalid
should be 'MULSS xmm7, xmm7'

rasm2 -d -a x86 -b 64 f2660f59ff

invalid
should be 'MULSD xmm7, xmm7'

source is here http://habrahabr.ru/company/intel/blog/200658/

regression test included in https://github.com/radare/r2-regressions/blob/master/t/disasm/x86_sse

@radare
Copy link
Collaborator

radare commented Nov 16, 2013

Please report to the udis86 repo

On 16 Nov 2013, at 13:46, Anton Kochkov notifications@github.com wrote:

rasm2 -d -a x86 -b 64 66f3f20f59ff

invalid
should be 'MULSD xmm7, xmm7'

rasm2 -d -a x86 -b 64 66f2f30f59ff

invalid
should be 'MULSS xmm7, xmm7'

rasm2 -d -a x86 -b 64 f2660f59ff

invalid
should be 'MULSD xmm7, xmm7'

source is here http://habrahabr.ru/company/intel/blog/200658/

regression test included in https://github.com/radare/r2-regressions/blob/master/t/disasm/x86_sse


Reply to this email directly or view it on GitHub.

@XVilka
Copy link
Contributor Author

XVilka commented Nov 16, 2013

@radare already done that. See upper comment. Looks like github does not sending email on all changes of the bug.

@jvoisin
Copy link
Contributor

jvoisin commented Nov 26, 2013

$ rasm2 -d -a x86 -b 64 66f3f20f59ff
o16 mulsd xmm7, xmm7
$
$ rasm2 -d -a x86 -b 64 66f2f30f59ff
o16 mulss xmm7, xmm7
$
$ rasm2 -d -a x86 -b 64 f2660f59ff
o16 mulsd xmm7, xmm7
$

This seems to be fixed.

@radare radare closed this as completed Nov 26, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants