Atomic operations on Uint8ClampedArray

Atomic operations on Uint8ClampedArray

BUG=chromium:497295
R=jarin@chromium.org
LOG=n

Committed: https://crrev.com/5b3700a49a403963e64fd44b529021130458b452
Cr-Commit-Position: refs/heads/master@{#29199}

[Jarin, 2015-06-22 07:04:04]

lgtm.

https://codereview.chromium.org/1201543002/diff/1/src/runtime/runtime-atomics.cc
File src/runtime/runtime-atomics.cc (right):

https://codereview.chromium.org/1201543002/diff/1/src/runtime/runtime-atomics...
src/runtime/runtime-atomics.cc:433: return value;
Hmm, I am really surprised this does not need any casting.

[binji, 2015-06-22 18:21:04]

https://codereview.chromium.org/1201543002/diff/1/src/runtime/runtime-atomics.cc
File src/runtime/runtime-atomics.cc (right):

https://codereview.chromium.org/1201543002/diff/1/src/runtime/runtime-atomics...
src/runtime/runtime-atomics.cc:433: return value;
On 2015/06/22 at 07:04:03, jarin wrote:
> Hmm, I am really surprised this does not need any casting.

I didn't notice it until you mentioned it. I checked, and it doesn't produce an
error or warning on clang/gcc with -Wall. Also:
http://en.cppreference.com/w/cpp/language/implicit_cast#Numeric_conversions

[binji, 2015-06-22 18:21:07]

The CQ bit was checked by binji@chromium.org

[commit-bot: I haz the power, 2015-06-22 18:21:20]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1201543002/1

[commit-bot: I haz the power, 2015-06-22 18:40:38]

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

[commit-bot: I haz the power, 2015-06-22 18:40:39]

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

[binji, 2015-06-22 18:42:07]

The CQ bit was checked by binji@chromium.org

[commit-bot: I haz the power, 2015-06-22 18:42:31]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1201543002/1

[commit-bot: I haz the power, 2015-06-22 18:51:14]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2015-06-22 18:51:33]

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