[esnext] prototype runtime implementation for async functions

src/factory.cc

Index: src/factory.cc
diff --git a/src/factory.cc b/src/factory.cc
index 9cebd3f1b470e8e743d163ee17fe69040445a113..7d1251e0b4f0c688a03df25e6f5978456876db4a 100644
--- a/src/factory.cc
+++ b/src/factory.cc
@@ -1227,6 +1227,7 @@ Handle<JSFunction> Factory::NewFunction(Handle<Map> map,
       map.is_identical_to(
           isolate()->sloppy_function_with_readonly_prototype_map()) ||
       map.is_identical_to(isolate()->strict_function_map()) ||
+      map.is_identical_to(isolate()->strict_function_without_prototype_map()) ||

[Dan Ehrenberg, 2016/05/04 22:52:36]

Why do we suddenly need this now, but we didn't need it for concise methods?

[caitp (gmail), 2016/05/06 18:45:26]

I have no idea --- but I'm pretty sure it is a mistake to omit it from here ---
because NewFunctionWithoutPrototype explicitly uses this map --- so it was just
broken before in debug builds if the function was strict.

Could split that out into a separate patch with a unit test or something, I
guess? But since it's a one liner I don't think it matters a lot

[Dan Ehrenberg, 2016/05/09 23:02:54]

Well, if it fixes a previous bug, then at least checking in tests for that bug
together with this patch would be a good thing to do. Splitting it into a
separate patch would be optimal still.

       // TODO(titzer): wasm_function_map() could be undefined here. ugly.
       (*map == context->get(Context::WASM_FUNCTION_MAP_INDEX)) ||
       map.is_identical_to(isolate()->proxy_function_map()));
@@ -1660,7 +1661,7 @@ void Factory::NewJSArrayStorage(Handle<JSArray> array,
 
 Handle<JSGeneratorObject> Factory::NewJSGeneratorObject(
     Handle<JSFunction> function) {
-  DCHECK(function->shared()->is_generator());
+  DCHECK(function->shared()->is_generator() || function->shared()->is_async());
   JSFunction::EnsureHasInitialMap(function);
   Handle<Map> map(function->initial_map());
   DCHECK_EQ(JS_GENERATOR_OBJECT_TYPE, map->instance_type());
@@ -2066,6 +2067,9 @@ Handle<SharedFunctionInfo> Factory::NewSharedFunctionInfo(
     shared->set_instance_class_name(isolate()->heap()->Generator_string());
     shared->DisableOptimization(kGenerator);
   }
+  if (IsAsyncFunction(kind)) {
+    shared->DisableOptimization(kGenerator);
+  }

[Dan Ehrenberg, 2016/05/04 22:52:36]

Nit: Any way you could merge this with the above DisableOptimization call, since
they're for the same reason and will both need to be changed in the near future?

   return shared;
 }