[wasm] Use the Float64Max/Min machine operators to implement F64Max/Min.

[wasm] Use the Float64Max/Min machine operators to implement F64Max/Min.

I had to adjust Float64Max/Min on x64 slightly to return the default
wasm NaN (0x7FF0000000000000) instead of the all-ones NaN
(0xFFFFFFFFFFFFFFFF).

R=titzer@chromium.org, bmeurer@chromium.org

Committed: https://crrev.com/96c90f6c64d7a9e4a49491c4a80debd78b376bcf
Cr-Commit-Position: refs/heads/master@{#38410}

[ahaas, 2016-08-02 16:34:09]

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

[commit-bot: I haz the power, 2016-08-02 16:34:19]

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, 2016-08-02 16:46:27]

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

[commit-bot: I haz the power, 2016-08-02 16:46:28]

Dry run: Try jobs failed on following builders:
  v8_linux64_gyp_rel_ng on master.tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux64_gyp_rel_ng/build...)
  v8_linux64_gyp_rel_ng_triggered on master.tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux64_gyp_rel_ng_trigg...)

[Benedikt Meurer, 2016-08-03 03:24:34]

There seems to be some cementation unhappiness. Otherwise LGTM on the
TurboFan/Crankshaft parts.

https://codereview.chromium.org/2204963002/diff/1/src/compiler/x64/code-gener...
File src/compiler/x64/code-generator-x64.cc (right):

https://codereview.chromium.org/2204963002/diff/1/src/compiler/x64/code-gener...
src/compiler/x64/code-generator-x64.cc:174: __ xorpd(result_, result_);
Please use Xorpd and Divsd here.

https://codereview.chromium.org/2204963002/diff/1/src/crankshaft/x64/lithium-...
File src/crankshaft/x64/lithium-codegen-x64.cc (right):

https://codereview.chromium.org/2204963002/diff/1/src/crankshaft/x64/lithium-...
src/crankshaft/x64/lithium-codegen-x64.cc:4696: __ xorpd(result_reg,
result_reg);
Please use Xorpd and Divsd here.

[ahaas, 2016-08-03 07:16:12]

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

[commit-bot: I haz the power, 2016-08-03 07:16:26]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[ahaas, 2016-08-03 07:17:14]

The try-bots failed because I forgot to update some tests to match the new wasm
semantics.

https://codereview.chromium.org/2204963002/diff/1/src/compiler/x64/code-gener...
File src/compiler/x64/code-generator-x64.cc (right):

https://codereview.chromium.org/2204963002/diff/1/src/compiler/x64/code-gener...
src/compiler/x64/code-generator-x64.cc:174: __ xorpd(result_, result_);
On 2016/08/03 at 03:24:34, Benedikt Meurer wrote:
> Please use Xorpd and Divsd here.

Done.

https://codereview.chromium.org/2204963002/diff/1/src/crankshaft/x64/lithium-...
File src/crankshaft/x64/lithium-codegen-x64.cc (right):

https://codereview.chromium.org/2204963002/diff/1/src/crankshaft/x64/lithium-...
src/crankshaft/x64/lithium-codegen-x64.cc:4696: __ xorpd(result_reg,
result_reg);
On 2016/08/03 at 03:24:34, Benedikt Meurer wrote:
> Please use Xorpd and Divsd here.

Done.

[commit-bot: I haz the power, 2016-08-03 07:28:12]

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

[commit-bot: I haz the power, 2016-08-03 07:28:12]

Dry run: Try jobs failed on following builders:
  v8_linux64_gyp_rel_ng on master.tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux64_gyp_rel_ng/build...)
  v8_linux64_gyp_rel_ng_triggered on master.tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux64_gyp_rel_ng_trigg...)

[ahaas, 2016-08-03 07:43:27]

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

[commit-bot: I haz the power, 2016-08-03 07:43:30]

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, 2016-08-03 07:58:04]

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

[commit-bot: I haz the power, 2016-08-03 07:58:05]

Dry run: Try jobs failed on following builders:
  v8_linux64_avx2_rel_ng on master.tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux64_avx2_rel_ng/buil...)
  v8_linux64_avx2_rel_ng_triggered on master.tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux64_avx2_rel_ng_trig...)

[ahaas, 2016-08-03 08:03:46]

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

[commit-bot: I haz the power, 2016-08-03 08:03:58]

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, 2016-08-03 08:14:07]

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

[commit-bot: I haz the power, 2016-08-03 08:14:07]

Dry run: Try jobs failed on following builders:
  v8_linux64_gyp_rel_ng on master.tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux64_gyp_rel_ng/build...)
  v8_linux64_gyp_rel_ng_triggered on master.tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux64_gyp_rel_ng_trigg...)

[ahaas, 2016-08-03 09:28:32]

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

[commit-bot: I haz the power, 2016-08-03 09:28:43]

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, 2016-08-03 09:35:56]

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

[commit-bot: I haz the power, 2016-08-03 09:35:57]

Dry run: Try jobs failed on following builders:
  v8_win_compile_dbg on master.tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_win_compile_dbg/builds/2...)

[ahaas, 2016-08-03 09:41:01]

On 2016/08/03 at 07:17:14, ahaas wrote:
> The try-bots failed because I forgot to update some tests to match the new
wasm semantics.
> 
>
https://codereview.chromium.org/2204963002/diff/1/src/compiler/x64/code-gener...
> File src/compiler/x64/code-generator-x64.cc (right):
> 
>
https://codereview.chromium.org/2204963002/diff/1/src/compiler/x64/code-gener...
> src/compiler/x64/code-generator-x64.cc:174: __ xorpd(result_, result_);
> On 2016/08/03 at 03:24:34, Benedikt Meurer wrote:
> > Please use Xorpd and Divsd here.
> 
> Done.
> 
>
https://codereview.chromium.org/2204963002/diff/1/src/crankshaft/x64/lithium-...
> File src/crankshaft/x64/lithium-codegen-x64.cc (right):
> 
>
https://codereview.chromium.org/2204963002/diff/1/src/crankshaft/x64/lithium-...
> src/crankshaft/x64/lithium-codegen-x64.cc:4696: __ xorpd(result_reg,
result_reg);
> On 2016/08/03 at 03:24:34, Benedikt Meurer wrote:
> > Please use Xorpd and Divsd here.
> 
> Done.

PTAL.

On ia32 the same nan-producing instruction was used as on x64, so I also
replaced it there by a 0/0.
Additionally, divsd(0.0/0.0) does not produce a float32.nan, so I added
OutOfLineLoadFloat32NaN and OutOfLineLoadFloat64NaN to produce a float32.nan and
a float64.nan, respectively.

[ahaas, 2016-08-04 09:33:08]

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

[commit-bot: I haz the power, 2016-08-04 09:33:12]

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, 2016-08-04 09:53:17]

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

[commit-bot: I haz the power, 2016-08-04 09:53:18]

Dry run: This issue passed the CQ dry run.

[ahaas, 2016-08-05 07:26:38]

The CQ bit was checked by ahaas@chromium.org

[ahaas, 2016-08-05 07:26:38]

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/2204963002/#ps100001
(title: "Typo")

[commit-bot: I haz the power, 2016-08-05 07:26:46]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[ahaas, 2016-08-05 07:27:44]

The CQ bit was unchecked by ahaas@chromium.org

[titzer, 2016-08-05 22:41:16]

On 2016/08/05 07:26:46, commit-bot: I haz the power wrote:
> CQ is trying da patch. Follow status at
>  
>
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

lgtm

[ahaas, 2016-08-08 05:27:27]

The CQ bit was checked by ahaas@chromium.org

[commit-bot: I haz the power, 2016-08-08 05:27:38]

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, 2016-08-08 06:06:16]

Committed patchset #6 (id:100001)

[commit-bot: I haz the power, 2016-08-08 06:06:45]

Description was changed from

==========
[wasm] Use the Float64Max/Min machine operators to implement F64Max/Min.

I had to adjust Float64Max/Min on x64 slightly to return the default
wasm NaN (0x7FF0000000000000) instead of the all-ones NaN
(0xFFFFFFFFFFFFFFFF).

R=titzer@chromium.org, bmeurer@chromium.org
==========

to

==========
[wasm] Use the Float64Max/Min machine operators to implement F64Max/Min.

I had to adjust Float64Max/Min on x64 slightly to return the default
wasm NaN (0x7FF0000000000000) instead of the all-ones NaN
(0xFFFFFFFFFFFFFFFF).

R=titzer@chromium.org, bmeurer@chromium.org

Committed: https://crrev.com/96c90f6c64d7a9e4a49491c4a80debd78b376bcf
Cr-Commit-Position: refs/heads/master@{#38410}
==========

[commit-bot: I haz the power, 2016-08-08 06:06:47]

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