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

src/x64/code-stubs-x64.cc

 // Copyright 2013 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_X64
 
 #include "src/code-stubs.h"
 #include "src/api-arguments.h"
 #include "src/bootstrapper.h"
 #include "src/codegen.h"
@@ skipping 5610 matching lines @@
   __ dq(i32);     // Int32Array
   __ dq(u32);     // Uint32Array
   __ dq(&abort);  // Float32Array
   __ dq(&abort);  // Float64Array
   __ dq(u8c);     // Uint8ClampedArray
 
   __ bind(&abort);
   __ Abort(kNoReason);
 }
 
-void ReturnInteger32(MacroAssembler* masm, XMMRegister dst, Register value,
-                     Label* use_heap_number) {
-  Label not_smi;
-  if (!value.is(rax)) {
-    __ movp(rax, value);
-  }
-  __ JumpIfNotValidSmiValue(rax, &not_smi, Label::kNear);
-  __ Integer32ToSmi(rax, rax);
-  __ Ret();
-
-  __ bind(&not_smi);
-  __ Cvtlsi2sd(dst, rax);
-  __ jmp(use_heap_number);
-}
-
-void ReturnUnsignedInteger32(MacroAssembler* masm, XMMRegister dst,
-                             Register value, Label* use_heap_number) {
-  Label not_smi;
-  if (!value.is(rax)) {
-    __ movp(rax, value);
-  }
-  __ JumpIfUIntNotValidSmiValue(rax, &not_smi, Label::kNear);
-  __ Integer32ToSmi(rax, rax);
-  __ Ret();
-
-  __ bind(&not_smi);
-  __ Cvtqsi2sd(dst, rax);
-  __ jmp(use_heap_number);
-}
-
-void ReturnAllocatedHeapNumber(MacroAssembler* masm, XMMRegister value,
-                               Register scratch) {
-  Label call_runtime;
-  __ AllocateHeapNumber(rax, scratch, &call_runtime);
-  __ Movsd(FieldOperand(rax, HeapNumber::kValueOffset), value);
-  __ Ret();
-
-  __ bind(&call_runtime);
-  {
-    FrameScope scope(masm, StackFrame::INTERNAL);
-    __ CallRuntimeSaveDoubles(Runtime::kAllocateHeapNumber);
-    __ Movsd(FieldOperand(rax, HeapNumber::kValueOffset), value);
-  }
-  __ Ret();
-}
-
 }  // anonymous namespace
 
 void AtomicsLoadStub::Generate(MacroAssembler* masm) {
   Register object = rdx;
   Register index = rax;  // Index is an untagged word32.
   Register backing_store = rbx;
   Label table;
 
   GetTypedArrayBackingStore(masm, backing_store, object, kScratchRegister);
   TypedArrayJumpTablePrologue(masm, object, rcx, kScratchRegister, &table);
 
   Label i8, u8, i16, u16, i32, u32;
 
   __ bind(&i8);
   __ movb(rax, Operand(backing_store, index, times_1, 0));
   __ movsxbl(rax, rax);
-  __ Integer32ToSmi(rax, rax);
   __ Ret();
 
   __ bind(&u8);
   __ movb(rax, Operand(backing_store, index, times_1, 0));
   __ movzxbl(rax, rax);
-  __ Integer32ToSmi(rax, rax);
   __ Ret();
 
   __ bind(&i16);
   __ movw(rax, Operand(backing_store, index, times_2, 0));
   __ movsxwl(rax, rax);
-  __ Integer32ToSmi(rax, rax);
   __ Ret();
 
   __ bind(&u16);
   __ movw(rax, Operand(backing_store, index, times_2, 0));
   __ movzxwl(rax, rax);
-  __ Integer32ToSmi(rax, rax);
   __ Ret();
 
-  Label use_heap_number;
-
   __ bind(&i32);
   __ movl(rax, Operand(backing_store, index, times_4, 0));
-  ReturnInteger32(masm, xmm0, rax, &use_heap_number);
+  __ Ret();
 
   __ bind(&u32);
   __ movl(rax, Operand(backing_store, index, times_4, 0));
-  ReturnUnsignedInteger32(masm, xmm0, rax, &use_heap_number);
-
-  __ bind(&use_heap_number);
-  ReturnAllocatedHeapNumber(masm, xmm0, rcx);
+  __ Ret();
 
   TypedArrayJumpTableEpilogue(masm, &table, &i8, &u8, &i16, &u16, &i32, &u32,
                               &u8);
 }
 
 #undef __
 
 }  // namespace internal
 }  // namespace v8
 
 #endif  // V8_TARGET_ARCH_X64