[turbofan] Avoid unnecessary copying of nodes during inlining.

[turbofan] Avoid unnecessary copying of nodes during inlining.

Previously we first created a temporary graph for the inlinee and then
copied over all the nodes to the actual graph. This however introduces
unnecessary complexity, and we can instead just create the inlinee
inside the target graph.

R=jarin@chromium.org

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

[Benedikt Meurer, 2016-05-25 05:21:16]

Hey Jaro,

Here's the inlining cleanup.
Please take a look.

Thanks,
Benedikt

[Jarin, 2016-05-25 06:46:19]

lgtm

[Jarin, 2016-05-25 06:46:45]

Description was changed from

==========
[turbofan] Avoid unnecessary copying of nodes during inlining.

Previously we first created a temporary graph for the inlinee and then
copied over all the nodes to the actual graph. This however introduces
unnecessary complexity, and we can instead just create the inlinee
inside the target graph.

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

to

==========
[turbofan] Avoid unnecessary copying of nodes during inlining.

Previously we first created a temporary graph for the inlinee and then
copied over all the nodes to the actual graph. This however introduces
unnecessary complexity, and we can instead just create the inlinee
inside the target graph.

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

[Benedikt Meurer, 2016-05-25 06:46:52]

The CQ bit was checked by bmeurer@chromium.org

[commit-bot: I haz the power, 2016-05-25 06:46:59]

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

[commit-bot: I haz the power, 2016-05-25 07:24:43]

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

[commit-bot: I haz the power, 2016-05-25 07:24:44]

Try jobs failed on following builders:
  v8_linux_rel_ng on tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux_rel_ng/builds/6288)
  v8_linux_rel_ng_triggered on tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux_rel_ng_triggered/b...)

[Benedikt Meurer, 2016-05-25 09:34:28]

The CQ bit was checked by bmeurer@chromium.org

[Benedikt Meurer, 2016-05-25 09:34:29]

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/2006353003/#ps40001
(title: "Don't use one EmptyFrameState cross functions, that gets mutated during
inlining.")

[commit-bot: I haz the power, 2016-05-25 09:34:35]

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

[commit-bot: I haz the power, 2016-05-25 10:04:40]

Description was changed from

==========
[turbofan] Avoid unnecessary copying of nodes during inlining.

Previously we first created a temporary graph for the inlinee and then
copied over all the nodes to the actual graph. This however introduces
unnecessary complexity, and we can instead just create the inlinee
inside the target graph.

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

to

==========
[turbofan] Avoid unnecessary copying of nodes during inlining.

Previously we first created a temporary graph for the inlinee and then
copied over all the nodes to the actual graph. This however introduces
unnecessary complexity, and we can instead just create the inlinee
inside the target graph.

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

[commit-bot: I haz the power, 2016-05-25 10:04:41]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-05-25 10:07:10]

Description was changed from

==========
[turbofan] Avoid unnecessary copying of nodes during inlining.

Previously we first created a temporary graph for the inlinee and then
copied over all the nodes to the actual graph. This however introduces
unnecessary complexity, and we can instead just create the inlinee
inside the target graph.

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

to

==========
[turbofan] Avoid unnecessary copying of nodes during inlining.

Previously we first created a temporary graph for the inlinee and then
copied over all the nodes to the actual graph. This however introduces
unnecessary complexity, and we can instead just create the inlinee
inside the target graph.

R=jarin@chromium.org

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

[commit-bot: I haz the power, 2016-05-25 10:07:11]

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