Revert "This is a speculative chain of reverts to improve a Chrome"

Reland moving Atomics builtins to C++

This reverts the previous revert, commit
5a04f4fd68d1d35d704cdc0dee0719c5354a8094.

Previously reverted changes:

>    Revert "[SAB] Move Atomics builtins to C++"
>
>     This reverts commit 2b9840d86f3b51f42918b6bf1a06ff8b62b2464d.
>
>     Revert "[SAB] Remove unreachable Uint8Clamped atomics paths"
>
>     This reverts commit d1160fb14f767f230f4d276b4bef49f846bb7a94.
>
>     Revert "Remove tiny unit test for MinSimple/MaxSimple"
>
>     This reverts commit 837760ecb7ff2ccaf7921ab46e7b473b313d4a4d.
>
>     Revert "Remove infrastructure for experimental JS natives"
>
>     This reverts commit 8cfe45b6f1880008d09d4310458b6e6fcb0522dd.

These changes were reverted to improve a perf regression on a Chrome
bot. Since then, the regression has reappeared, then disappeared again
all from seemingly unrelated changes.

BUG=v8:6033
TBR=adamk@chromium.org,hpayer@chromium.org,yangguo@chromium.org

Review-Url: https://codereview.chromium.org/2732213005
Cr-Commit-Position: refs/heads/master@{#43758}
Committed: https://chromium.googlesource.com/v8/v8/+/65200967b735be0cd91593ffb03fb96ee90df1a0

[binji, 2017-03-08 02:01:00]

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

[commit-bot: I haz the power, 2017-03-08 02:01:08]

Dry run: 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-03-08 02:26:21]

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

[commit-bot: I haz the power, 2017-03-08 02:26:21]

Dry run: This issue passed the CQ dry run.

[binji, 2017-03-08 02:43:46]

binji@chromium.org changed reviewers:
+ adamk@chromium.org, littledan@chromium.org

[adamk, 2017-03-08 06:03:29]

Thanks for putting this together!  But:

1. We should wait till I+TF makes it to Canary to land this (we're trying not to
land anything big before then, to make sure rolls are smooth)

and

2. More CL description please

[binji, 2017-03-10 19:19:54]

Description was changed from

==========
Revert "This is a speculative chain of reverts to improve a Chrome"

This reverts commit 5a04f4fd68d1d35d704cdc0dee0719c5354a8094.

BUG=v8:6033
==========

to

==========
Reland moving Atomics builtins to C++

This reverts the previous revert, commit
5a04f4fd68d1d35d704cdc0dee0719c5354a8094.

Previously reverted changes:

>    Revert "[SAB] Move Atomics builtins to C++"
>
>     This reverts commit 2b9840d86f3b51f42918b6bf1a06ff8b62b2464d.
>
>     Revert "[SAB] Remove unreachable Uint8Clamped atomics paths"
>
>     This reverts commit d1160fb14f767f230f4d276b4bef49f846bb7a94.
>
>     Revert "Remove tiny unit test for MinSimple/MaxSimple"
>
>     This reverts commit 837760ecb7ff2ccaf7921ab46e7b473b313d4a4d.
>
>     Revert "Remove infrastructure for experimental JS natives"

These changes were reverted to improve a perf regression on a Chrome
bot. Since then, the regression has reappeared, then disappeared again
all from seemingly unrelated changes.

BUG=v8:6033
==========

[binji, 2017-03-10 19:21:21]

OK, how's this for a better description?

[adamk, 2017-03-10 19:22:37]

Looks good...let's aim to land this on Monday?

[binji, 2017-03-10 19:28:10]

On 2017/03/10 19:22:37, adamk wrote:
> Looks good...let's aim to land this on Monday?

sgtm

[adamk, 2017-03-10 19:30:47]

lgtm so this can land once the tree is un-throttled

[binji, 2017-03-10 19:32:10]

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

[commit-bot: I haz the power, 2017-03-10 19:32:16]

Dry run: 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-03-10 19:55:58]

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

[commit-bot: I haz the power, 2017-03-10 19:55:59]

Dry run: This issue passed the CQ dry run.

[binji, 2017-03-13 18:48:02]

The CQ bit was checked by binji@chromium.org

[binji, 2017-03-13 18:48:03]

The patchset sent to the CQ was uploaded after l-g-t-m from adamk@chromium.org
Link to the patchset: https://codereview.chromium.org/2732213005/#ps60001
(title: "merge")

[commit-bot: I haz the power, 2017-03-13 18:48:14]

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-03-13 18:52:24]

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

[commit-bot: I haz the power, 2017-03-13 18:52:25]

Try jobs failed on following builders:
  v8_presubmit on master.tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_presubmit/builds/36502)

[binji, 2017-03-13 19:08:41]

Description was changed from

==========
Reland moving Atomics builtins to C++

This reverts the previous revert, commit
5a04f4fd68d1d35d704cdc0dee0719c5354a8094.

Previously reverted changes:

>    Revert "[SAB] Move Atomics builtins to C++"
>
>     This reverts commit 2b9840d86f3b51f42918b6bf1a06ff8b62b2464d.
>
>     Revert "[SAB] Remove unreachable Uint8Clamped atomics paths"
>
>     This reverts commit d1160fb14f767f230f4d276b4bef49f846bb7a94.
>
>     Revert "Remove tiny unit test for MinSimple/MaxSimple"
>
>     This reverts commit 837760ecb7ff2ccaf7921ab46e7b473b313d4a4d.
>
>     Revert "Remove infrastructure for experimental JS natives"

These changes were reverted to improve a perf regression on a Chrome
bot. Since then, the regression has reappeared, then disappeared again
all from seemingly unrelated changes.

BUG=v8:6033
==========

to

==========
Reland moving Atomics builtins to C++

This reverts the previous revert, commit
5a04f4fd68d1d35d704cdc0dee0719c5354a8094.

Previously reverted changes:

>    Revert "[SAB] Move Atomics builtins to C++"
>
>     This reverts commit 2b9840d86f3b51f42918b6bf1a06ff8b62b2464d.
>
>     Revert "[SAB] Remove unreachable Uint8Clamped atomics paths"
>
>     This reverts commit d1160fb14f767f230f4d276b4bef49f846bb7a94.
>
>     Revert "Remove tiny unit test for MinSimple/MaxSimple"
>
>     This reverts commit 837760ecb7ff2ccaf7921ab46e7b473b313d4a4d.
>
>     Revert "Remove infrastructure for experimental JS natives"
>
>     This reverts commit 8cfe45b6f1880008d09d4310458b6e6fcb0522dd.

These changes were reverted to improve a perf regression on a Chrome
bot. Since then, the regression has reappeared, then disappeared again
all from seemingly unrelated changes.

BUG=v8:6033
==========

[binji, 2017-03-13 19:10:13]

Description was changed from

==========
Reland moving Atomics builtins to C++

This reverts the previous revert, commit
5a04f4fd68d1d35d704cdc0dee0719c5354a8094.

Previously reverted changes:

>    Revert "[SAB] Move Atomics builtins to C++"
>
>     This reverts commit 2b9840d86f3b51f42918b6bf1a06ff8b62b2464d.
>
>     Revert "[SAB] Remove unreachable Uint8Clamped atomics paths"
>
>     This reverts commit d1160fb14f767f230f4d276b4bef49f846bb7a94.
>
>     Revert "Remove tiny unit test for MinSimple/MaxSimple"
>
>     This reverts commit 837760ecb7ff2ccaf7921ab46e7b473b313d4a4d.
>
>     Revert "Remove infrastructure for experimental JS natives"
>
>     This reverts commit 8cfe45b6f1880008d09d4310458b6e6fcb0522dd.

These changes were reverted to improve a perf regression on a Chrome
bot. Since then, the regression has reappeared, then disappeared again
all from seemingly unrelated changes.

BUG=v8:6033
==========

to

==========
Reland moving Atomics builtins to C++

This reverts the previous revert, commit
5a04f4fd68d1d35d704cdc0dee0719c5354a8094.

Previously reverted changes:

>    Revert "[SAB] Move Atomics builtins to C++"
>
>     This reverts commit 2b9840d86f3b51f42918b6bf1a06ff8b62b2464d.
>
>     Revert "[SAB] Remove unreachable Uint8Clamped atomics paths"
>
>     This reverts commit d1160fb14f767f230f4d276b4bef49f846bb7a94.
>
>     Revert "Remove tiny unit test for MinSimple/MaxSimple"
>
>     This reverts commit 837760ecb7ff2ccaf7921ab46e7b473b313d4a4d.
>
>     Revert "Remove infrastructure for experimental JS natives"
>
>     This reverts commit 8cfe45b6f1880008d09d4310458b6e6fcb0522dd.

These changes were reverted to improve a perf regression on a Chrome
bot. Since then, the regression has reappeared, then disappeared again
all from seemingly unrelated changes.

BUG=v8:6033
TBR=adamk@chromium.org,hpayer@chromium.org,yangguo@chromium.org
==========

[binji, 2017-03-13 20:57:34]

The CQ bit was checked by binji@chromium.org

[commit-bot: I haz the power, 2017-03-13 20:57:37]

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-03-13 20:59:18]

CQ is committing da patch.

Bot data: {"patchset_id": 60001, "attempt_start_ts": 1489438654490240,
"parent_rev": "aba151b92fbd45763160c046ae75e1505c1ca4cf", "commit_rev":
"65200967b735be0cd91593ffb03fb96ee90df1a0"}

[commit-bot: I haz the power, 2017-03-13 20:59:24]

Description was changed from

==========
Reland moving Atomics builtins to C++

This reverts the previous revert, commit
5a04f4fd68d1d35d704cdc0dee0719c5354a8094.

Previously reverted changes:

>    Revert "[SAB] Move Atomics builtins to C++"
>
>     This reverts commit 2b9840d86f3b51f42918b6bf1a06ff8b62b2464d.
>
>     Revert "[SAB] Remove unreachable Uint8Clamped atomics paths"
>
>     This reverts commit d1160fb14f767f230f4d276b4bef49f846bb7a94.
>
>     Revert "Remove tiny unit test for MinSimple/MaxSimple"
>
>     This reverts commit 837760ecb7ff2ccaf7921ab46e7b473b313d4a4d.
>
>     Revert "Remove infrastructure for experimental JS natives"
>
>     This reverts commit 8cfe45b6f1880008d09d4310458b6e6fcb0522dd.

These changes were reverted to improve a perf regression on a Chrome
bot. Since then, the regression has reappeared, then disappeared again
all from seemingly unrelated changes.

BUG=v8:6033
TBR=adamk@chromium.org,hpayer@chromium.org,yangguo@chromium.org
==========

to

==========
Reland moving Atomics builtins to C++

This reverts the previous revert, commit
5a04f4fd68d1d35d704cdc0dee0719c5354a8094.

Previously reverted changes:

>    Revert "[SAB] Move Atomics builtins to C++"
>
>     This reverts commit 2b9840d86f3b51f42918b6bf1a06ff8b62b2464d.
>
>     Revert "[SAB] Remove unreachable Uint8Clamped atomics paths"
>
>     This reverts commit d1160fb14f767f230f4d276b4bef49f846bb7a94.
>
>     Revert "Remove tiny unit test for MinSimple/MaxSimple"
>
>     This reverts commit 837760ecb7ff2ccaf7921ab46e7b473b313d4a4d.
>
>     Revert "Remove infrastructure for experimental JS natives"
>
>     This reverts commit 8cfe45b6f1880008d09d4310458b6e6fcb0522dd.

These changes were reverted to improve a perf regression on a Chrome
bot. Since then, the regression has reappeared, then disappeared again
all from seemingly unrelated changes.

BUG=v8:6033
TBR=adamk@chromium.org,hpayer@chromium.org,yangguo@chromium.org

Review-Url: https://codereview.chromium.org/2732213005
Cr-Commit-Position: refs/heads/master@{#43758}
Committed:
https://chromium.googlesource.com/v8/v8/+/65200967b735be0cd91593ffb03fb96ee90...
==========

[commit-bot: I haz the power, 2017-03-13 20:59:26]

Committed patchset #4 (id:60001) as
https://chromium.googlesource.com/v8/v8/+/65200967b735be0cd91593ffb03fb96ee90...