Revision: 10722
Author:   [email protected]
Date:     Thu Feb 16 06:16:12 2012
Log:      Increase ARM/MIPS simulators stack safety margin to 1024 bytes

Based on r9686 (10e1ea5).

BUG=
TEST=mozilla regress-355497.js

Review URL: https://chromiumcodereview.appspot.com/9309030
Patch from Daniel Kalmar <[email protected]>.
http://code.google.com/p/v8/source/detail?r=10722

Modified:
 /branches/bleeding_edge/src/arm/simulator-arm.cc
 /branches/bleeding_edge/src/mips/simulator-mips.cc

=======================================
--- /branches/bleeding_edge/src/arm/simulator-arm.cc Fri Jan 13 05:09:52 2012 +++ /branches/bleeding_edge/src/arm/simulator-arm.cc Thu Feb 16 06:16:12 2012
@@ -1277,9 +1277,9 @@

// Returns the limit of the stack area to enable checking for stack overflows.
 uintptr_t Simulator::StackLimit() const {
- // Leave a safety margin of 512 bytes to prevent overrunning the stack when + // Leave a safety margin of 1024 bytes to prevent overrunning the stack when
   // pushing values.
-  return reinterpret_cast<uintptr_t>(stack_) + 512;
+  return reinterpret_cast<uintptr_t>(stack_) + 1024;
 }


=======================================
--- /branches/bleeding_edge/src/mips/simulator-mips.cc Mon Jan 16 04:38:59 2012 +++ /branches/bleeding_edge/src/mips/simulator-mips.cc Thu Feb 16 06:16:12 2012
@@ -1369,9 +1369,9 @@

// Returns the limit of the stack area to enable checking for stack overflows.
 uintptr_t Simulator::StackLimit() const {
- // Leave a safety margin of 512 bytes to prevent overrunning the stack when + // Leave a safety margin of 1024 bytes to prevent overrunning the stack when
   // pushing values.
-  return reinterpret_cast<uintptr_t>(stack_) + 512;
+  return reinterpret_cast<uintptr_t>(stack_) + 1024;
 }


--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev

Reply via email to