http://codereview.chromium.org/8256015/diff/2002/src/arm/full-codegen-arm.cc File src/arm/full-codegen-arm.cc (right):
http://codereview.chromium.org/8256015/diff/2002/src/arm/full-codegen-arm.cc#newcode1023 src/arm/full-codegen-arm.cc:1023: __ mov(r1, Operand(Smi::FromInt(1))); // smi indicates slow check On 2011/10/20 14:25:31, Rico wrote:
Capitalize start of comment + below
Done. http://codereview.chromium.org/8256015/diff/2002/src/arm/full-codegen-arm.cc#newcode1030 src/arm/full-codegen-arm.cc:1030: __ Push(r1, r0); // map or smi, and array On 2011/10/20 14:25:31, Rico wrote:
how can r1 be a map, we either load smi 1 or smi 0 above right?
Done. http://codereview.chromium.org/8256015/diff/2002/src/arm/full-codegen-arm.cc#newcode1060 src/arm/full-codegen-arm.cc:1060: // TODO(rossberg): What if only a prototype is a proxy? Not specified. On 2011/10/20 14:25:31, Rico wrote:
Please file a bug
Will note it in bug v8:1543. http://codereview.chromium.org/8256015/diff/2002/src/ia32/full-codegen-ia32.cc File src/ia32/full-codegen-ia32.cc (right): http://codereview.chromium.org/8256015/diff/2002/src/ia32/full-codegen-ia32.cc#newcode1014 src/ia32/full-codegen-ia32.cc:1014: __ push(eax); On 2011/10/20 14:25:31, Rico wrote:
Add comment about what eax is
Done. http://codereview.chromium.org/8256015/diff/2002/src/v8natives.js File src/v8natives.js (right): http://codereview.chromium.org/8256015/diff/2002/src/v8natives.js#newcode1013 src/v8natives.js:1013: global.print("GetOwnEnumerablePropertyNames"); On 2011/10/20 14:25:31, Rico wrote:
Debug print?
Oops. http://codereview.chromium.org/8256015/diff/2002/src/x64/full-codegen-x64.cc File src/x64/full-codegen-x64.cc (right): http://codereview.chromium.org/8256015/diff/2002/src/x64/full-codegen-x64.cc#newcode989 src/x64/full-codegen-x64.cc:989: __ push(rax); On 2011/10/20 14:25:31, Rico wrote:
Add comment about what rax is
Done. http://codereview.chromium.org/8256015/diff/2002/test/mjsunit/harmony/proxies-for.js File test/mjsunit/harmony/proxies-for.js (right): http://codereview.chromium.org/8256015/diff/2002/test/mjsunit/harmony/proxies-for.js#newcode28 test/mjsunit/harmony/proxies-for.js:28: // Flags: --harmony-proxies On 2011/10/20 14:25:31, Rico wrote:
General in this file: Add semicolons
To avoid conflicts, will defer until my open CLs are all in. http://codereview.chromium.org/8256015/ -- v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev
