I have tested this via the testing of the (currently unfinished) OOL
constant
pool CL I have. In that CL I change the frame size, and these are all the
places I had to change the hard-coded constants to get it to pass all the V8
tests. It's entirely possible that I've missed some others though, but I'm
not
sure how else to find them
https://codereview.chromium.org/60763006/diff/30001/src/arm/builtins-arm.cc
File src/arm/builtins-arm.cc (right):
https://codereview.chromium.org/60763006/diff/30001/src/arm/builtins-arm.cc#newcode1346
src/arm/builtins-arm.cc:1346: __ add(fp, sp, Operand(3 * kPointerSize));
On 2013/11/20 09:31:37, ulan wrote:
Did you mean to remove this line?
Yes, good catch! Merge error issue. Done.
https://codereview.chromium.org/60763006/
--
--
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/groups/opt_out.
