http://codereview.chromium.org/1164002/diff/1/16
File src/arm/virtual-frame-arm.cc (right):

http://codereview.chromium.org/1164002/diff/1/16#newcode51
src/arm/virtual-frame-arm.cc:51: // All elements are in memory on ARM
(ie, synced).
On 2010/03/22 13:12:35, Kasper Lund wrote:
I guess this is called on ARM. Do you want to try to inline it?

Not called.

Removed.

http://codereview.chromium.org/1164002/diff/1/7
File src/jump-target-heavy.cc (right):

http://codereview.chromium.org/1164002/diff/1/7#newcode46
src/jump-target-heavy.cc:46: #ifdef DEBUG
On 2010/03/22 13:12:35, Kasper Lund wrote:
Maybe move these nasty macros above the first method?

They were only used in 2 places so I got rid of them instead.

http://codereview.chromium.org/1164002

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

To unsubscribe from this group, send email to v8-dev+unsubscribegooglegroups.com or reply 
to this email with the words "REMOVE ME" as the subject.

Reply via email to