[wasm][arm] Add an additional stack check for functions with big frames.

src/runtime/runtime-wasm.cc

 // Copyright 2016 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/arguments.h"
 #include "src/assembler.h"
 #include "src/compiler/wasm-compiler.h"
 #include "src/conversions.h"
@@ skipping 118 matching lines @@
   return ThrowRuntimeError(isolate, message_id, 0, false);
 }
 
 RUNTIME_FUNCTION(Runtime_ThrowWasmError) {
   DCHECK_EQ(2, args.length());
   CONVERT_SMI_ARG_CHECKED(message_id, 0);
   CONVERT_SMI_ARG_CHECKED(byte_offset, 1);
   return ThrowRuntimeError(isolate, message_id, byte_offset, true);
 }
 
+RUNTIME_FUNCTION(Runtime_ThrowWasmStackOverflow) {
+  HandleScope scope(isolate);
+  DCHECK_NULL(isolate->context());
+  isolate->set_context(GetWasmContextOnStackTop(isolate));
+  SealHandleScope shs(isolate);

[Michael Starzinger, 2017/03/20 14:54:51]

nit: The {Isolate::StackOverflow} method creates its own handle scope, not much
sense in sealing the current one I think. Can we just drop this?

[ahaas, 2017/03/20 15:16:23]

I removed the HandleScope and moved the SealHandleScope to line 1.

+  DCHECK_LE(0, args.length());

[Michael Starzinger, 2017/03/20 14:54:51]

nit: DCHECK_EQ? Also, please move the check up to right after the HandleScope,
to keep it in sync with other runtime functions.

[ahaas, 2017/03/20 15:16:23]

Done.

+  return isolate->StackOverflow();
+}
+
 RUNTIME_FUNCTION(Runtime_WasmThrowTypeError) {
   HandleScope scope(isolate);
   DCHECK_EQ(0, args.length());
   THROW_NEW_ERROR_RETURN_FAILURE(
       isolate, NewTypeError(MessageTemplate::kWasmTrapTypeError));
 }
 
 RUNTIME_FUNCTION(Runtime_WasmThrow) {
   HandleScope scope(isolate);
   DCHECK_EQ(2, args.length());
@@ skipping 72 matching lines @@
 
   // Check if this is a real stack overflow.
   StackLimitCheck check(isolate);
   if (check.JsHasOverflowed()) return isolate->StackOverflow();
 
   return isolate->stack_guard()->HandleInterrupts();
 }
 
 }  // namespace internal
 }  // namespace v8