Migrate %TypedArray%.prototype.fill to C++

src/builtins/builtins-typedarray.cc

 // Copyright 2016 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.
 
 #include "src/builtins/builtins-utils.h"
 #include "src/builtins/builtins.h"
 #include "src/code-stub-assembler.h"
 #include "src/counters.h"
 #include "src/objects-inl.h"
 
@@ skipping 381 matching lines @@
 }  // namespace
 
 BUILTIN(TypedArrayPrototypeCopyWithin) {
   HandleScope scope(isolate);
 
   Handle<JSTypedArray> array;
   const char* method = "%TypedArray%.prototype.copyWithin";
   ASSIGN_RETURN_FAILURE_ON_EXCEPTION(
       isolate, array, JSTypedArray::Validate(isolate, args.receiver(), method));
 
-  if (V8_UNLIKELY(array->WasNeutered())) return *array;
-
   int64_t len = array->length_value();
   int64_t to = 0;
   int64_t from = 0;
   int64_t final = len;
 
   if (V8_LIKELY(args.length() > 1)) {
     Handle<Object> num;
     ASSIGN_RETURN_FAILURE_ON_EXCEPTION(
         isolate, num, Object::ToInteger(isolate, args.at<Object>(1)));
     to = CapRelativeIndex(num, 0, len);
@@ skipping 35 matching lines @@
   to = to * element_size;
   from = from * element_size;
   count = count * element_size;
 
   uint8_t* data = static_cast<uint8_t*>(elements->DataPtr());
   std::memmove(data + to, data + from, count);
 
   return *array;
 }
 
+BUILTIN(TypedArrayPrototypeFill) {
+  HandleScope scope(isolate);
+
+  Handle<JSTypedArray> array;
+  const char* method = "%TypedArray%.prototype.fill";
+  ASSIGN_RETURN_FAILURE_ON_EXCEPTION(
+      isolate, array, JSTypedArray::Validate(isolate, args.receiver(), method));
+
+  int64_t len = array->length_value();
+  int64_t start = 0;
+  int64_t end = len;
+  double value = 0.0;
+
+  if (V8_LIKELY(args.length() > 1)) {
+    Handle<Object> num;
+    ASSIGN_RETURN_FAILURE_ON_EXCEPTION(
+        isolate, num, Object::ToNumber(args.at<Object>(1)));
+
+    if (num->IsSmi()) {
+      value = Smi::cast(*num)->value();
+    } else {
+      DCHECK(num->IsHeapNumber());
+      value = HeapNumber::cast(*num)->value();
+    }
+    if (array->type() == kExternalUint8ClampedArray) {
+      value = std::min<double>(std::max<double>(0, value), 255);
+    }
+
+    Handle<Object> optional = args.atOrUndefined(isolate, 2);
+    if (!optional->IsUndefined(isolate)) {
+      ASSIGN_RETURN_FAILURE_ON_EXCEPTION(
+          isolate, num, Object::ToInteger(isolate, optional));
+      start = CapRelativeIndex(num, 0, len);
+
+      optional = args.atOrUndefined(isolate, 3);
+      if (!optional->IsUndefined(isolate)) {
+        ASSIGN_RETURN_FAILURE_ON_EXCEPTION(
+            isolate, num, Object::ToInteger(isolate, optional));
+        end = CapRelativeIndex(num, 0, len);
+      }
+    }
+  }
+
+  int64_t count = end - start;
+  if (count <= 0) return *array;
+
+  if (V8_UNLIKELY(array->WasNeutered())) return *array;
+
+  // Ensure processed indexes are within array bounds
+  DCHECK_GE(start, 0);
+  DCHECK_LT(start, len);
+  DCHECK_GE(end, 0);
+  DCHECK_LE(end, len);
+  DCHECK_LE(count, len);
+
+  switch (array->type()) {
+#define TYPED_ARRAY_FILL(Type, type, TYPE, ctype, size)                \
+  case kExternal##Type##Array: {                                       \
+    ctype* backing_store =                                             \
+        static_cast<ctype*>(array->GetBuffer()->backing_store());      \

[Choongwoo Han, 2017/03/08 08:18:30]

You should consider byte_offset for backing_store.

+    ctype cast_value = static_cast<ctype>(value);                         \
+    std::fill(backing_store + start, backing_store + end, cast_value); \
+    break;                                                             \
+  }
+
+    TYPED_ARRAYS(TYPED_ARRAY_FILL)
+#undef TYPED_ARRAY_FILL
+  }
+
+  return *array;
+}
+
 }  // namespace internal
 }  // namespace v8