Do not embed is_auto_setup_scope into the compilation of native calls.

runtime/vm/native_entry.cc

 // Copyright (c) 2011, the Dart project authors.  Please see the AUTHORS file
 // for details. All rights reserved. Use of this source code is governed by a
 // BSD-style license that can be found in the LICENSE file.
 
 #include "vm/native_entry.h"
 
 #include "include/dart_api.h"
 
 #include "vm/bootstrap.h"
 #include "vm/code_patcher.h"
@@ skipping 73 matching lines @@
     ASSERT(!lib.IsNull());
     const uint8_t* r = ResolveSymbolInLibrary(lib, pc);
     if (r != NULL) {
       return r;
     }
   }
   return NULL;
 }
 
 
-uword NativeEntry::NativeCallWrapperEntry() {
-  uword entry = reinterpret_cast<uword>(NativeEntry::NativeCallWrapper);
-#if defined(USING_SIMULATOR) && !defined(TARGET_ARCH_DBC)
-  // DBC does not use redirections unlike other simulators.
-  entry = Simulator::RedirectExternalReference(
-      entry, Simulator::kNativeCall, NativeEntry::kNumCallWrapperArguments);
-#endif
-  return entry;
-}
-
-
 bool NativeEntry::ReturnValueIsError(NativeArguments* arguments) {
   RawObject* retval = arguments->ReturnValue();
   return (retval->IsHeapObject() &&
           RawObject::IsErrorClassId(retval->GetClassId()));
 }
 
 
 void NativeEntry::PropagateErrors(NativeArguments* arguments) {
   Thread* thread = arguments->thread();
   thread->UnwindScopes(thread->top_exit_frame_info());
 
   // The thread->zone() is different here than before we unwound.
   const Object& error =
       Object::Handle(thread->zone(), arguments->ReturnValue());
   Exceptions::PropagateError(Error::Cast(error));
   UNREACHABLE();
 }
 
 
-void NativeEntry::NativeCallWrapper(Dart_NativeArguments args,
-                                    Dart_NativeFunction func) {
-  CHECK_STACK_ALIGNMENT;
-  NativeCallWrapperNoStackCheck(args, func);
+uword NativeEntry::NoScopeNativeCallWrapperEntry() {
+  uword entry = reinterpret_cast<uword>(NativeEntry::NoScopeNativeCallWrapper);
+#if defined(USING_SIMULATOR) && !defined(TARGET_ARCH_DBC)
+  // DBC does not use redirections unlike other simulators.
+  entry = Simulator::RedirectExternalReference(
+      entry, Simulator::kNativeCall, NativeEntry::kNumCallWrapperArguments);
+#endif
+  return entry;
 }
 
 
-void NativeEntry::NativeCallWrapperNoStackCheck(Dart_NativeArguments args,
-                                                Dart_NativeFunction func) {
+void NativeEntry::NoScopeNativeCallWrapper(Dart_NativeArguments args,
+                                           Dart_NativeFunction func) {
+  CHECK_STACK_ALIGNMENT;
+  NoScopeNativeCallWrapperNoStackCheck(args, func);
+}
+
+
+void NativeEntry::NoScopeNativeCallWrapperNoStackCheck(
+    Dart_NativeArguments args,
+    Dart_NativeFunction func) {
   VERIFY_ON_TRANSITION;
   NativeArguments* arguments = reinterpret_cast<NativeArguments*>(args);
   /* Tell MemorySanitizer 'arguments' is initialized by generated code. */
   MSAN_UNPOISON(arguments, sizeof(*arguments));
   Thread* thread = arguments->thread();
   ASSERT(thread->execution_state() == Thread::kThreadInGenerated);
-  if (!arguments->IsNativeAutoSetupScope()) {
+  {
     TransitionGeneratedToNative transition(thread);
     func(args);
     if (ReturnValueIsError(arguments)) {
       PropagateErrors(arguments);
     }
-  } else {
+  }
+  ASSERT(thread->execution_state() == Thread::kThreadInGenerated);
+  VERIFY_ON_TRANSITION;
+}
+
+
+uword NativeEntry::AutoScopeNativeCallWrapperEntry() {
+  uword entry =
+      reinterpret_cast<uword>(NativeEntry::AutoScopeNativeCallWrapper);
+#if defined(USING_SIMULATOR) && !defined(TARGET_ARCH_DBC)
+  // DBC does not use redirections unlike other simulators.
+  entry = Simulator::RedirectExternalReference(
+      entry, Simulator::kNativeCall, NativeEntry::kNumCallWrapperArguments);
+#endif
+  return entry;
+}
+
+
+void NativeEntry::AutoScopeNativeCallWrapper(Dart_NativeArguments args,
+                                             Dart_NativeFunction func) {
+  CHECK_STACK_ALIGNMENT;
+  AutoScopeNativeCallWrapperNoStackCheck(args, func);
+}
+
+
+void NativeEntry::AutoScopeNativeCallWrapperNoStackCheck(
+    Dart_NativeArguments args,
+    Dart_NativeFunction func) {
+  VERIFY_ON_TRANSITION;
+  NativeArguments* arguments = reinterpret_cast<NativeArguments*>(args);
+  /* Tell MemorySanitizer 'arguments' is initialized by generated code. */
+  MSAN_UNPOISON(arguments, sizeof(*arguments));
+  Thread* thread = arguments->thread();
+  ASSERT(thread->execution_state() == Thread::kThreadInGenerated);
+  {
     Isolate* isolate = thread->isolate();
     ApiState* state = isolate->api_state();
     ASSERT(state != NULL);
     ApiLocalScope* current_top_scope = thread->api_top_scope();
     ApiLocalScope* scope = thread->api_reusable_scope();
     TRACE_NATIVE_CALL("0x%" Px "", reinterpret_cast<uintptr_t>(func));
     TransitionGeneratedToNative transition(thread);
     if (scope == NULL) {
       scope =
           new ApiLocalScope(current_top_scope, thread->top_exit_frame_info());
@@ skipping 22 matching lines @@
   }
   ASSERT(thread->execution_state() == Thread::kThreadInGenerated);
   VERIFY_ON_TRANSITION;
 }
 
 
 // DBC does not support lazy native call linking.
 #if !defined(TARGET_ARCH_DBC)
 static NativeFunction ResolveNativeFunction(Zone* zone,
                                             const Function& func,
-                                            bool* is_bootstrap_native) {
+                                            bool* is_bootstrap_native,
+                                            bool* is_auto_scope) {
   const Class& cls = Class::Handle(zone, func.Owner());
   const Library& library = Library::Handle(zone, cls.library());
 
   *is_bootstrap_native =
       Bootstrap::IsBootstapResolver(library.native_entry_resolver());
 
   const String& native_name = String::Handle(zone, func.native_name());
   ASSERT(!native_name.IsNull());
 
   const int num_params = NativeArguments::ParameterCountForResolution(func);
-  bool auto_setup_scope = true;
   return NativeEntry::ResolveNative(library, native_name, num_params,
-                                    &auto_setup_scope);
+                                    is_auto_scope);
 }
 
 
 uword NativeEntry::LinkNativeCallEntry() {
   uword entry = reinterpret_cast<uword>(NativeEntry::LinkNativeCall);
 #if defined(USING_SIMULATOR)
   entry = Simulator::RedirectExternalReference(
       entry, Simulator::kBootstrapNativeCall, NativeEntry::kNumArguments);
 #endif
   return entry;
 }
 
 
 void NativeEntry::LinkNativeCall(Dart_NativeArguments args) {
   CHECK_STACK_ALIGNMENT;
   VERIFY_ON_TRANSITION;
   NativeArguments* arguments = reinterpret_cast<NativeArguments*>(args);
   /* Tell MemorySanitizer 'arguments' is initialized by generated code. */
   MSAN_UNPOISON(arguments, sizeof(*arguments));
   TRACE_NATIVE_CALL("%s", "LinkNative");
 
   NativeFunction target_function = NULL;
-  bool call_through_wrapper = false;
+  bool is_bootstrap_native = false;
+  bool is_auto_scope = true;
 
   {
     TransitionGeneratedToVM transition(arguments->thread());
-    StackZone zone(arguments->thread());
+    StackZone stack_zone(arguments->thread());
+    Zone* zone = stack_zone.GetZone();
 
     DartFrameIterator iterator;
     StackFrame* caller_frame = iterator.NextFrame();
 
-    const Code& code = Code::Handle(caller_frame->LookupDartCode());
-    const Function& func = Function::Handle(code.function());
+    const Code& code = Code::Handle(zone, caller_frame->LookupDartCode());
+    const Function& func = Function::Handle(zone, code.function());
 
     if (FLAG_trace_natives) {
       OS::Print("Resolving native target for %s\n", func.ToCString());
     }
 
-    bool is_bootstrap_native = false;
-    target_function = ResolveNativeFunction(arguments->thread()->zone(), func,
-                                            &is_bootstrap_native);
+    target_function =
+        ResolveNativeFunction(arguments->thread()->zone(), func,
+                              &is_bootstrap_native, &is_auto_scope);
     ASSERT(target_function != NULL);
 
 #if defined(DEBUG)
     {
       NativeFunction current_function = NULL;
       const Code& current_trampoline =
-          Code::Handle(CodePatcher::GetNativeCallAt(caller_frame->pc(), code,
-                                                    &current_function));
+          Code::Handle(zone, CodePatcher::GetNativeCallAt(
+                                 caller_frame->pc(), code, &current_function));
 #if !defined(USING_SIMULATOR)
       ASSERT(current_function ==
              reinterpret_cast<NativeFunction>(LinkNativeCall));
 #else
       ASSERT(
           current_function ==
           reinterpret_cast<NativeFunction>(Simulator::RedirectExternalReference(
               reinterpret_cast<uword>(LinkNativeCall),
               Simulator::kBootstrapNativeCall, NativeEntry::kNumArguments)));
 #endif
       ASSERT(current_trampoline.raw() ==
-             StubCode::CallBootstrapCFunction_entry()->code());
+             StubCode::CallBootstrapNative_entry()->code());
     }
 #endif
 
-    call_through_wrapper = !is_bootstrap_native;
-    const Code& trampoline =
-        Code::Handle(call_through_wrapper
-                         ? StubCode::CallNativeCFunction_entry()->code()
-                         : StubCode::CallBootstrapCFunction_entry()->code());
-
     NativeFunction patch_target_function = target_function;
+    Code& trampoline = Code::Handle(zone);
+    if (is_bootstrap_native) {
+      trampoline = StubCode::CallBootstrapNative_entry()->code();
 #if defined(USING_SIMULATOR)
-    if (!call_through_wrapper) {
       patch_target_function =
           reinterpret_cast<NativeFunction>(Simulator::RedirectExternalReference(
               reinterpret_cast<uword>(patch_target_function),
               Simulator::kBootstrapNativeCall, NativeEntry::kNumArguments));
+#endif
+    } else if (is_auto_scope) {
+      trampoline = StubCode::CallAutoScopeNative_entry()->code();
+    } else {
+      trampoline = StubCode::CallNoScopeNative_entry()->code();
     }
-#endif
 
     CodePatcher::PatchNativeCallAt(caller_frame->pc(), code,
                                    patch_target_function, trampoline);
 
     if (FLAG_trace_natives) {
       OS::Print("    -> %p (%s)\n", target_function,
                 is_bootstrap_native ? "bootstrap" : "non-bootstrap");
     }
   }
   VERIFY_ON_TRANSITION;
 
   // Tail-call resolved target.
-  if (call_through_wrapper) {
+  if (is_bootstrap_native) {
+    target_function(arguments);
+  } else if (is_auto_scope) {
     // Because this call is within a compilation unit, Clang doesn't respect
     // the ABI alignment here.
-    NativeEntry::NativeCallWrapperNoStackCheck(
+    NativeEntry::AutoScopeNativeCallWrapperNoStackCheck(
         args, reinterpret_cast<Dart_NativeFunction>(target_function));
   } else {
-    target_function(arguments);
+    // Because this call is within a compilation unit, Clang doesn't respect
+    // the ABI alignment here.
+    NativeEntry::NoScopeNativeCallWrapperNoStackCheck(
+        args, reinterpret_cast<Dart_NativeFunction>(target_function));
   }
 }
 #endif  // !defined(TARGET_ARCH_DBC)
 
 
 }  // namespace dart