Fixed bug in extractps instruction on ia32 and x64

Fixed bug in extractps instruction on ia32 and x64

Also fixed wrong roundsd instruction disassemble output

BUG=

[Weiliang, 2013-10-12 08:23:16]

Hi Yang, 
extractps is not used in current V8. I found this bug when prototyping SIMD in
V8. Of course, we could just delete extractps instruction, but from my view, I
hope it could be kept there.

[Sven Panne, 2013-10-14 07:43:54]

On 2013/10/12 08:23:16, Weiliang wrote:
> Hi Yang, 
> extractps is not used in current V8. I found this bug when prototyping SIMD in
> V8. Of course, we could just delete extractps instruction, but from my view, I
> hope it could be kept there.

Either we should fix it or remove the instruction, but given that it is needed
for some prototyping, I would vote for fixing it. Can you please add test cases
to test/cctest/test-{assembler,disasm}-{ia32,x64}.cc, because having unused
*and* untested stuff is not nice. :-)

[Weiliang, 2013-10-15 07:45:56]

On 2013/10/14 07:43:54, Sven Panne wrote:
> On 2013/10/12 08:23:16, Weiliang wrote:
> > Hi Yang, 
> > extractps is not used in current V8. I found this bug when prototyping SIMD
in
> > V8. Of course, we could just delete extractps instruction, but from my view,
I
> > hope it could be kept there.
> 
> Either we should fix it or remove the instruction, but given that it is needed
> for some prototyping, I would vote for fixing it. Can you please add test
cases
> to test/cctest/test-{assembler,disasm}-{ia32,x64}.cc, because having unused
> *and* untested stuff is not nice. :-)

Hi Sven, 
Thanks a lot for your comments. The testcases are added to
test/cctest/test-{assembler,disasm}-{ia32,x64}.cc. 
Could you please have a look. Thanks~

[Sven Panne, 2013-10-15 08:00:29]

lgtm

[Sven Panne, 2013-10-15 12:35:55]

NOT LGTM, actually. :-/ This CL makes the test suite fail in debug mode due to
missing CpuFeatureScopes and some IsSupported/IsEnabled confusion. I'll close
this issue a re-upload a fixed one.

[Weiliang, 2013-10-16 00:22:59]

On 2013/10/15 12:35:55, Sven Panne wrote:
> NOT LGTM, actually. :-/ This CL makes the test suite fail in debug mode due to
> missing CpuFeatureScopes and some IsSupported/IsEnabled confusion. I'll close
> this issue a re-upload a fixed one.

Please try the new CL. :)

[Sven Panne, 2013-10-16 07:21:34]

On 2013/10/16 00:22:59, Weiliang wrote:
> Please try the new CL. :)

There seems to be a misunderstanding here: I landed
https://code.google.com/p/v8/source/detail?r=17217 yesterday plus a follow-up
fix https://code.google.com/p/v8/source/detail?r=17218. Is there anything to be
done after these CLs?

[Weiliang, 2013-10-16 08:24:51]

On 2013/10/16 07:21:34, Sven Panne wrote:
> On 2013/10/16 00:22:59, Weiliang wrote:
> > Please try the new CL. :)
> 
> There seems to be a misunderstanding here: I landed
> https://code.google.com/p/v8/source/detail?r=17217 yesterday plus a follow-up
> fix https://code.google.com/p/v8/source/detail?r=17218. Is there anything to
be
> done after these CLs?

See, thanks~