All changes made, committed.
http://codereview.chromium.org/53005/diff/1/2 File src/jump-target.cc (right): http://codereview.chromium.org/53005/diff/1/2#newcode275 Line 275: // Set the register counts and positions. On 2009/03/24 11:28:12, Kevin Millikin wrote: > Indices, not positions. Done. http://codereview.chromium.org/53005/diff/1/2#newcode282 Line 282: if (current.is_register()) { On 2009/03/24 11:28:12, Kevin Millikin wrote: > else if? Done. http://codereview.chromium.org/53005/diff/1/6 File src/virtual-frame-arm.h (right): http://codereview.chromium.org/53005/diff/1/6#newcode381 Line 381: // of updating a pointer to the register's location in the frame. On 2009/03/24 11:28:12, Kevin Millikin wrote: > "The index of" not "a pointer to". Done. http://codereview.chromium.org/53005/diff/1/6#newcode386 Line 386: // pointer to the register's in the frame. On 2009/03/24 11:28:12, Kevin Millikin wrote: > "Index", not "pointer". To the register's what? Done. http://codereview.chromium.org/53005/diff/1/4 File src/virtual-frame-ia32.h (right): http://codereview.chromium.org/53005/diff/1/4#newcode342 Line 342: int frame_location_[kNumRegisters]; On 2009/03/24 11:28:12, Kevin Millikin wrote: > frame_location_ is not a good name for the registers' indices in the frame. > "register_indices_"? "register_locations_"? Done. http://codereview.chromium.org/53005/diff/1/4#newcode380 Line 380: // of updating a pointer to the register's location in the frame. On 2009/03/24 11:28:12, Kevin Millikin wrote: > Index, not pointer. Done. http://codereview.chromium.org/53005/diff/1/4#newcode385 Line 385: // pointer to the register's in the frame. On 2009/03/24 11:28:12, Kevin Millikin wrote: > Index, not pointer. The register's what? Done. http://codereview.chromium.org/53005 --~--~---------~--~----~------------~-------~--~----~ v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev -~----------~----~----~----~------~----~------~--~---
