X87: [builtins] Make Math.max and Math.min fast by default.

X87: [builtins] Make Math.max and Math.min fast by default.

  port cb9b801069c432e745f0651cf00d65ef88cd7f06 (r33582)

  original commit message:
  The previous versions of Math.max and Math.min made it difficult to
  optimize those (that's why we already have custom code in Crankshaft),
  and due to lack of ideas what to do about the variable number of
  arguments, we will probably need to stick in special code in TurboFan
  as well; so inlining those builtins is off the table, hence there's no
  real advantage in having them around as "not quite JS" with extra work
  necessary in the optimizing compilers to still make those builtins
  somewhat fast in cases where we cannot inline them (also there's a
  tricky deopt loop in Crankshaft related to Math.min and Math.max, but
  that will be dealt with later).

  So to sum up: Instead of trying to make Math.max and Math.min semi-fast
  in the optimizing compilers with weird work-arounds support %_Arguments
  %_ArgumentsLength, we do provide the optimal code as native builtins
  instead and call it a day (which gives a nice performance boost on some
  benchmarks).

BUG=

Committed: https://crrev.com/8944d36fd57aa398a911c7f1e567baf6ddb14b24
Cr-Commit-Position: refs/heads/master@{#33652}

[zhengxing.li, 2016-02-02 02:04:43]

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

[zhengxing.li, 2016-02-02 02:04:43]

PTAL, thanks!

[Weiliang, 2016-02-02 02:12:28]

lgtm

[zhengxing.li, 2016-02-02 02:19:21]

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

[commit-bot: I haz the power, 2016-02-02 02:19:39]

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

[commit-bot: I haz the power, 2016-02-02 02:47:13]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-02-02 02:47:57]

Description was changed from

==========
X87: [builtins] Make Math.max and Math.min fast by default.

  port cb9b801069c432e745f0651cf00d65ef88cd7f06 (r33582)

  original commit message:
  The previous versions of Math.max and Math.min made it difficult to
  optimize those (that's why we already have custom code in Crankshaft),
  and due to lack of ideas what to do about the variable number of
  arguments, we will probably need to stick in special code in TurboFan
  as well; so inlining those builtins is off the table, hence there's no
  real advantage in having them around as "not quite JS" with extra work
  necessary in the optimizing compilers to still make those builtins
  somewhat fast in cases where we cannot inline them (also there's a
  tricky deopt loop in Crankshaft related to Math.min and Math.max, but
  that will be dealt with later).

  So to sum up: Instead of trying to make Math.max and Math.min semi-fast
  in the optimizing compilers with weird work-arounds support %_Arguments
  %_ArgumentsLength, we do provide the optimal code as native builtins
  instead and call it a day (which gives a nice performance boost on some
  benchmarks).

BUG=
==========

to

==========
X87: [builtins] Make Math.max and Math.min fast by default.

  port cb9b801069c432e745f0651cf00d65ef88cd7f06 (r33582)

  original commit message:
  The previous versions of Math.max and Math.min made it difficult to
  optimize those (that's why we already have custom code in Crankshaft),
  and due to lack of ideas what to do about the variable number of
  arguments, we will probably need to stick in special code in TurboFan
  as well; so inlining those builtins is off the table, hence there's no
  real advantage in having them around as "not quite JS" with extra work
  necessary in the optimizing compilers to still make those builtins
  somewhat fast in cases where we cannot inline them (also there's a
  tricky deopt loop in Crankshaft related to Math.min and Math.max, but
  that will be dealt with later).

  So to sum up: Instead of trying to make Math.max and Math.min semi-fast
  in the optimizing compilers with weird work-arounds support %_Arguments
  %_ArgumentsLength, we do provide the optimal code as native builtins
  instead and call it a day (which gives a nice performance boost on some
  benchmarks).

BUG=

Committed: https://crrev.com/8944d36fd57aa398a911c7f1e567baf6ddb14b24
Cr-Commit-Position: refs/heads/master@{#33652}
==========

[commit-bot: I haz the power, 2016-02-02 02:47:57]

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