From 72975d35974ed79b9e819fcf68ee44dc34c88a3b Mon Sep 17 00:00:00 2001 From: Michael Starzinger Date: Wed, 8 Mar 2017 16:01:35 +0100 Subject: [PATCH] [bootstrapper] Bump stack requirement for native compilation. This bumps the required stack space gap when the bootstrapper kicks off compilation of native scripts during genesis. The stack requirements are now higher for simulators in no-snapshot mode with Ignition being used. R=jochen@chromium.org TEST=mjsunit/regress/regress-681984 BUG=v8:6066 Change-Id: I65df388d1b57c09db124ac9a85d380b4edb7d260 Reviewed-on: https://chromium-review.googlesource.com/451275 Commit-Queue: Michael Starzinger Reviewed-by: Jochen Eisinger Cr-Commit-Position: refs/heads/master@{#43675} --- src/bootstrapper.cc | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/bootstrapper.cc b/src/bootstrapper.cc index 8e7c0fc0ae0..f76aa35bbea 100644 --- a/src/bootstrapper.cc +++ b/src/bootstrapper.cc @@ -3132,7 +3132,7 @@ bool Bootstrapper::CompileNative(Isolate* isolate, Vector name, // environment has been at least partially initialized. Add a stack check // before entering JS code to catch overflow early. StackLimitCheck check(isolate); - if (check.JsHasOverflowed(1 * KB)) { + if (check.JsHasOverflowed(4 * KB)) { isolate->StackOverflow(); return false; }