[turbofan] Introduce simplified operator NumberAbs.

[turbofan] Introduce simplified operator NumberAbs.

Add NumberAbs operator to implement an inline version of Math.abs, that
can be optimized and eliminated. We don't use any speculation here, but
for now stick to the information we can infer (this way we avoid the
inherent deopt loops that Crankshaft has around Math.abs).

CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
R=jarin@chromium.org
BUG=v8:5086
Committed: https://crrev.com/f50a601ffa507ec540e661f67bc752fe88114631
Cr-Commit-Position: refs/heads/master@{#37306}

[Jarin, 2016-06-27 13:24:35]

lgtm

[Benedikt Meurer, 2016-06-27 13:24:47]

The CQ bit was checked by bmeurer@chromium.org

[commit-bot: I haz the power, 2016-06-27 13:24:52]

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-06-27 13:35:09]

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

[commit-bot: I haz the power, 2016-06-27 13:35:10]

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...)

[Benedikt Meurer, 2016-06-27 13:35:25]

The CQ bit was checked by bmeurer@chromium.org

[Benedikt Meurer, 2016-06-27 13:35:25]

The patchset sent to the CQ was uploaded after l-g-t-m from jarin@chromium.org
Link to the patchset: https://codereview.chromium.org/2096403002/#ps20001
(title: "Hotfix for minus zero")

[commit-bot: I haz the power, 2016-06-27 13:35:29]

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-06-27 14:10:38]

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

[commit-bot: I haz the power, 2016-06-27 14:10:39]

Try jobs failed on following builders:
  v8_linux_arm64_rel_ng on master.tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux_arm64_rel_ng/build...)
  v8_linux_arm64_rel_ng_triggered on master.tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux_arm64_rel_ng_trigg...)

[Benedikt Meurer, 2016-06-27 19:02:58]

Description was changed from

==========
[turbofan] Introduce simplified operator NumberAbs.

Add NumberAbs operator to implement an inline version of Math.abs, that
can be optimized and eliminated. We don't use any speculation here, but
for now stick to the information we can infer (this way we avoid the
inherent deopt loops that Crankshaft has around Math.abs).

R=jarin@chromium.org
BUG=v8:5086
==========

to

==========
[turbofan] Introduce simplified operator NumberAbs.

Add NumberAbs operator to implement an inline version of Math.abs, that
can be optimized and eliminated. We don't use any speculation here, but
for now stick to the information we can infer (this way we avoid the
inherent deopt loops that Crankshaft has around Math.abs).

CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
R=jarin@chromium.org
BUG=v8:5086
==========

[Benedikt Meurer, 2016-06-27 19:03:08]

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

[commit-bot: I haz the power, 2016-06-27 19:03:14]

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

[Benedikt Meurer, 2016-06-27 19:14:45]

The CQ bit was unchecked by bmeurer@chromium.org

[Benedikt Meurer, 2016-06-27 19:14:51]

The CQ bit was checked by bmeurer@chromium.org

[Benedikt Meurer, 2016-06-27 19:14:51]

The patchset sent to the CQ was uploaded after l-g-t-m from jarin@chromium.org
Link to the patchset: https://codereview.chromium.org/2096403002/#ps40001
(title: "Use std::fabs instead of std::abs.")

[commit-bot: I haz the power, 2016-06-27 19:14:57]

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-06-27 19:41:28]

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

[commit-bot: I haz the power, 2016-06-27 19:41:29]

Try jobs failed on following builders:
  v8_linux_arm64_rel_ng on master.tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux_arm64_rel_ng/build...)
  v8_linux_arm64_rel_ng_triggered on master.tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux_arm64_rel_ng_trigg...)

[Benedikt Meurer, 2016-06-27 19:45:33]

The CQ bit was checked by bmeurer@chromium.org

[Benedikt Meurer, 2016-06-27 19:45:33]

The patchset sent to the CQ was uploaded after l-g-t-m from jarin@chromium.org
Link to the patchset: https://codereview.chromium.org/2096403002/#ps60001
(title: "Really fix the typing rule now.")

[commit-bot: I haz the power, 2016-06-27 19:45:37]

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-06-27 19:47:47]

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

[commit-bot: I haz the power, 2016-06-27 19:47:48]

Try jobs failed on following builders:
  v8_linux_gcc_compile_rel on master.tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux_gcc_compile_rel/bu...)
  v8_linux_mipsel_compile_rel on master.tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux_mipsel_compile_rel...)
  v8_mac_rel_ng on master.tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_mac_rel_ng/builds/3942)

[Benedikt Meurer, 2016-06-27 19:59:11]

The CQ bit was checked by bmeurer@chromium.org

[Benedikt Meurer, 2016-06-27 19:59:11]

The patchset sent to the CQ was uploaded after l-g-t-m from jarin@chromium.org
Link to the patchset: https://codereview.chromium.org/2096403002/#ps100001
(title: "Fix compile errors")

[commit-bot: I haz the power, 2016-06-27 19:59:17]

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-06-27 20:01:27]

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

[commit-bot: I haz the power, 2016-06-27 20:01:28]

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_linux_dbg_ng on master.tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux_dbg_ng/builds/8017)

[Benedikt Meurer, 2016-06-28 03:50:27]

The CQ bit was checked by bmeurer@chromium.org

[Benedikt Meurer, 2016-06-28 03:50:28]

The patchset sent to the CQ was uploaded after l-g-t-m from jarin@chromium.org
Link to the patchset: https://codereview.chromium.org/2096403002/#ps120001
(title: "This time fix the compile error for realz")

[commit-bot: I haz the power, 2016-06-28 03:50:37]

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-06-28 04:34:30]

Description was changed from

==========
[turbofan] Introduce simplified operator NumberAbs.

Add NumberAbs operator to implement an inline version of Math.abs, that
can be optimized and eliminated. We don't use any speculation here, but
for now stick to the information we can infer (this way we avoid the
inherent deopt loops that Crankshaft has around Math.abs).

CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
R=jarin@chromium.org
BUG=v8:5086
==========

to

==========
[turbofan] Introduce simplified operator NumberAbs.

Add NumberAbs operator to implement an inline version of Math.abs, that
can be optimized and eliminated. We don't use any speculation here, but
for now stick to the information we can infer (this way we avoid the
inherent deopt loops that Crankshaft has around Math.abs).

CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
R=jarin@chromium.org
BUG=v8:5086
==========

[commit-bot: I haz the power, 2016-06-28 04:34:31]

Committed patchset #7 (id:120001)

[commit-bot: I haz the power, 2016-06-28 04:37:12]

Description was changed from

==========
[turbofan] Introduce simplified operator NumberAbs.

Add NumberAbs operator to implement an inline version of Math.abs, that
can be optimized and eliminated. We don't use any speculation here, but
for now stick to the information we can infer (this way we avoid the
inherent deopt loops that Crankshaft has around Math.abs).

CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
R=jarin@chromium.org
BUG=v8:5086
==========

to

==========
[turbofan] Introduce simplified operator NumberAbs.

Add NumberAbs operator to implement an inline version of Math.abs, that
can be optimized and eliminated. We don't use any speculation here, but
for now stick to the information we can infer (this way we avoid the
inherent deopt loops that Crankshaft has around Math.abs).

CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
R=jarin@chromium.org
BUG=v8:5086

Committed: https://crrev.com/f50a601ffa507ec540e661f67bc752fe88114631
Cr-Commit-Position: refs/heads/master@{#37306}
==========

[commit-bot: I haz the power, 2016-06-28 04:37:13]

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