Revision: 11236
Author: [email protected]
Date: Thu Apr 5 05:32:35 2012
Log: Fix stack overflows on Windows x64.
[email protected]
TEST=win 64 not red anymore
Review URL: https://chromiumcodereview.appspot.com/10008005
http://code.google.com/p/v8/source/detail?r=11236
Modified:
/branches/bleeding_edge/src/flag-definitions.h
/branches/bleeding_edge/test/mjsunit/regress/regress-119429.js
=======================================
--- /branches/bleeding_edge/src/flag-definitions.h Fri Mar 23 06:33:11 2012
+++ /branches/bleeding_edge/src/flag-definitions.h Thu Apr 5 05:32:35 2012
@@ -310,7 +310,9 @@
DEFINE_bool(break_on_abort, true, "always cause a debug break before
aborting")
// execution.cc
-DEFINE_int(stack_size, kPointerSize * 128,
+// Slightly less than 1MB on 64-bit, since Windows' default stack size for
+// the main execution thread is 1MB for both 32 and 64-bit.
+DEFINE_int(stack_size, kPointerSize * 125,
"default size of stack region v8 is allowed to use (in kBytes)")
// frames.cc
=======================================
--- /branches/bleeding_edge/test/mjsunit/regress/regress-119429.js Tue Apr
3 06:45:56 2012
+++ /branches/bleeding_edge/test/mjsunit/regress/regress-119429.js Thu Apr
5 05:32:35 2012
@@ -29,7 +29,7 @@
var d = 0;
function recurse() {
- if (++d == 26130) { // A magic number just below stack overflow on ia32
+ if (++d == 25515) { // A magic number just below stack overflow on ia32
%DebugBreak();
}
recurse();
--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev
