Use just one marking deque.

Use just one marking deque.

BUG=

[jochen (gone - plz use gerrit), 2014-12-01 14:32:03]

jochen@chromium.org changed reviewers:
+ jochen@chromium.org

[jochen (gone - plz use gerrit), 2014-12-01 14:32:04]

is it possible to avoid marking everybody friend of everybody else? I think
exposing the required methods in the public API is preferable over friends,
because the latter makes it to easy to mess around with each other's private
bits.

https://codereview.chromium.org/770453003/diff/100001/src/heap/mark-compact.cc
File src/heap/mark-compact.cc (right):

https://codereview.chromium.org/770453003/diff/100001/src/heap/mark-compact.c...
src/heap/mark-compact.cc:2033:
DCHECK(!Marking::IsWhite(Marking::MarkBitFrom(object)));
why was this inverted?

[Hannes Payer (out of office), 2014-12-01 14:53:24]

I unfriended them.

https://codereview.chromium.org/770453003/diff/100001/src/heap/mark-compact.cc
File src/heap/mark-compact.cc (right):

https://codereview.chromium.org/770453003/diff/100001/src/heap/mark-compact.c...
src/heap/mark-compact.cc:2033:
DCHECK(!Marking::IsWhite(Marking::MarkBitFrom(object)));
On 2014/12/01 14:32:04, jochen (slow) wrote:
> why was this inverted?

Because it can be grey or black.

[jochen (gone - plz use gerrit), 2014-12-01 14:55:53]

lgtm

[Hannes Payer (out of office), 2014-12-01 15:01:48]

The CQ bit was checked by hpayer@chromium.org

[commit-bot: I haz the power, 2014-12-01 15:02:52]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/770453003/130001

[commit-bot: I haz the power, 2014-12-01 15:59:38]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2014-12-01 15:59:39]

Try jobs failed on following builders:
  v8_linux_dbg on tryserver.v8
(http://build.chromium.org/p/tryserver.v8/builders/v8_linux_dbg/builds/1527)

[Hannes Payer (out of office), 2014-12-01 16:34:32]

The CQ bit was checked by hpayer@chromium.org

[commit-bot: I haz the power, 2014-12-01 16:35:28]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/770453003/40002

[commit-bot: I haz the power, 2014-12-01 17:02:59]

Committed patchset #9 (id:40002)