[modules] AST and parser rework.

src/runtime/runtime-function.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/runtime/runtime-utils.h"
 
 #include "src/accessors.h"
 #include "src/arguments.h"
 #include "src/compiler.h"
 #include "src/frames-inl.h"
@@ skipping 21 matching lines @@
 
 RUNTIME_FUNCTION(Runtime_FunctionSetName) {
   HandleScope scope(isolate);
   DCHECK(args.length() == 2);
 
   CONVERT_ARG_HANDLE_CHECKED(JSFunction, f, 0);
   CONVERT_ARG_HANDLE_CHECKED(String, name, 1);
 
   name = String::Flatten(name);
   f->shared()->set_name(*name);
-  return isolate->heap()->undefined_value();
+  return *f;

[adamk, 2016/07/13 18:38:22]

I think you won't need this change after you change how this gets set.

[neis, 2016/07/14 10:28:24]

Done.

 }
 
 
 RUNTIME_FUNCTION(Runtime_FunctionRemovePrototype) {
   SealHandleScope shs(isolate);
   DCHECK(args.length() == 1);
 
   CONVERT_ARG_CHECKED(JSFunction, f, 0);
   CHECK(f->RemovePrototype());
   f->shared()->SetConstructStub(
@@ skipping 243 matching lines @@
   DCHECK_EQ(1, args.length());
   CONVERT_ARG_HANDLE_CHECKED(JSReceiver, function, 0);
   return function->IsJSBoundFunction()
              ? *JSBoundFunction::ToString(
                    Handle<JSBoundFunction>::cast(function))
              : *JSFunction::ToString(Handle<JSFunction>::cast(function));
 }
 
 }  // namespace internal
 }  // namespace v8