[heap] Drop filler objects on marking deque.

[heap] Drop filler objects on marking deque.

BUG=

Committed: https://crrev.com/9005b5845a0ce13bf6f8ffdf0fdac35fe62a2269
Cr-Commit-Position: refs/heads/master@{#38220}

[Hannes Payer (out of office), 2016-08-01 16:40:05]

The CQ bit was checked by hpayer@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-08-01 16:40:12]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[Hannes Payer (out of office), 2016-08-01 16:43:54]

hpayer@chromium.org changed reviewers:
+ ulan@chromium.org

[Hannes Payer (out of office), 2016-08-01 16:43:54]

https://codereview.chromium.org/2202633002/diff/1/src/heap/incremental-markin...
File src/heap/incremental-marking.cc (right):

https://codereview.chromium.org/2202633002/diff/1/src/heap/incremental-markin...
src/heap/incremental-marking.cc:902: if (obj->IsFiller()) {
If you think that checking the three free different filler maps here is faster,
I am happy to change it to that. Overall, I think it is cleaner to call the
abstract method on the instance type, which is a bottleneck for the filler
types.

https://codereview.chromium.org/2202633002/diff/1/src/heap/incremental-markin...
src/heap/incremental-marking.cc:934: ProcessMarkingDeque(0, FORCE_COMPLETION);
I don't think the accounting overhead matters here. The TODO from above would
avoid the incremental marking visiting overhead completely.

[ulan, 2016-08-01 16:49:43]

https://codereview.chromium.org/2202633002/diff/1/src/heap/incremental-markin...
File src/heap/incremental-marking.cc (right):

https://codereview.chromium.org/2202633002/diff/1/src/heap/incremental-markin...
src/heap/incremental-marking.cc:897: (bytes_processed < bytes_to_process ||
completion)) {
I think it should be completion == FORCE_COMPLETION

[Hannes Payer (out of office), 2016-08-01 16:54:06]

The CQ bit was checked by hpayer@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-08-01 16:54:11]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[ulan, 2016-08-01 16:54:53]

lgtm

[Hannes Payer (out of office), 2016-08-01 17:11:39]

https://codereview.chromium.org/2202633002/diff/1/src/heap/incremental-markin...
File src/heap/incremental-marking.cc (right):

https://codereview.chromium.org/2202633002/diff/1/src/heap/incremental-markin...
src/heap/incremental-marking.cc:897: (bytes_processed < bytes_to_process ||
completion)) {
On 2016/08/01 16:49:43, ulan wrote:
> I think it should be completion == FORCE_COMPLETION

Yes, thanks.

[commit-bot: I haz the power, 2016-08-01 17:29:03]

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

[commit-bot: I haz the power, 2016-08-01 17:29:04]

Dry run: Try jobs failed on following builders:
  v8_linux_arm_rel_ng on master.tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux_arm_rel_ng/builds/...)
  v8_linux_arm_rel_ng_triggered on master.tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux_arm_rel_ng_trigger...)

[Hannes Payer (out of office), 2016-08-01 19:03:46]

The CQ bit was checked by hpayer@chromium.org

[Hannes Payer (out of office), 2016-08-01 19:03:46]

The patchset sent to the CQ was uploaded after l-g-t-m from ulan@chromium.org
Link to the patchset: https://codereview.chromium.org/2202633002/#ps40001
(title: "change dcheck order")

[commit-bot: I haz the power, 2016-08-01 19:03:53]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-08-01 19:28:04]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-08-01 19:29:28]

Description was changed from

==========
[heap] Drop filler objects on marking deque.

BUG=
==========

to

==========
[heap] Drop filler objects on marking deque.

BUG=

Committed: https://crrev.com/9005b5845a0ce13bf6f8ffdf0fdac35fe62a2269
Cr-Commit-Position: refs/heads/master@{#38220}
==========

[commit-bot: I haz the power, 2016-08-01 19:29:29]

Patchset 3 (id:??) landed as
https://crrev.com/9005b5845a0ce13bf6f8ffdf0fdac35fe62a2269
Cr-Commit-Position: refs/heads/master@{#38220}