LGTM!

http://codereview.chromium.org/9173001/diff/11001/src/mark-compact.cc
File src/mark-compact.cc (right):

http://codereview.chromium.org/9173001/diff/11001/src/mark-compact.cc#newcode3644
src/mark-compact.cc:3644: space->MarkPageForLazySweep(p);
I would call it MarkPageForLazySweeping

http://codereview.chromium.org/9173001/diff/11001/src/spaces.h
File src/spaces.h (right):

http://codereview.chromium.org/9173001/diff/11001/src/spaces.h#newcode1663
src/spaces.h:1663: intptr_t unswept_dead_bytes_;
I would call them free not dead (I think object can be dead but not the
byte).

http://codereview.chromium.org/9173001/

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

Reply via email to