asmjit / asmdb

Instructions database and utilities for X86/X64 and ARM (THUMB/A32/A64) architectures.
The Unlicense
328 stars 46 forks source link

possible typo in movq (and maybe movd is as well) #22

Open robertmuth opened 2 years ago

robertmuth commented 2 years ago

movq is the only instruction using this descriptor: r64[63:0]/m64 "movq" , "W:xmm[63:0], r64[63:0]/m64"

should this be just r64/m64 ? Especially since the MR variant looks like: "movq" , "W:r64/m64, xmm[63:0]"

movd seems also suspicious: "movd" , "W:r32[31:0]/m32, xmm[31:0]"
"movd" , "W:xmm[31:0], R:r32[31:0]/m32"

but the use of r32[31:0] sees to be more widespread,

kobalicek commented 2 years ago

Yeah, I think you are right and that the indexes are redundant in these cases, I mean r32[31:0] is essentially r32

kobalicek commented 2 years ago

BTW I 'm planning to fix some of the issues that you have reported, I just didn't find a time window for that yet