[heap] Pause black allocation during GCs

[heap] Pause black allocation during GCs

There is no point in doing black allocation here as we then have to
iterate the objects for various reasons. The marker does the same work
but can be moved outside of the atomic pause.

BUG=chromium:581412
Review-Url: https://codereview.chromium.org/2862563002
Cr-Commit-Position: refs/heads/master@{#45063}
Committed: https://chromium.googlesource.com/v8/v8/+/17a0a575a3795141dfccabc688b3fa39f226e186

[Michael Lippautz, 2017-05-03 12:35:43]

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

[Michael Lippautz, 2017-05-03 12:35:44]

ptal

[Michael Lippautz, 2017-05-03 12:35:47]

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

[commit-bot: I haz the power, 2017-05-03 12:35:49]

Dry run: CQ is trying da patch.

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

[payer, 2017-05-03 12:53:53]

hpayer@google.com changed reviewers:
+ hpayer@google.com

[payer, 2017-05-03 12:53:54]

lgtm

[Hannes Payer (out of office), 2017-05-03 13:02:05]

lgtm

[Hannes Payer (out of office), 2017-05-03 13:02:39]

Description was changed from

==========
[heap] Pause black allocation during GCs

There is no point in doing black allocation here as we then have to
iterate the objects for various reasons. The marker does the same work
but can be moved outside of the atomic pause.

BUG=chromium:581412
==========

to

==========
[heap] Pause black allocation during GCs

There is no point in doing black allocation here as we then have to
iterate the objects for various reasons. The marker does the same work
but can be moved outside of the atomic pause.

BUG=chromium:581412
==========

[Hannes Payer (out of office), 2017-05-03 13:02:39]

hpayer@chromium.org changed reviewers:
- hpayer@google.com

[commit-bot: I haz the power, 2017-05-03 13:06:26]

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

[commit-bot: I haz the power, 2017-05-03 13:06:27]

Dry run: This issue passed the CQ dry run.

[Michael Lippautz, 2017-05-03 13:57:01]

The CQ bit was checked by mlippautz@chromium.org

[commit-bot: I haz the power, 2017-05-03 13:57:04]

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, 2017-05-03 13:58:22]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1493819821390010, "parent_rev":
"c3fda2b8037e90203467f19ff324179a36595f10", "commit_rev":
"17a0a575a3795141dfccabc688b3fa39f226e186"}

[commit-bot: I haz the power, 2017-05-03 13:58:29]

Description was changed from

==========
[heap] Pause black allocation during GCs

There is no point in doing black allocation here as we then have to
iterate the objects for various reasons. The marker does the same work
but can be moved outside of the atomic pause.

BUG=chromium:581412
==========

to

==========
[heap] Pause black allocation during GCs

There is no point in doing black allocation here as we then have to
iterate the objects for various reasons. The marker does the same work
but can be moved outside of the atomic pause.

BUG=chromium:581412

Review-Url: https://codereview.chromium.org/2862563002
Cr-Commit-Position: refs/heads/master@{#45063}
Committed:
https://chromium.googlesource.com/v8/v8/+/17a0a575a3795141dfccabc688b3fa39f22...
==========

[commit-bot: I haz the power, 2017-05-03 13:58:30]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/v8/v8/+/17a0a575a3795141dfccabc688b3fa39f22...