lgtm

with a better name for Mark/Unmark routines :-)


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

http://codereview.chromium.org/9241010/diff/1/src/mark-compact.cc#newcode3636
src/mark-compact.cc:3636: space->MarkPageForLazySweeping(p);
this is pretty confusing. we are both marking it for lazy sweepign and
releasing it immediately.

maybe rename MarkPageForLazySweeping into something else?

http://codereview.chromium.org/9241010/

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

Reply via email to