[turbofan] Don't check for neutered array buffers eagerly.

[turbofan] Don't check for neutered array buffers eagerly.

We don't need to check for neutered array buffers unless at least one
JSArrayBuffer has been neutered (i.e. detached in TC39 speak). For this
we introduce a protector cell that get's invalidated on first call to
the JSArrayBuffer::Neuter() method.

R=jarin@chromium.org,ulan@chromium.org
BUG=v8:5267
Committed: https://crrev.com/36e3af325aad839bef36ba3670cb09bb8cf0dc86
Cr-Commit-Position: refs/heads/master@{#41021}

[Benedikt Meurer, 2016-11-16 10:07:47]

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

[commit-bot: I haz the power, 2016-11-16 10:07:49]

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

[Jarin, 2016-11-16 10:13:43]

I am not too wild about introducing another non-local performance cliff, but if
you think this is what we need, then lgtm.

[ulan, 2016-11-16 10:22:40]

heap lgtm

[Benedikt Meurer, 2016-11-16 10:26:00]

Well, we're playing this benchmark game unfortunately, and the cliff is not too
bad (this particular one).

[Benedikt Meurer, 2016-11-16 10:26:06]

The CQ bit was unchecked by bmeurer@chromium.org

[Benedikt Meurer, 2016-11-16 10:26:10]

The CQ bit was checked by bmeurer@chromium.org

[commit-bot: I haz the power, 2016-11-16 10:26:20]

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-11-16 10:35:36]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-11-17 22:35:36]

Description was changed from

==========
[turbofan] Don't check for neutered array buffers eagerly.

We don't need to check for neutered array buffers unless at least one
JSArrayBuffer has been neutered (i.e. detached in TC39 speak). For this
we introduce a protector cell that get's invalidated on first call to
the JSArrayBuffer::Neuter() method.

R=jarin@chromium.org,ulan@chromium.org
BUG=v8:5267
==========

to

==========
[turbofan] Don't check for neutered array buffers eagerly.

We don't need to check for neutered array buffers unless at least one
JSArrayBuffer has been neutered (i.e. detached in TC39 speak). For this
we introduce a protector cell that get's invalidated on first call to
the JSArrayBuffer::Neuter() method.

R=jarin@chromium.org,ulan@chromium.org
BUG=v8:5267

Committed: https://crrev.com/36e3af325aad839bef36ba3670cb09bb8cf0dc86
Cr-Commit-Position: refs/heads/master@{#41021}
==========

[commit-bot: I haz the power, 2016-11-17 22:35:37]

Patchset 1 (id:??) landed as
https://crrev.com/36e3af325aad839bef36ba3670cb09bb8cf0dc86
Cr-Commit-Position: refs/heads/master@{#41021}