VM: Don't depend on unoptimized code when inlining and creating deoptimization info.

VM: Don't depend on unoptimized code when inlining and creating deoptimization info.

This is the first of two steps of removing the requirement to have unoptimized
code when optimizing a function.  This would allow optimizing a function without
generating unoptimized code first.

The only remaining dependency is block reordering (BlockScheduler::AssignWeights) since
the edge counters are currently found in the PC descriptors array associated with
unoptimized code.

BUG=
R=asiva@google.com, iposva@google.com

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

[Florian Schneider, 2015-08-26 11:10:08]

fschneider@google.com changed reviewers:
+ asiva@google.com, iposva@google.com

[srdjan, 2015-08-26 15:43:48]

srdjan@google.com changed reviewers:
+ srdjan@google.com

[srdjan, 2015-08-26 15:43:48]

DBC

https://codereview.chromium.org/1314143002/diff/1/runtime/vm/flow_graph_compi...
File runtime/vm/flow_graph_compiler.cc (right):

https://codereview.chromium.org/1314143002/diff/1/runtime/vm/flow_graph_compi...
runtime/vm/flow_graph_compiler.cc:392: const Script& script =
Script::Handle(instr->env()->function().script());
zone() (and below as well)

[siva, 2015-08-26 17:34:10]

lgtm

[Ivan Posva, 2015-08-26 19:29:10]

LGTM -ip

[Florian Schneider, 2015-08-27 09:32:31]

https://codereview.chromium.org/1314143002/diff/1/runtime/vm/flow_graph_compi...
File runtime/vm/flow_graph_compiler.cc (right):

https://codereview.chromium.org/1314143002/diff/1/runtime/vm/flow_graph_compi...
runtime/vm/flow_graph_compiler.cc:392: const Script& script =
Script::Handle(instr->env()->function().script());
On 2015/08/26 15:43:48, srdjan wrote:
> zone() (and below as well)

Done.

[Florian Schneider, 2015-08-27 09:32:55]

Committed patchset #2 (id:20001) manually as
bad2786f6cb85bfe5fe189a3a972705163a239fa (presubmit successful).