[turbofan] Refined cost model for instruction scheduling on arm64.

[turbofan] Refined cost model for instruction scheduling on arm64.

R=jarin@chromium.org

Committed: https://crrev.com/2bc60707775813c555fcd01e4cbeffeb4c0bf3e6
Cr-Commit-Position: refs/heads/master@{#35752}

[baptiste.afsa1, 2016-04-21 15:57:12]

PTAL

[Jarin, 2016-04-24 09:32:10]

lgtm with a nit.

https://codereview.chromium.org/1905883003/diff/1/src/compiler/arm64/instruct...
File src/compiler/arm64/instruction-scheduler-arm64.cc (right):

https://codereview.chromium.org/1905883003/diff/1/src/compiler/arm64/instruct...
src/compiler/arm64/instruction-scheduler-arm64.cc:206: // fallthrough
I am not a big fan of non-trivial fallthroughs. Would it be so much work to say
" else { return 1; }" here?

[baptiste.afsa1, 2016-04-25 08:02:20]

The CQ bit was checked by baptiste.afsa@arm.com

[baptiste.afsa1, 2016-04-25 08:02:21]

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/1905883003/#ps20001
(title: "addressed comment")

[commit-bot: I haz the power, 2016-04-25 08:02:36]

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

[commit-bot: I haz the power, 2016-04-25 08:22:21]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-04-25 08:23:14]

Description was changed from

==========
[turbofan] Refined cost model for instruction scheduling on arm64.

R=jarin@chromium.org
==========

to

==========
[turbofan] Refined cost model for instruction scheduling on arm64.

R=jarin@chromium.org

Committed: https://crrev.com/2bc60707775813c555fcd01e4cbeffeb4c0bf3e6
Cr-Commit-Position: refs/heads/master@{#35752}
==========

[commit-bot: I haz the power, 2016-04-25 08:23:15]

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