[turbofan] Fix optimized lowering of Math.imul.

[turbofan] Fix optimized lowering of Math.imul.

We eagerly inserted Int32Mul for Math.imul during builtin lowering and
messed up with the types, which confused the representation selection.
This adds a proper NumberImul operator, and fixes the builtin reducer to
do the right thing according to the spec.

R=mstarzinger@chromium.org
BUG=v8:5006
LOG=n

Committed: https://crrev.com/fa7460adbcc544dcd1483999b553e0cdcab43b51
Cr-Commit-Position: refs/heads/master@{#36219}

[Benedikt Meurer, 2016-05-12 12:58:10]

Hey Michi,

Next bug fix.
Please take a look.

Thanks,
Benedikt

[Michael Starzinger, 2016-05-12 13:04:06]

LGTM.

[Benedikt Meurer, 2016-05-12 13:09:54]

The CQ bit was checked by bmeurer@chromium.org

[commit-bot: I haz the power, 2016-05-12 13:10:02]

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

[commit-bot: I haz the power, 2016-05-12 13:33:32]

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

[commit-bot: I haz the power, 2016-05-12 13:33:33]

Try jobs failed on following builders:
  v8_linux_dbg_ng on tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux_dbg_ng/builds/5686)
  v8_linux_dbg_ng_triggered on tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux_dbg_ng_triggered/b...)

[Benedikt Meurer, 2016-05-12 18:04:14]

The CQ bit was checked by bmeurer@chromium.org

[Benedikt Meurer, 2016-05-12 18:04:15]

The patchset sent to the CQ was uploaded after l-g-t-m from
mstarzinger@chromium.org
Link to the patchset: https://codereview.chromium.org/1971163002/#ps20001
(title: "Fix unittests. Reorder as pointed out by Michi.")

[commit-bot: I haz the power, 2016-05-12 18:04:17]

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

[commit-bot: I haz the power, 2016-05-12 18:42:21]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-05-12 18:43:38]

Description was changed from

==========
[turbofan] Fix optimized lowering of Math.imul.

We eagerly inserted Int32Mul for Math.imul during builtin lowering and
messed up with the types, which confused the representation selection.
This adds a proper NumberImul operator, and fixes the builtin reducer to
do the right thing according to the spec.

R=mstarzinger@chromium.org
BUG=v8:5006
LOG=n
==========

to

==========
[turbofan] Fix optimized lowering of Math.imul.

We eagerly inserted Int32Mul for Math.imul during builtin lowering and
messed up with the types, which confused the representation selection.
This adds a proper NumberImul operator, and fixes the builtin reducer to
do the right thing according to the spec.

R=mstarzinger@chromium.org
BUG=v8:5006
LOG=n

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

[commit-bot: I haz the power, 2016-05-12 18:43:39]

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