diff options
Diffstat (limited to 'deps/v8/src/runtime/runtime-generator.cc')
-rw-r--r-- | deps/v8/src/runtime/runtime-generator.cc | 83 |
1 files changed, 22 insertions, 61 deletions
diff --git a/deps/v8/src/runtime/runtime-generator.cc b/deps/v8/src/runtime/runtime-generator.cc index bb63a3d0d0..96486736e1 100644 --- a/deps/v8/src/runtime/runtime-generator.cc +++ b/deps/v8/src/runtime/runtime-generator.cc @@ -5,7 +5,6 @@ #include "src/runtime/runtime-utils.h" #include "src/arguments.h" -#include "src/debug/debug.h" #include "src/factory.h" #include "src/frames-inl.h" #include "src/objects-inl.h" @@ -15,76 +14,30 @@ namespace internal { RUNTIME_FUNCTION(Runtime_CreateJSGeneratorObject) { HandleScope scope(isolate); - DCHECK(args.length() == 2); + DCHECK_EQ(2, args.length()); CONVERT_ARG_HANDLE_CHECKED(JSFunction, function, 0); CONVERT_ARG_HANDLE_CHECKED(Object, receiver, 1); CHECK(IsResumableFunction(function->shared()->kind())); - Handle<FixedArray> operand_stack; - if (function->shared()->HasBytecodeArray()) { - // New-style generators. - DCHECK(!function->shared()->HasBaselineCode()); - int size = function->shared()->bytecode_array()->register_count(); - operand_stack = isolate->factory()->NewFixedArray(size); - } else { - // Old-style generators. - DCHECK(function->shared()->HasBaselineCode()); - operand_stack = isolate->factory()->empty_fixed_array(); - } + // Underlying function needs to have bytecode available. + DCHECK(function->shared()->HasBytecodeArray()); + DCHECK(!function->shared()->HasBaselineCode()); + int size = function->shared()->bytecode_array()->register_count(); + Handle<FixedArray> register_file = isolate->factory()->NewFixedArray(size); Handle<JSGeneratorObject> generator = isolate->factory()->NewJSGeneratorObject(function); generator->set_function(*function); generator->set_context(isolate->context()); generator->set_receiver(*receiver); - generator->set_operand_stack(*operand_stack); + generator->set_register_file(*register_file); generator->set_continuation(JSGeneratorObject::kGeneratorExecuting); return *generator; } -RUNTIME_FUNCTION(Runtime_SuspendJSGeneratorObject) { - HandleScope handle_scope(isolate); - DCHECK(args.length() == 1); - CONVERT_ARG_HANDLE_CHECKED(JSGeneratorObject, generator_object, 0); - - JavaScriptFrameIterator stack_iterator(isolate); - JavaScriptFrame* frame = stack_iterator.frame(); - CHECK(IsResumableFunction(frame->function()->shared()->kind())); - DCHECK_EQ(frame->function(), generator_object->function()); - DCHECK(frame->function()->shared()->is_compiled()); - DCHECK(!frame->function()->IsOptimized()); - - isolate->debug()->RecordAsyncFunction(generator_object); - - // The caller should have saved the context and continuation already. - DCHECK_EQ(generator_object->context(), Context::cast(frame->context())); - DCHECK_LT(0, generator_object->continuation()); - - // We expect there to be at least two values on the operand stack: the return - // value of the yield expression, and the arguments to this runtime call. - // Neither of those should be saved. - int operands_count = frame->ComputeOperandsCount(); - DCHECK_GE(operands_count, 1 + args.length()); - operands_count -= 1 + args.length(); - - if (operands_count == 0) { - // Although it's semantically harmless to call this function with an - // operands_count of zero, it is also unnecessary. - DCHECK_EQ(generator_object->operand_stack(), - isolate->heap()->empty_fixed_array()); - } else { - Handle<FixedArray> operand_stack = - isolate->factory()->NewFixedArray(operands_count); - frame->SaveOperandStack(*operand_stack); - generator_object->set_operand_stack(*operand_stack); - } - - return isolate->heap()->undefined_value(); -} - RUNTIME_FUNCTION(Runtime_GeneratorClose) { HandleScope scope(isolate); - DCHECK(args.length() == 1); + DCHECK_EQ(1, args.length()); CONVERT_ARG_HANDLE_CHECKED(JSGeneratorObject, generator, 0); generator->set_continuation(JSGeneratorObject::kGeneratorClosed); @@ -94,7 +47,7 @@ RUNTIME_FUNCTION(Runtime_GeneratorClose) { RUNTIME_FUNCTION(Runtime_GeneratorGetFunction) { HandleScope scope(isolate); - DCHECK(args.length() == 1); + DCHECK_EQ(1, args.length()); CONVERT_ARG_HANDLE_CHECKED(JSGeneratorObject, generator, 0); return generator->function(); @@ -102,15 +55,23 @@ RUNTIME_FUNCTION(Runtime_GeneratorGetFunction) { RUNTIME_FUNCTION(Runtime_GeneratorGetReceiver) { HandleScope scope(isolate); - DCHECK(args.length() == 1); + DCHECK_EQ(1, args.length()); CONVERT_ARG_HANDLE_CHECKED(JSGeneratorObject, generator, 0); return generator->receiver(); } +RUNTIME_FUNCTION(Runtime_GeneratorGetContext) { + HandleScope scope(isolate); + DCHECK_EQ(1, args.length()); + CONVERT_ARG_HANDLE_CHECKED(JSGeneratorObject, generator, 0); + + return generator->context(); +} + RUNTIME_FUNCTION(Runtime_GeneratorGetInputOrDebugPos) { HandleScope scope(isolate); - DCHECK(args.length() == 1); + DCHECK_EQ(1, args.length()); CONVERT_ARG_HANDLE_CHECKED(JSGeneratorObject, generator, 0); return generator->input_or_debug_pos(); @@ -118,7 +79,7 @@ RUNTIME_FUNCTION(Runtime_GeneratorGetInputOrDebugPos) { RUNTIME_FUNCTION(Runtime_GeneratorGetResumeMode) { HandleScope scope(isolate); - DCHECK(args.length() == 1); + DCHECK_EQ(1, args.length()); CONVERT_ARG_HANDLE_CHECKED(JSGeneratorObject, generator, 0); return Smi::FromInt(generator->resume_mode()); @@ -126,7 +87,7 @@ RUNTIME_FUNCTION(Runtime_GeneratorGetResumeMode) { RUNTIME_FUNCTION(Runtime_GeneratorGetContinuation) { HandleScope scope(isolate); - DCHECK(args.length() == 1); + DCHECK_EQ(1, args.length()); CONVERT_ARG_HANDLE_CHECKED(JSGeneratorObject, generator, 0); return Smi::FromInt(generator->continuation()); @@ -134,7 +95,7 @@ RUNTIME_FUNCTION(Runtime_GeneratorGetContinuation) { RUNTIME_FUNCTION(Runtime_GeneratorGetSourcePosition) { HandleScope scope(isolate); - DCHECK(args.length() == 1); + DCHECK_EQ(1, args.length()); CONVERT_ARG_HANDLE_CHECKED(JSGeneratorObject, generator, 0); if (!generator->is_suspended()) return isolate->heap()->undefined_value(); |