[turbofan] Change TruncateFloat32ToUint64 to TryTruncateFloat32ToUint64.

[turbofan] Change TruncateFloat32ToUint64 to TryTruncateFloat32ToUint64.

TryTruncateFloat32ToUint64 converts a float32 to a uint64. Additionally it
provides an optional second return value which indicates whether the conversion
succeeded (i.e. float32 value was within uint64 range) or not.

I implemented the new operator on x64, arm64, and mips64. @v8-ppc-ports, can you
please take care of the ppc64 implementation of the second output?

Additionally I fixed a bug on x64 and mips64 in the implementation of
TryTruncateFloat64ToUint64. Cases where the input value was between -1 and 0
were handled incorrectly.

R=titzer@chromium.org, v8-arm-ports@googlegroups.com, v8-mips-ports@googlegroups.com

Committed: https://crrev.com/97161a29eda804ea17ba127e78ddaecee0a259a9
Cr-Commit-Position: refs/heads/master@{#32796}

[ahaas, 2015-12-09 17:10:01]

https://codereview.chromium.org/1512023002/diff/1/src/mips64/macro-assembler-...
File src/mips64/macro-assembler-mips64.cc (right):

https://codereview.chromium.org/1512023002/diff/1/src/mips64/macro-assembler-...
src/mips64/macro-assembler-mips64.cc:1783: BranchF(&fail, &fail, lt, scratch,
kDoubleRegZero);
@v8-mips-ports: originally I wanted to use BranchF32 here, but then some tests
fail. What's the reason why BranchF has to be used here?

[paul.l..., 2015-12-10 06:51:57]

paul.lind@imgtec.com changed reviewers:
+ paul.lind@imgtec.com

[paul.l..., 2015-12-10 06:51:57]

https://codereview.chromium.org/1512023002/diff/1/src/mips64/macro-assembler-...
File src/mips64/macro-assembler-mips64.cc (right):

https://codereview.chromium.org/1512023002/diff/1/src/mips64/macro-assembler-...
src/mips64/macro-assembler-mips64.cc:1783: BranchF(&fail, &fail, lt, scratch,
kDoubleRegZero);
On 2015/12/09 17:10:01, ahaas wrote:
> @v8-mips-ports: originally I wanted to use BranchF32 here, but then some tests
> fail. What's the reason why BranchF has to be used here?

This is clearly broken, but I could not spot the bug tonight. I've created an
internal issue for it. Why don't you land this with the BranchF() when you are
ready, and we'll fix it in followup CL.

[balazs.kilvady, 2015-12-10 15:20:15]

We created a MIPS64 fix, PTAL: https://codereview.chromium.org/1510493012/ or we
can land it as a followup CL if you prefer that.

[ahaas, 2015-12-10 15:31:42]

On 2015/12/10 at 15:20:15, balazs.kilvady wrote:
> We created a MIPS64 fix, PTAL: https://codereview.chromium.org/1510493012/ or
we can land it as a followup CL if you prefer that.

I added the code to this CL. Thanks!

[titzer, 2015-12-11 12:43:26]

On 2015/12/10 15:31:42, ahaas wrote:
> On 2015/12/10 at 15:20:15, balazs.kilvady wrote:
> > We created a MIPS64 fix, PTAL: https://codereview.chromium.org/1510493012/
or
> we can land it as a followup CL if you prefer that.
> 
> I added the code to this CL. Thanks!

lgtm

[ahaas, 2015-12-11 12:47:32]

The CQ bit was checked by ahaas@chromium.org

[commit-bot: I haz the power, 2015-12-11 12:47:35]

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

[commit-bot: I haz the power, 2015-12-11 14:07:04]

Committed patchset #4 (id:60001)

[commit-bot: I haz the power, 2015-12-11 14:07:24]

Description was changed from

==========
[turbofan] Change TruncateFloat32ToUint64 to TryTruncateFloat32ToUint64.

TryTruncateFloat32ToUint64 converts a float32 to a uint64. Additionally it
provides an optional second return value which indicates whether the conversion
succeeded (i.e. float32 value was within uint64 range) or not.

I implemented the new operator on x64, arm64, and mips64. @v8-ppc-ports, can you
please take care of the ppc64 implementation of the second output?

Additionally I fixed a bug on x64 and mips64 in the implementation of
TryTruncateFloat64ToUint64. Cases where the input value was between -1 and 0
were handled incorrectly.

R=titzer@chromium.org, v8-arm-ports@googlegroups.com,
v8-mips-ports@googlegroups.com
==========

to

==========
[turbofan] Change TruncateFloat32ToUint64 to TryTruncateFloat32ToUint64.

TryTruncateFloat32ToUint64 converts a float32 to a uint64. Additionally it
provides an optional second return value which indicates whether the conversion
succeeded (i.e. float32 value was within uint64 range) or not.

I implemented the new operator on x64, arm64, and mips64. @v8-ppc-ports, can you
please take care of the ppc64 implementation of the second output?

Additionally I fixed a bug on x64 and mips64 in the implementation of
TryTruncateFloat64ToUint64. Cases where the input value was between -1 and 0
were handled incorrectly.

R=titzer@chromium.org, v8-arm-ports@googlegroups.com,
v8-mips-ports@googlegroups.com

Committed: https://crrev.com/97161a29eda804ea17ba127e78ddaecee0a259a9
Cr-Commit-Position: refs/heads/master@{#32796}
==========

[commit-bot: I haz the power, 2015-12-11 14:07:24]

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