[v8-dev] [v8] r13260 committed - Refactoring only: Use CheckSSE2OperandIsInt32 instead of copy-n-paste ...

Fri, 21 Dec 2012 02:20:02 -0800 

Revision: 13260
Author:   [email protected]
Date:     Fri Dec 21 02:19:42 2012
Log: Refactoring only: Use CheckSSE2OperandIsInt32 instead of copy-n-paste code. 

Review URL: https://codereview.chromium.org/11666004
http://code.google.com/p/v8/source/detail?r=13260

Modified:
 /branches/bleeding_edge/src/ia32/code-stubs-ia32.cc

=======================================

--- /branches/bleeding_edge/src/ia32/code-stubs-ia32.cc	Tue Dec 18 08:25:45  
2012
+++ /branches/bleeding_edge/src/ia32/code-stubs-ia32.cc	Fri Dec 21 02:19:42  
2012

@@ -1757,12 +1757,8 @@
           }
           // Check result type if it is currently Int32.
           if (result_type_ <= BinaryOpIC::INT32) {
-            __ cvttsd2si(ecx, Operand(xmm0));
-            __ cvtsi2sd(xmm2, ecx);
-            __ pcmpeqd(xmm2, xmm0);
-            __ movmskpd(ecx, xmm2);
-            __ test(ecx, Immediate(1));
-            __ j(zero, &not_int32);
+            FloatingPointHelper::CheckSSE2OperandIsInt32(
+                masm, &not_int32, xmm0, ecx, xmm2);
           }

           BinaryOpStub_GenerateHeapResultAllocation(masm, &call_runtime,  
mode_);

           __ movdbl(FieldOperand(eax, HeapNumber::kValueOffset), xmm0);

--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev














    











					Reply via email to