Initial split of precompilation code from compiler.cc

Initial split of precompilation code from compiler.cc

This removes precompiler code from the regular JIT-mode VM, and allows us to add
more precompilation-specific optimizations without affecting the JIT compiler.

Remove JIT-related code from precompiler pipeline:
No type feedback, no OSR, no deopt, no block reordering, no background compilation

With precompilation, use precompiler pipeline for static initializers and const expressions as well.


BUG=
R=rmacnak@google.com

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

[Florian Schneider, 2016-02-03 22:09:54]

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

[Florian Schneider, 2016-02-03 22:09:55]

This only deals with compiler.cc. flow_graph_optimizer.cc will be next.

[rmacnak, 2016-02-04 00:36:39]

dart_precompiled_runtime doesn't link for me.

https://codereview.chromium.org/1663163003/diff/1/runtime/vm/compiler.cc
File runtime/vm/compiler.cc (right):

https://codereview.chromium.org/1663163003/diff/1/runtime/vm/compiler.cc#newc...
runtime/vm/compiler.cc:589: volatile bool use_speculative_inlining = false;
volatile -> const

https://codereview.chromium.org/1663163003/diff/1/runtime/vm/precompiler.cc
File runtime/vm/precompiler.cc (right):

https://codereview.chromium.org/1663163003/diff/1/runtime/vm/precompiler.cc#n...
runtime/vm/precompiler.cc:1748: "SelectRepresentations");
Maybe we should skip these two.

[Florian Schneider, 2016-02-05 01:55:52]

Hm, I can't reproduce the linker error on my Mac/Linux machines.

https://codereview.chromium.org/1663163003/diff/1/runtime/vm/compiler.cc
File runtime/vm/compiler.cc (right):

https://codereview.chromium.org/1663163003/diff/1/runtime/vm/compiler.cc#newc...
runtime/vm/compiler.cc:589: volatile bool use_speculative_inlining = false;
On 2016/02/04 00:36:39, rmacnak wrote:
> volatile -> const

Done.

https://codereview.chromium.org/1663163003/diff/1/runtime/vm/precompiler.cc
File runtime/vm/precompiler.cc (right):

https://codereview.chromium.org/1663163003/diff/1/runtime/vm/precompiler.cc#n...
runtime/vm/precompiler.cc:1748: "SelectRepresentations");
On 2016/02/04 00:36:39, rmacnak wrote:
> Maybe we should skip these two.

I'd rather leave them in for now. They may actually help 32-bit platforms once
we have enough type information.

[rmacnak, 2016-02-08 22:43:23]

lgtm

Builds for me after your latest change

https://codereview.chromium.org/1663163003/diff/1/runtime/vm/precompiler.cc
File runtime/vm/precompiler.cc (right):

https://codereview.chromium.org/1663163003/diff/1/runtime/vm/precompiler.cc#n...
runtime/vm/precompiler.cc:1748: "SelectRepresentations");
On 2016/02/05 01:55:52, Florian Schneider wrote:
> On 2016/02/04 00:36:39, rmacnak wrote:
> > Maybe we should skip these two.
> 
> I'd rather leave them in for now. They may actually help 32-bit platforms once
> we have enough type information.

Acknowledged.

[Florian Schneider, 2016-02-09 10:29:18]

Description was changed from

==========
Initial split of precompilation code from compiler.cc

This removes precompiler code from the regular JIT-mode VM, and allows us to add
more precompilation-specific optimizations without affecting the JIT compiler.

Remove JIT-related code from precompiler pipeline:
No type feedback, no OSR, no deopt, no block reordering, no background
compilation

With precompilation, use precompiler pipeline for static initializers and const
expressions as well.


BUG=
==========

to

==========
Initial split of precompilation code from compiler.cc

This removes precompiler code from the regular JIT-mode VM, and allows us to add
more precompilation-specific optimizations without affecting the JIT compiler.

Remove JIT-related code from precompiler pipeline:
No type feedback, no OSR, no deopt, no block reordering, no background
compilation

With precompilation, use precompiler pipeline for static initializers and const
expressions as well.


BUG=
R=rmacnak@google.com

Committed:
https://github.com/dart-lang/sdk/commit/117c7b0d3eee57e13b7260f4b35dd771d047ee8e
==========

[Florian Schneider, 2016-02-09 10:29:20]

Committed patchset #4 (id:60001) manually as
117c7b0d3eee57e13b7260f4b35dd771d047ee8e (presubmit successful).