[Kernel] replace function debuggable field with originalAsyncMarker field

runtime/vm/kernel_reader.cc

Index: runtime/vm/kernel_reader.cc
diff --git a/runtime/vm/kernel_reader.cc b/runtime/vm/kernel_reader.cc
index 752e42e667910e2f0b488177fb4fd96d654f8c8a..409bd46fc130014eae832f396d0977c690bf71a8 100644
--- a/runtime/vm/kernel_reader.cc
+++ b/runtime/vm/kernel_reader.cc
@@ -427,7 +427,26 @@ void KernelReader::ReadProcedure(const dart::Library& library,
   function.set_end_token_pos(kernel_procedure->end_position());
   owner.AddFunction(function);
   function.set_kernel_function(kernel_procedure);
-  function.set_is_debuggable(kernel_procedure->function()->debuggable());
+
+  function.set_is_debuggable(
+      kernel_procedure->function()->dart_async_marker() == FunctionNode::kSync);
+  switch (kernel_procedure->function()->dart_async_marker()) {
+    case FunctionNode::kSyncStar:
+      function.set_modifier(RawFunction::kSyncGen);
+      break;
+    case FunctionNode::kAsync:
+      function.set_modifier(RawFunction::kAsync);
+      break;
+    case FunctionNode::kAsyncStar:
+      function.set_modifier(RawFunction::kAsyncGen);
+      break;
+    default:
+      // no special modifier
+      break;
+  }
+  function.set_is_generated_body(kernel_procedure->function()->async_marker() ==

[Kevin Millikin (Google), 2017/02/22 10:46:03]

This is safe but we won't see kSyncYielding here so you could also change it to
an ASSERT.

[jensj, 2017/02/22 11:12:35]

Changing to
```
ASSERT(kernel_procedure->function()->async_marker() == FunctionNode::kSync);
```

+                                 FunctionNode::kSyncYielding);
+
   if (native_name != NULL) {
     function.set_native_name(*native_name);
   }