[debugger] fix stepping over await calls for ignition generators.

src/interpreter/interpreter-assembler.cc

 // Copyright 2015 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/interpreter/interpreter-assembler.h"
 
 #include <limits>
 #include <ostream>
 
 #include "src/code-factory.h"
@@ skipping 602 matching lines @@
 }
 
 Node* InterpreterAssembler::StackCheckTriggeredInterrupt() {
   Node* sp = LoadStackPointer();
   Node* stack_limit = Load(
       MachineType::Pointer(),
       ExternalConstant(ExternalReference::address_of_stack_limit(isolate())));
   return UintPtrLessThan(sp, stack_limit);
 }
 
+Node* InterpreterAssembler::DebuggerIsActive() {
+  return Load(
+      MachineType::Pointer(),

[Benedikt Meurer, 2016/06/14 03:45:36]

This doesn't look quite right here. The is_active_ field is a bool, so we
probably need MachineType::Int8() here, although I'm not sure that C++ always
stores bool into int8. Maybe we can explicitly change the fields in debug.h to
int8_t to be sure?

[rmcilroy, 2016/06/14 06:23:11]

Yes, it's implementation defined. It's probably always a byte on compilers we
use, but I agree we should change the field to be sure.

[Yang, 2016/06/14 09:17:02]

I changed this. We now don't load is_active_ anymore, but last_step_action_
instead.

+      ExternalConstant(ExternalReference::debug_is_active_address(isolate())));
+}
+
 void InterpreterAssembler::Abort(BailoutReason bailout_reason) {
   disable_stack_check_across_call_ = true;
   Node* abort_id = SmiTag(Int32Constant(bailout_reason));
   CallRuntime(Runtime::kAbort, GetContext(), abort_id);
   disable_stack_check_across_call_ = false;
 }
 
 void InterpreterAssembler::AbortIfWordNotEqual(Node* lhs, Node* rhs,
                                                BailoutReason bailout_reason) {
   Label ok(this), abort(this, Label::kDeferred);
@@ skipping 128 matching lines @@
     Goto(&loop);
   }
   Bind(&done_loop);
 
   return array;
 }
 
 }  // namespace interpreter
 }  // namespace internal
 }  // namespace v8