[Atomics] Handle conversion to SMI in builtin, not code stub.

src/ia32/code-stubs-ia32.cc

 // Copyright 2012 the V8 project authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #if V8_TARGET_ARCH_IA32
 
 #include "src/code-stubs.h"
 #include "src/api-arguments.h"
 #include "src/base/bits.h"
 #include "src/bootstrapper.h"
@@ skipping 5884 matching lines @@
   __ dd(i32);     // Int32Array
   __ dd(u32);     // Uint32Array
   __ dd(&abort);  // Float32Array
   __ dd(&abort);  // Float64Array
   __ dd(u8c);     // Uint8ClampedArray
 
   __ bind(&abort);
   __ Abort(kNoReason);
 }
 
-void ReturnInteger32(MacroAssembler* masm, XMMRegister dst, Register value,
-                     Register scratch, Label* use_heap_number) {
-  Label not_smi;
-  if (!value.is(eax)) {
-    __ mov(eax, value);
-  }
-  __ JumpIfNotValidSmiValue(eax, scratch, &not_smi, Label::kNear);
-  __ SmiTag(eax);
-  __ Ret();
-
-  __ bind(&not_smi);
-  __ Cvtsi2sd(dst, eax);
-  __ jmp(use_heap_number);
-}
-
-void ReturnUnsignedInteger32(MacroAssembler* masm, XMMRegister dst,
-                             Register value, XMMRegister scratch,
-                             Label* use_heap_number) {
-  Label not_smi;
-  if (!value.is(eax)) {
-    __ mov(eax, value);
-  }
-  __ JumpIfUIntNotValidSmiValue(eax, &not_smi, Label::kNear);
-  __ SmiTag(eax);
-  __ Ret();
-
-  __ bind(&not_smi);
-  // Convert [0, 2**32-1] -> [-2**31, 2**31-1].
-  __ add(eax, Immediate(-0x7fffffff - 1));  // -0x80000000 parses incorrectly.
-  __ Cvtsi2sd(dst, eax);
-  __ mov(eax, Immediate(0x4f000000));  // 2**31 as IEEE float
-  __ movd(scratch, eax);
-  __ cvtss2sd(scratch, scratch);
-  __ addsd(dst, scratch);
-  __ jmp(use_heap_number);
-}
-
-void ReturnAllocatedHeapNumber(MacroAssembler* masm, XMMRegister value,
-                               Register scratch, Register scratch2) {
-  Label call_runtime;
-  __ AllocateHeapNumber(eax, scratch, scratch2, &call_runtime);
-  __ movsd(FieldOperand(eax, HeapNumber::kValueOffset), value);
-  __ Ret();
-
-  __ bind(&call_runtime);
-  {
-    FrameScope scope(masm, StackFrame::INTERNAL);
-    __ CallRuntimeSaveDoubles(Runtime::kAllocateHeapNumber);
-    __ movsd(FieldOperand(eax, HeapNumber::kValueOffset), value);
-  }
-  __ Ret();
-}
-
 }  // anonymous namespace
 
 void AtomicsLoadStub::Generate(MacroAssembler* masm) {
   Register object = edx;
   Register index = eax;  // Index is an untagged word32.
   Register backing_store = ebx;
   Label table;
 
   GetTypedArrayBackingStore(masm, backing_store, object, ecx);
   TypedArrayJumpTablePrologue(masm, object, ecx, esi, &table);
 
   Label i8, u8, i16, u16, i32, u32;
 
   __ bind(&i8);
   __ mov_b(eax, Operand(backing_store, index, times_1, 0));
   __ movsx_b(eax, eax);
-  __ SmiTag(eax);
   __ Ret();
 
   __ bind(&u8);
   __ mov_b(eax, Operand(backing_store, index, times_1, 0));
   __ movzx_b(eax, eax);
-  __ SmiTag(eax);
   __ Ret();
 
   __ bind(&i16);
   __ mov_w(eax, Operand(backing_store, index, times_2, 0));
   __ movsx_w(eax, eax);
-  __ SmiTag(eax);
   __ Ret();
 
   __ bind(&u16);
   __ mov_w(eax, Operand(backing_store, index, times_2, 0));
   __ movzx_w(eax, eax);
-  __ SmiTag(eax);
   __ Ret();
 
-  Label use_heap_number;
-
   __ bind(&i32);
   __ mov(eax, Operand(backing_store, index, times_4, 0));
-  ReturnInteger32(masm, xmm0, eax, ecx, &use_heap_number);
+  __ Ret();
 
   __ bind(&u32);
   __ mov(eax, Operand(backing_store, index, times_4, 0));
-  ReturnUnsignedInteger32(masm, xmm0, eax, xmm1, &use_heap_number);
-
-  __ bind(&use_heap_number);
-  ReturnAllocatedHeapNumber(masm, xmm0, ecx, edx);
+  __ Ret();
 
   TypedArrayJumpTableEpilogue(masm, &table, &i8, &u8, &i16, &u16, &i32, &u32,
                               &u8);
 }
 
 #undef __
 
 }  // namespace internal
 }  // namespace v8
 
 #endif  // V8_TARGET_ARCH_IA32