[turbofan] Remove inlining support for the deprecated pipeline.

[turbofan] Remove inlining support for the deprecated pipeline.

The deprecated pipeline is used for asm.js only, where we forcibly
disable inlining anyways (for performance reasons), so inlining via
the AstGraphBuilder is essentially dead code by now, thus there's no
point in trying to keep that around in the code base.

Also nuke the test-run-inlining.cc file, which would require some heavy
surgery (for probably little benefit), and move the useful tests for
mjsunit tests instead.

BUG=v8:2206, v8:5657
R=yangguo@chromium.org

Committed: https://crrev.com/76fd6f25a9ad4e02fe28beff97f8f083f1d48880
Cr-Commit-Position: refs/heads/master@{#41245}

[Benedikt Meurer, 2016-11-24 07:02:23]

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

[commit-bot: I haz the power, 2016-11-24 07:02:31]

Dry run: 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-11-24 07:28:37]

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

[commit-bot: I haz the power, 2016-11-24 07:28:38]

Dry run: This issue passed the CQ dry run.

[Yang, 2016-11-24 07:30:42]

On 2016/11/24 07:02:23, Benedikt Meurer wrote:

lgtm

[Yang, 2016-11-24 07:30:49]

https://codereview.chromium.org/2527053002/diff/1/src/compiler/pipeline.cc
File src/compiler/pipeline.cc (right):

https://codereview.chromium.org/2527053002/diff/1/src/compiler/pipeline.cc#ne...
src/compiler/pipeline.cc:565: if (FLAG_turbo_inlining) {
just make this an 'else if'?

[Benedikt Meurer, 2016-11-24 07:34:01]

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

[Benedikt Meurer, 2016-11-24 07:34:08]

https://codereview.chromium.org/2527053002/diff/1/src/compiler/pipeline.cc
File src/compiler/pipeline.cc (right):

https://codereview.chromium.org/2527053002/diff/1/src/compiler/pipeline.cc#ne...
src/compiler/pipeline.cc:565: if (FLAG_turbo_inlining) {
On 2016/11/24 07:30:49, Yang wrote:
> just make this an 'else if'?

Done.

[commit-bot: I haz the power, 2016-11-24 07:34:09]

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-11-24 07:34:21]

The CQ bit was unchecked by bmeurer@chromium.org

[Benedikt Meurer, 2016-11-24 07:34:30]

The CQ bit was checked by bmeurer@chromium.org

[Benedikt Meurer, 2016-11-24 07:34:30]

The patchset sent to the CQ was uploaded after l-g-t-m from yangguo@chromium.org
Link to the patchset: https://codereview.chromium.org/2527053002/#ps20001
(title: "Address nit.")

[commit-bot: I haz the power, 2016-11-24 07:34:39]

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-11-24 07:59:27]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1479972870214930,
"parent_rev": "9840bdc377234d384ee2122df6871fe4df022d57", "commit_rev":
"c95ac47b7e83193cd59f2f1275e5f2e8e3e01596"}

[commit-bot: I haz the power, 2016-11-24 07:59:33]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-11-24 08:00:03]

Description was changed from

==========
[turbofan] Remove inlining support for the deprecated pipeline.

The deprecated pipeline is used for asm.js only, where we forcibly
disable inlining anyways (for performance reasons), so inlining via
the AstGraphBuilder is essentially dead code by now, thus there's no
point in trying to keep that around in the code base.

Also nuke the test-run-inlining.cc file, which would require some heavy
surgery (for probably little benefit), and move the useful tests for
mjsunit tests instead.

BUG=v8:2206,v8:5657
R=yangguo@chromium.org
==========

to

==========
[turbofan] Remove inlining support for the deprecated pipeline.

The deprecated pipeline is used for asm.js only, where we forcibly
disable inlining anyways (for performance reasons), so inlining via
the AstGraphBuilder is essentially dead code by now, thus there's no
point in trying to keep that around in the code base.

Also nuke the test-run-inlining.cc file, which would require some heavy
surgery (for probably little benefit), and move the useful tests for
mjsunit tests instead.

BUG=v8:2206,v8:5657
R=yangguo@chromium.org

Committed: https://crrev.com/76fd6f25a9ad4e02fe28beff97f8f083f1d48880
Cr-Commit-Position: refs/heads/master@{#41245}
==========

[commit-bot: I haz the power, 2016-11-24 08:00:04]

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