Hydrogen stubs for array constructors

src/builtins.cc

Index: src/builtins.cc
diff --git a/src/builtins.cc b/src/builtins.cc
index e593d5c2c971b6378149790c71ada2501dfe2abf..df13c59425c6589e1b021c1acd6e70eab960758e 100644
--- a/src/builtins.cc
+++ b/src/builtins.cc
@@ -200,9 +200,8 @@ BUILTIN(EmptyFunction) {
 
 RUNTIME_FUNCTION(MaybeObject*, ArrayConstructor_StubFailure) {
   CONVERT_ARG_STUB_CALLER_ARGS(caller_args);
-  // ASSERT(args.length() == 3);
-  Handle<JSFunction> function = args.at<JSFunction>(1);
-  Handle<Object> type_info = args.at<Object>(2);
+  // ASSERT(args.length() == 2);

[Hannes Payer (out of office), 2013/04/18 11:14:39]

Is this ASSERT still needed?

[mvstanton, 2013/04/18 13:39:26]

It's good to have, and I should enable the darn thing. Enabled.

+  Handle<Object> type_info = args.at<Object>(1);
 
   JSArray* array = NULL;
   bool holey = false;
@@ -234,8 +233,7 @@ RUNTIME_FUNCTION(MaybeObject*, ArrayConstructor_StubFailure) {
     }
   }
 
-  ASSERT(function->has_initial_map());
-  ElementsKind kind = function->initial_map()->elements_kind();
+  ElementsKind kind = GetInitialFastElementsKind();
   if (holey) {
     kind = GetHoleyElementsKind(kind);
   }