Revision: 24131
Author: [email protected]
Date: Tue Sep 23 07:45:52 2014 UTC
Log: Version 3.28.71.11 (merged r24079)
ARM: Make stack limit stricter to account for large buffers in
MacroAssembler.
BUG=chromium:405338
LOG=Y
[email protected]
Review URL: https://codereview.chromium.org/594033003
https://code.google.com/p/v8/source/detail?r=24131
Modified:
/branches/3.28/src/flag-definitions.h
/branches/3.28/src/globals.h
/branches/3.28/src/version.cc
=======================================
--- /branches/3.28/src/flag-definitions.h Fri Aug 8 15:46:17 2014 UTC
+++ /branches/3.28/src/flag-definitions.h Tue Sep 23 07:45:52 2014 UTC
@@ -460,9 +460,7 @@
DEFINE_BOOL(hard_abort, true, "abort by crashing")
// execution.cc
-// Slightly less than 1MB, since Windows' default stack size for
-// the main execution thread is 1MB for both 32 and 64-bit.
-DEFINE_INT(stack_size, 984,
+DEFINE_INT(stack_size, V8_DEFAULT_STACK_SIZE_KB,
"default size of stack region v8 is allowed to use (in kBytes)")
// frames.cc
=======================================
--- /branches/3.28/src/globals.h Thu Aug 14 08:46:03 2014 UTC
+++ /branches/3.28/src/globals.h Tue Sep 23 07:45:52 2014 UTC
@@ -69,6 +69,18 @@
// Determine whether the architecture uses an out-of-line constant pool.
#define V8_OOL_CONSTANT_POOL 0
+#ifdef V8_TARGET_ARCH_ARM
+// Set stack limit lower for ARM than for other architectures because
+// stack allocating MacroAssembler takes 120K bytes.
+// See issue crbug.com/405338
+#define V8_DEFAULT_STACK_SIZE_KB 864
+#else
+// Slightly less than 1MB, since Windows' default stack size for
+// the main execution thread is 1MB for both 32 and 64-bit.
+#define V8_DEFAULT_STACK_SIZE_KB 984
+#endif
+
+
// Support for alternative bool type. This is only enabled if the code is
// compiled with USE_MYBOOL defined. This catches some nasty type bugs.
// For instance, 'bool b = "false";' results in b == true! This is a hidden
=======================================
--- /branches/3.28/src/version.cc Thu Sep 18 12:11:48 2014 UTC
+++ /branches/3.28/src/version.cc Tue Sep 23 07:45:52 2014 UTC
@@ -35,7 +35,7 @@
#define MAJOR_VERSION 3
#define MINOR_VERSION 28
#define BUILD_NUMBER 71
-#define PATCH_LEVEL 10
+#define PATCH_LEVEL 11
// Use 1 for candidates and 0 otherwise.
// (Boolean macro values are not supported by all preprocessors.)
#define IS_CANDIDATE_VERSION 0
--
--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev
---
You received this message because you are subscribed to the Google Groups "v8-dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to [email protected].
For more options, visit https://groups.google.com/d/optout.
