X87: [turbofan] Change Float64Max/Float64Min to JavaScript semantics.

X87: [turbofan] Change Float64Max/Float64Min to JavaScript semantics.

  port ba092fb09abb6ba3b7154444510c3b4cbdcf65f5 (r37971)

  original commit message:
  So far we don't have a useful way to inline Math.max or Math.min in
  TurboFan optimized code. This adds new operators NumberMax and NumberMin
  and changes the Float64Max/Float64Min operators to have JavaScript
  semantics instead of the C++ semantics that it had previously.

  This also removes support for recognizing the tenary case in the
  CommonOperatorReducer, since that doesn't seem to have any positive
  impact (and actually doesn't show up in regular JavaScript, where
  people use Math.max/Math.min instead).

BUG=

  Drive-by-fix: Also nuke the unused Float32Max/Float32Min operators.
Committed: https://crrev.com/691e6d39e7110d01dfbc4010296e5871a2374965
Cr-Commit-Position: refs/heads/master@{#38119}

[zhengxing.li, 2016-07-28 07:55:47]

zhengxing.li@intel.com changed reviewers:
+ bmeurer@chromium.org, chunyang.dai@intel.com, weiliang.lin@intel.com

[zhengxing.li, 2016-07-28 07:55:47]

PTAL, thanks!

[Weiliang, 2016-07-28 08:13:52]

lgtm

[zhengxing.li, 2016-07-28 08:33:03]

The CQ bit was checked by zhengxing.li@intel.com

[commit-bot: I haz the power, 2016-07-28 08:33:15]

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-07-28 09:00:16]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-07-28 09:03:05]

Description was changed from

==========
X87: [turbofan] Change Float64Max/Float64Min to JavaScript semantics.

  port ba092fb09abb6ba3b7154444510c3b4cbdcf65f5 (r37971)

  original commit message:
  So far we don't have a useful way to inline Math.max or Math.min in
  TurboFan optimized code. This adds new operators NumberMax and NumberMin
  and changes the Float64Max/Float64Min operators to have JavaScript
  semantics instead of the C++ semantics that it had previously.

  This also removes support for recognizing the tenary case in the
  CommonOperatorReducer, since that doesn't seem to have any positive
  impact (and actually doesn't show up in regular JavaScript, where
  people use Math.max/Math.min instead).

BUG=

  Drive-by-fix: Also nuke the unused Float32Max/Float32Min operators.
==========

to

==========
X87: [turbofan] Change Float64Max/Float64Min to JavaScript semantics.

  port ba092fb09abb6ba3b7154444510c3b4cbdcf65f5 (r37971)

  original commit message:
  So far we don't have a useful way to inline Math.max or Math.min in
  TurboFan optimized code. This adds new operators NumberMax and NumberMin
  and changes the Float64Max/Float64Min operators to have JavaScript
  semantics instead of the C++ semantics that it had previously.

  This also removes support for recognizing the tenary case in the
  CommonOperatorReducer, since that doesn't seem to have any positive
  impact (and actually doesn't show up in regular JavaScript, where
  people use Math.max/Math.min instead).

BUG=

  Drive-by-fix: Also nuke the unused Float32Max/Float32Min operators.
Committed: https://crrev.com/691e6d39e7110d01dfbc4010296e5871a2374965
Cr-Commit-Position: refs/heads/master@{#38119}
==========

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

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