Addressed comments
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) { On 2012/09/12 16:31:03, Jakob wrote:
nit: can use target_descriptors here
Done. 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); On 2012/09/12 16:31:03, Jakob wrote:
nit: please swap this line and the empty line below
Done. https://chromiumcodereview.appspot.com/10909007/ -- v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev
