[turbofan] Stage --turbo-inlining behind --turbo flag.

[turbofan] Stage --turbo-inlining behind --turbo flag.

This stages the general purpose inlining mechanism in TurboFan and
also disables the remaining tests that still fail. We do this to get
test coverage early and to avoid regressing inlining as we go along.

R=jarin@chromium.org,mstarzinger@chromium.org
BUG=v8:4493
LOG=n

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

[Benedikt Meurer, 2015-10-20 10:29:56]

Next try staging this beast... PTAL

[Jarin, 2015-10-20 10:31:03]

LGTM with a nit.

https://codereview.chromium.org/1419623002/diff/1/src/flag-definitions.h
File src/flag-definitions.h (right):

https://codereview.chromium.org/1419623002/diff/1/src/flag-definitions.h#newc...
src/flag-definitions.h:415: DEFINE_IMPLICATION(turbo, turbo_asm_deoptimization)
Could you remove this line, then?

[Michael Starzinger, 2015-10-20 10:32:09]

LGTM.

https://codereview.chromium.org/1419623002/diff/1/src/flag-definitions.h
File src/flag-definitions.h (right):

https://codereview.chromium.org/1419623002/diff/1/src/flag-definitions.h#newc...
src/flag-definitions.h:415: DEFINE_IMPLICATION(turbo, turbo_asm_deoptimization)
On 2015/10/20 10:31:03, Jarin wrote:
> Could you remove this line, then?

+1, also the empty newline above.

[Benedikt Meurer, 2015-10-20 10:32:49]

https://codereview.chromium.org/1419623002/diff/1/src/flag-definitions.h
File src/flag-definitions.h (right):

https://codereview.chromium.org/1419623002/diff/1/src/flag-definitions.h#newc...
src/flag-definitions.h:415: DEFINE_IMPLICATION(turbo, turbo_asm_deoptimization)
On 2015/10/20 10:32:09, Michael Starzinger wrote:
> On 2015/10/20 10:31:03, Jarin wrote:
> > Could you remove this line, then?
> 
> +1, also the empty newline above.

Done.

[Benedikt Meurer, 2015-10-20 10:32:54]

The CQ bit was checked by bmeurer@chromium.org

[Benedikt Meurer, 2015-10-20 10:32:54]

The patchset sent to the CQ was uploaded after l-g-t-m from jarin@chromium.org,
mstarzinger@chromium.org
Link to the patchset: https://codereview.chromium.org/1419623002/#ps20001
(title: "Address comments")

[commit-bot: I haz the power, 2015-10-20 10:32:58]

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

[commit-bot: I haz the power, 2015-10-20 10:56:56]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-10-20 10:57:08]

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