Do not follow accessor map transitions when marking descriptor arrays.

Do not follow accessor map transitions when marking descriptor arrays.

Note that we currently have no such transitions, but we will in the future, and
we have to avoid keeping maps live too long.

Committed: https://code.google.com/p/v8/source/detail?r=10495

[Michael Starzinger, 2012-01-24 13:28:29]

https://chromiumcodereview.appspot.com/9212045/diff/1/src/mark-compact.cc
File src/mark-compact.cc (right):

https://chromiumcodereview.appspot.com/9212045/diff/1/src/mark-compact.cc#new...
src/mark-compact.cc:1675: bool MarkCompactCollector::MarkUnmarked(Object* value)
{
The name is kind of misleading, because it not only handles unmarked objects,
but marked ones as well.

My suggestion is rename this to use a different signature and also to move it to
mark-compact-inl.h. Proposed new signature is:

void MarkCompactCollector::MarkObject(HeapObject* obj);

https://chromiumcodereview.appspot.com/9212045/diff/1/src/mark-compact.cc#new...
src/mark-compact.cc:1684: bool MarkCompactCollector::MarkUnmarkedAndPush(Object*
value) {
Boolean return value is never used, can we drop it?

Can we also move this to mark-compact-inl.h and rename it to
MarkObjectAndPush()?

[Sven Panne, 2012-01-24 16:08:11]

Incorporated online and offline comments/suggestions. PTAL...

[Michael Starzinger, 2012-01-25 08:45:48]

LGTM.