Implemented missing instructions in ia32 and x64 disassembler....

Implemented missing instructions in ia32 and x64 disassembler.

ia32: fld(i), fldpi
x64: fld(i), fldpi, cvtsd2si, cvttsd2si


Committed: http://code.google.com/p/v8/source/detail?r=5525

[Vladislav Kaznacheev, 2010-09-24 11:50:16]

Hi Lasse,

While debugging r5515 (dec_b support to ia32 and x64 disassembler) I noticed
quite a few "Unimplemented instruction" lines in the code. 

I fixed that (at least nothing unimplemented shows up for sunspider tests). 

Please take a look, I might be doing something not quite optimal.

[Vladislav Kaznacheev, 2010-09-24 11:50:16]

Hi Lasse,

While debugging r5515 (dec_b support to ia32 and x64 disassembler) I noticed
quite a few "Unimplemented instruction" lines in the code. 

I fixed that (at least nothing unimplemented shows up for sunspider tests). 

Please take a look, I might be doing something not quite optimal.

[Lasse Reichstein, 2010-09-24 12:19:16]

LGTM

http://codereview.chromium.org/3471011/diff/1/3
File src/x64/disasm-x64.cc (right):

http://codereview.chromium.org/3471011/diff/1/3#newcode1116
src/x64/disasm-x64.cc:1116: current += PrintOperands("cvttss2si",
REG_OPER_OP_ORDER, current);
Could this function be used for cvt[t]sd2si above?
(If it includes the operand size, it seems it should work the same).

[Vladislav Kaznacheev, 2010-09-24 12:55:05]

Thanks for the review! Submitting.

http://codereview.chromium.org/3471011/diff/1/3
File src/x64/disasm-x64.cc (right):

http://codereview.chromium.org/3471011/diff/1/3#newcode1116
src/x64/disasm-x64.cc:1116: current += PrintOperands("cvttss2si",
REG_OPER_OP_ORDER, current);
This was the first thing I tried. PrintOperand would call PrintRightOperand
instead of PrintRightXMM operand. Teaching it to do this right would be more
complex, I think.
On 2010/09/24 12:19:16, Lasse Reichstein wrote:
> Could this function be used for cvt[t]sd2si above?
> (If it includes the operand size, it seems it should work the same).