http://codereview.chromium.org/8230009/diff/1/src/incremental-marking.cc File src/incremental-marking.cc (right):
http://codereview.chromium.org/8230009/diff/1/src/incremental-marking.cc#newcode523 src/incremental-marking.cc:523: (obj->IsFreeSpace() && Marking::IsWhite(mark_bit))); Does IsFreeSpace() cover two-pointer-filler maps? I think we should also check for IsFiller() here. http://codereview.chromium.org/8230009/diff/1/src/incremental-marking.cc#newcode535 src/incremental-marking.cc:535: (obj->IsFreeSpace() && Marking::IsWhite(mark_bit))); Same as above. http://codereview.chromium.org/8230009/diff/1/src/incremental-marking.cc#newcode744 src/incremental-marking.cc:744: (obj->IsFreeSpace() && Marking::IsWhite(obj_mark_bit))); Same as above. http://codereview.chromium.org/8230009/ -- v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev
