LGTM

http://codereview.chromium.org/3134027/diff/1/9
File src/objects-debug.cc (right):

http://codereview.chromium.org/3134027/diff/1/9#newcode654
src/objects-debug.cc:654: ASSERT(instance_descriptors() ==
Heap::empty_descriptor_array());
minor cosmetic: maybe ASSERT_EQ is better.

http://codereview.chromium.org/3134027/diff/1/11
File src/objects.cc (right):

http://codereview.chromium.org/3134027/diff/1/11#newcode2102
src/objects.cc:2102: // Caching for global objects is not worth it
(there too few of them).
minor: there <are> too few of them

http://codereview.chromium.org/3134027/show

--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev

Reply via email to