LGTM with nits.
https://chromiumcodereview.appspot.com/10909007/diff/18002/src/objects.cc File src/objects.cc (right): https://chromiumcodereview.appspot.com/10909007/diff/18002/src/objects.cc#newcode7443 src/objects.cc:7443: target->instance_descriptors() == descriptors) { nit: can use target_descriptors here https://chromiumcodereview.appspot.com/10909007/diff/18002/src/x64/macro-assembler-x64.cc File src/x64/macro-assembler-x64.cc (right): https://chromiumcodereview.appspot.com/10909007/diff/18002/src/x64/macro-assembler-x64.cc#newcode2931 src/x64/macro-assembler-x64.cc:2931: bind(&fail); nit: please swap this line and the empty line below https://chromiumcodereview.appspot.com/10909007/ -- v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev
