Allocate typed array for rempio2 result.

Allocate typed array for rempio2 result.

R=svenpanne@chromium.org
BUG=chromium:451670
LOG=N

Committed: https://crrev.com/a44511d4131c74c6a0f9622af9ccaab83a5235bc
Cr-Commit-Position: refs/heads/master@{#26482}

[Sven Panne, 2015-02-06 07:37:28]

LGTM with nits.

https://codereview.chromium.org/866553003/diff/40001/src/runtime/runtime-math...
File src/runtime/runtime-maths.cc (right):

https://codereview.chromium.org/866553003/diff/40001/src/runtime/runtime-math...
src/runtime/runtime-maths.cc:70: double* y =
reinterpret_cast<double*>(backing_store);
A static_cast is enough here and a smaller sledgehammer for the task at hand.
;-) Perhaps one should templatize backing_store(), most uses have to cast,
anyway (but this would definitely be a separate CL).

https://codereview.chromium.org/866553003/diff/40001/src/runtime/runtime-math...
src/runtime/runtime-maths.cc:72: return Smi::FromInt(n);
Just inline "n", its name is not that helpful, anyway.

https://codereview.chromium.org/866553003/diff/40001/src/third_party/fdlibm/f...
File src/third_party/fdlibm/fdlibm.js (right):

https://codereview.chromium.org/866553003/diff/40001/src/third_party/fdlibm/f...
src/third_party/fdlibm/fdlibm.js:27: var rempio2result;
Extend the comment above.

[Yang, 2015-02-06 09:31:02]

New patchsets have been uploaded after l-g-t-m from svenpanne@chromium.org

[Yang, 2015-02-06 09:31:48]

On 2015/02/06 07:37:28, Sven Panne wrote:
> LGTM with nits.
> 
>
https://codereview.chromium.org/866553003/diff/40001/src/runtime/runtime-math...
> File src/runtime/runtime-maths.cc (right):
> 
>
https://codereview.chromium.org/866553003/diff/40001/src/runtime/runtime-math...
> src/runtime/runtime-maths.cc:70: double* y =
> reinterpret_cast<double*>(backing_store);
> A static_cast is enough here and a smaller sledgehammer for the task at hand.
> ;-) Perhaps one should templatize backing_store(), most uses have to cast,
> anyway (but this would definitely be a separate CL).
> 
>
https://codereview.chromium.org/866553003/diff/40001/src/runtime/runtime-math...
> src/runtime/runtime-maths.cc:72: return Smi::FromInt(n);
> Just inline "n", its name is not that helpful, anyway.
> 
>
https://codereview.chromium.org/866553003/diff/40001/src/third_party/fdlibm/f...
> File src/third_party/fdlibm/fdlibm.js (right):
> 
>
https://codereview.chromium.org/866553003/diff/40001/src/third_party/fdlibm/f...
> src/third_party/fdlibm/fdlibm.js:27: var rempio2result;
> Extend the comment above.

Addressed comments.

[Yang, 2015-02-06 09:31:51]

The CQ bit was checked by yangguo@chromium.org

[commit-bot: I haz the power, 2015-02-06 09:32:33]

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

[commit-bot: I haz the power, 2015-02-06 09:55:03]

Committed patchset #4 (id:60001)

[commit-bot: I haz the power, 2015-02-06 09:55:18]

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