Fix stack check wraparound problem for ARM simulator.

Fix stack check wraparound problem for ARM simulator.

Committed: http://code.google.com/p/v8/source/detail?r=385

[iposva, 2008-09-17 13:18:02]

Recording off-line discussion... -ip

http://codereview.chromium.org/2926/diff/1/2
File src/builtins-ia32.cc (right):

http://codereview.chromium.org/2926/diff/1/2#newcode532
Line 532: __ j(greater, &okay, taken);
This should be a signed compare in case the stack already went past the
stack_guard by pushing the internal frame above. A negative amount of space is
not going to go over well. (Same problem on ARM).

Also we need to watch out for the stack_guard_limit not containing a stack
pointer but the preemption limit.

[Søren Thygesen Gjesse, 2008-09-26 10:55:19]

LGTM

http://codereview.chromium.org/2926/diff/4/204
File src/builtins-ia32.cc (right):

http://codereview.chromium.org/2926/diff/4/204#newcode527
Line 527: __ j(above, &no_preemption, taken);
void Ia32CodeGenerator::CheckStack() uses above_equal.