Introduce LiteralsArray to hide it's implementation.

src/factory.cc

 // Copyright 2014 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/factory.h"
 
 #include "src/allocation-site-scopes.h"
 #include "src/base/bits.h"
 #include "src/bootstrapper.h"
 #include "src/conversions.h"
@@ skipping 1306 matching lines @@
     DCHECK(!cached.code->marked_for_deoptimization());
     DCHECK(result->shared()->is_compiled());
     result->ReplaceCode(cached.code);
   }
 
   if (cached.literals != nullptr) {
     result->set_literals(cached.literals);
 
   } else if (!info->bound()) {
     int number_of_literals = info->num_literals();
-    Handle<FixedArray> literals = NewFixedArray(number_of_literals, pretenure);
+    Handle<LiteralsArray> literals = NewLiteralsArray(
+        handle(info->feedback_vector()), number_of_literals, pretenure);
     result->set_literals(*literals);
     // Cache context-specific literals.
     if (FLAG_cache_optimized_code) {
       Handle<Context> native_context(context->native_context());
       SharedFunctionInfo::AddToOptimizedCodeMap(
           info, native_context, undefined_value(), literals, BailoutId::None());
     }
   }
 
   return result;
@@ skipping 741 matching lines @@
   shared->set_kind(kind);
   shared->set_num_literals(number_of_literals);
   if (IsGeneratorFunction(kind)) {
     shared->set_instance_class_name(isolate()->heap()->Generator_string());
     shared->DisableOptimization(kGenerator);
   }
   return shared;
 }
 
 
+Handle<LiteralsArray> Factory::NewLiteralsArray(

[Igor Sheludko, 2015/09/29 08:40:41]

I think we don't need this method, just call LiteralsArray::New() instead.

+    Handle<TypeFeedbackVector> vector, int number_of_literals,
+    PretenureFlag pretenure) {
+  return LiteralsArray::Allocate(isolate(), vector, number_of_literals,
+                                 pretenure);
+}
+
+
 Handle<JSMessageObject> Factory::NewJSMessageObject(
     MessageTemplate::Template message, Handle<Object> argument,
     int start_position, int end_position, Handle<Object> script,
     Handle<Object> stack_frames) {
   Handle<Map> map = message_object_map();
   Handle<JSMessageObject> message_obj = New<JSMessageObject>(map, NEW_SPACE);
   message_obj->set_properties(*empty_fixed_array(), SKIP_WRITE_BARRIER);
   message_obj->initialize_elements();
   message_obj->set_elements(*empty_fixed_array(), SKIP_WRITE_BARRIER);
   message_obj->set_type(message);
@@ skipping 275 matching lines @@
 }
 
 
 Handle<Object> Factory::ToBoolean(bool value) {
   return value ? true_value() : false_value();
 }
 
 
 }  // namespace internal
 }  // namespace v8