LGTM
http://codereview.chromium.org/2693002/diff/18001/19003 File src/arm/assembler-arm.h (right): http://codereview.chromium.org/2693002/diff/18001/19003#newcode646 src/arm/assembler-arm.h:646: // Return sequence is: The comments describe the return sequence. Update? http://codereview.chromium.org/2693002/diff/18001/19006 File src/arm/debug-arm.cc (right): http://codereview.chromium.org/2693002/diff/18001/19006#newcode97 src/arm/debug-arm.cc:97: // Patch the code changing the return from JS function sequence from This talks about the return sequence. The rest of the comment seems to have been updated correctly. http://codereview.chromium.org/2693002/diff/18001/19006#newcode101 src/arm/debug-arm.cc:101: // to a call to the debug break return code. return code -> code? http://codereview.chromium.org/2693002/diff/18001/19016 File src/debug.cc (right): http://codereview.chromium.org/2693002/diff/18001/19016#newcode133 src/debug.cc:133: // There is always a possible break point as a debug break slot. as -> at http://codereview.chromium.org/2693002/show -- v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev
