VM: [Kernel] Don't emit ReturnInstr that are not connected to the graph.

VM: [Kernel] Don't emit ReturnInstr that are not connected to the graph.

This breaks inlining because it expects that all collected ReturnInstr
are strongly connected to the rest of the graph.

Source based FlowGraphBuilder achieves that by eagerly stopping construction
as constructed fragment is closed (e.g. Throw is emitted).

R=kustermann@google.com
BUG=

Committed: https://github.com/dart-lang/sdk/commit/584788a47c5433ff2db2ffff03a3a33cecbb17ad

[kustermann, 2016-11-24 12:18:22]

lgtm

[Vyacheslav Egorov (Google), 2016-11-24 15:19:55]

Description was changed from

==========
VM: [Kernel] Don't emit ReturnInstr that are not connected to the graph.

This breaks inlining because it expects that all collected ReturnInstr
are strongly connected to the rest of the graph.

Source based FlowGraphBuilder achieves that by eagerly stopping construction
as constructed fragment is closed (e.g. Throw is emitted).

R=kustermann@google.com
BUG=
==========

to

==========
VM: [Kernel] Don't emit ReturnInstr that are not connected to the graph.

This breaks inlining because it expects that all collected ReturnInstr
are strongly connected to the rest of the graph.

Source based FlowGraphBuilder achieves that by eagerly stopping construction
as constructed fragment is closed (e.g. Throw is emitted).

R=kustermann@google.com
BUG=

Committed:
https://github.com/dart-lang/sdk/commit/584788a47c5433ff2db2ffff03a3a33cecbb17ad
==========

[Vyacheslav Egorov (Google), 2016-11-24 15:19:56]

Committed patchset #1 (id:1) manually as
584788a47c5433ff2db2ffff03a3a33cecbb17ad (presubmit successful).