[Atomics] Remove Atomics code stubs; use TF ops

[Atomics] Remove Atomics code stubs; use TF ops

This is a much cleaner solution, which won't require nearly as much
architecture-specific code. Thanks bmeurer@!

BUG=v8:4614
LOG=y
R=bmeurer@chromium.org,jarin@chromium.org

Committed: https://crrev.com/6ff5881b1def45b35384572f61327e42563a89c3
Cr-Commit-Position: refs/heads/master@{#35547}

[binji, 2016-04-14 22:15:47]

Description was changed from

==========
[Atomics] Remove Atomics code stubs; use TF ops

This is a much cleaner solution, which won't require nearly as much
architecture-specific code. Thanks bmeurer@!

BUG=v8:4614
LOG=y
R=bmeurer@chromium.org
==========

to

==========
[Atomics] Remove Atomics code stubs; use TF ops

This is a much cleaner solution, which won't require nearly as much
architecture-specific code. Thanks bmeurer@!

BUG=v8:4614
LOG=y
R=bmeurer@chromium.org,jarin@chromium.org
==========

[binji, 2016-04-14 22:16:49]

binji@chromium.org changed reviewers:
+ jarin@chromium.org

[binji, 2016-04-14 22:16:50]

PTAL

[Benedikt Meurer, 2016-04-15 06:59:13]

Woohoo, awesome! LGTM.

[Rodolph Perfetta, 2016-04-15 14:36:03]

rodolph.perfetta@gmail.com changed reviewers:
+ rodolph.perfetta@gmail.com

[Rodolph Perfetta, 2016-04-15 14:36:04]

DBC

https://codereview.chromium.org/1891033002/diff/40001/src/compiler/arm64/code...
File src/compiler/arm64/code-generator-arm64.cc (right):

https://codereview.chromium.org/1891033002/diff/40001/src/compiler/arm64/code...
src/compiler/arm64/code-generator-arm64.cc:1427:
ASSEMBLE_ATOMIC_LOAD_INTEGER(Ldr);
For Ldrb/sb/h/sh the amount of data loaded is given by the mnemonic, for Ldr it
will depend on the size of the target register and here you are using a 64-bit
one not 32-bit.

[binji, 2016-04-15 21:00:13]

https://codereview.chromium.org/1891033002/diff/40001/src/compiler/arm64/code...
File src/compiler/arm64/code-generator-arm64.cc (right):

https://codereview.chromium.org/1891033002/diff/40001/src/compiler/arm64/code...
src/compiler/arm64/code-generator-arm64.cc:1427:
ASSEMBLE_ATOMIC_LOAD_INTEGER(Ldr);
On 2016/04/15 14:36:03, Rodolph Perfetta wrote:
> For Ldrb/sb/h/sh the amount of data loaded is given by the mnemonic, for Ldr
it
> will depend on the size of the target register and here you are using a 64-bit
> one not 32-bit.

Oops, right. Done

[binji, 2016-04-15 21:05:32]

FYI: mbrandy@, jyan@. sorry for the churn

[binji, 2016-04-15 21:05:40]

The CQ bit was checked by binji@chromium.org

[binji, 2016-04-15 21:05:41]

The patchset sent to the CQ was uploaded after l-g-t-m from bmeurer@chromium.org
Link to the patchset: https://codereview.chromium.org/1891033002/#ps100001
(title: "remove s390 code stub")

[commit-bot: I haz the power, 2016-04-15 21:05:52]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1891033002/100001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1891033002/100001

[commit-bot: I haz the power, 2016-04-16 06:47:43]

Description was changed from

==========
[Atomics] Remove Atomics code stubs; use TF ops

This is a much cleaner solution, which won't require nearly as much
architecture-specific code. Thanks bmeurer@!

BUG=v8:4614
LOG=y
R=bmeurer@chromium.org,jarin@chromium.org
==========

to

==========
[Atomics] Remove Atomics code stubs; use TF ops

This is a much cleaner solution, which won't require nearly as much
architecture-specific code. Thanks bmeurer@!

BUG=v8:4614
LOG=y
R=bmeurer@chromium.org,jarin@chromium.org
==========

[commit-bot: I haz the power, 2016-04-16 06:47:44]

Committed patchset #6 (id:100001)

[commit-bot: I haz the power, 2016-04-16 06:49:13]

Description was changed from

==========
[Atomics] Remove Atomics code stubs; use TF ops

This is a much cleaner solution, which won't require nearly as much
architecture-specific code. Thanks bmeurer@!

BUG=v8:4614
LOG=y
R=bmeurer@chromium.org,jarin@chromium.org
==========

to

==========
[Atomics] Remove Atomics code stubs; use TF ops

This is a much cleaner solution, which won't require nearly as much
architecture-specific code. Thanks bmeurer@!

BUG=v8:4614
LOG=y
R=bmeurer@chromium.org,jarin@chromium.org

Committed: https://crrev.com/6ff5881b1def45b35384572f61327e42563a89c3
Cr-Commit-Position: refs/heads/master@{#35547}
==========

[commit-bot: I haz the power, 2016-04-16 06:49:14]

Patchset 6 (id:??) landed as
https://crrev.com/6ff5881b1def45b35384572f61327e42563a89c3
Cr-Commit-Position: refs/heads/master@{#35547}

[Michael Achenbach, 2016-04-16 13:11:05]

A revert of this CL (patchset #6 id:100001) has been created in
https://codereview.chromium.org/1895643002/ by machenbach@chromium.org.

The reason for reverting is: [Sheriff] Breaks
https://build.chromium.org/p/client.v8/builders/V8%20Linux%20ASAN%20mipsel%20....