Add libdart_precompiled_runtime target to runtime/BUILD.gn

Add libdart_precompiled_runtime target to runtime/BUILD.gn

This allows the embedder using the gn build files build a VM runtime without compiler-related code, used to run a precompiled snapshot.

It is a copy of libdart, but has the macro DART_PRECOMPILED_RUNTIME defined which excludes all
 compiler-related code, not needed to run precompiled code from the binary.


BUG=
R=chinmaygarde@google.com

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

[Florian Schneider, 2016-02-24 21:47:11]

Description was changed from

==========
Add libdart_precompiled_runtime target to runtime/BUILD.gn

It has the macro DART_PRECOMPILED_RUNTIME defined which excludes all
 compiler-related code, not needed to run precompiled code from the binary.

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

to

==========
Add libdart_precompiled_runtime target to runtime/BUILD.gn

It is a copy of libdart, but has the macro DART_PRECOMPILED_RUNTIME defined
which excludes all
 compiler-related code, not needed to run precompiled code from the binary.

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

[Florian Schneider, 2016-02-24 21:48:49]

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

[Florian Schneider, 2016-02-24 21:56:44]

Description was changed from

==========
Add libdart_precompiled_runtime target to runtime/BUILD.gn

It is a copy of libdart, but has the macro DART_PRECOMPILED_RUNTIME defined
which excludes all
 compiler-related code, not needed to run precompiled code from the binary.

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

to

==========
Add libdart_precompiled_runtime target to runtime/BUILD.gn

It is a copy of libdart, but has the macro DART_PRECOMPILED_RUNTIME defined
which excludes all
 compiler-related code, not needed to run precompiled code from the binary.

In the standalone x64 VM this saves around 3M from the VM binary size (dart vs.
dart_precompiled_runtime)

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

[rmacnak, 2016-02-25 00:42:46]

https://codereview.chromium.org/1738503002/diff/1/runtime/BUILD.gn
File runtime/BUILD.gn (left):

https://codereview.chromium.org/1738503002/diff/1/runtime/BUILD.gn#oldcode53
runtime/BUILD.gn:53: if (is_ios || is_mac) {
We can't remove this until there is another way to build the snapshot generator.

[Florian Schneider, 2016-02-25 17:30:45]

Description was changed from

==========
Add libdart_precompiled_runtime target to runtime/BUILD.gn

It is a copy of libdart, but has the macro DART_PRECOMPILED_RUNTIME defined
which excludes all
 compiler-related code, not needed to run precompiled code from the binary.

In the standalone x64 VM this saves around 3M from the VM binary size (dart vs.
dart_precompiled_runtime)

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

to

==========
Add libdart_precompiled_runtime target to runtime/BUILD.gn

It is a copy of libdart, but has the macro DART_PRECOMPILED_RUNTIME defined
which excludes all
 compiler-related code, not needed to run precompiled code from the binary.

This allows the embedder using the gn build files build a VM runtime without
compiler-related code, used to run a precompiled snapshot.

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

[Florian Schneider, 2016-02-25 17:31:01]

Description was changed from

==========
Add libdart_precompiled_runtime target to runtime/BUILD.gn

It is a copy of libdart, but has the macro DART_PRECOMPILED_RUNTIME defined
which excludes all
 compiler-related code, not needed to run precompiled code from the binary.

This allows the embedder using the gn build files build a VM runtime without
compiler-related code, used to run a precompiled snapshot.

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

to

==========
Add libdart_precompiled_runtime target to runtime/BUILD.gn

This allows the embedder using the gn build files build a VM runtime without
compiler-related code, used to run a precompiled snapshot.

It is a copy of libdart, but has the macro DART_PRECOMPILED_RUNTIME defined
which excludes all
 compiler-related code, not needed to run precompiled code from the binary.


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

[Florian Schneider, 2016-02-25 17:31:24]

Patchset #2 (id:20001) has been deleted

[Florian Schneider, 2016-02-25 17:31:33]

Patchset #2 (id:40001) has been deleted

[Florian Schneider, 2016-02-25 17:31:39]

Patchset #2 (id:60001) has been deleted

[Florian Schneider, 2016-02-25 17:32:06]

ptal

[Chinmay, 2016-02-25 21:27:23]

On 2016/02/25 at 17:32:06, fschneider wrote:
> ptal

This looks good to me and I see the savings in size of the final binary. I am
sure there is an easier way to make the configs inherit one another for a
clearer GN file. But we can figure it out in a later patch if you want. lgtm
(but I am not an owner or anything)

[Florian Schneider, 2016-02-25 21:44:32]

Description was changed from

==========
Add libdart_precompiled_runtime target to runtime/BUILD.gn

This allows the embedder using the gn build files build a VM runtime without
compiler-related code, used to run a precompiled snapshot.

It is a copy of libdart, but has the macro DART_PRECOMPILED_RUNTIME defined
which excludes all
 compiler-related code, not needed to run precompiled code from the binary.


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

to

==========
Add libdart_precompiled_runtime target to runtime/BUILD.gn

This allows the embedder using the gn build files build a VM runtime without
compiler-related code, used to run a precompiled snapshot.

It is a copy of libdart, but has the macro DART_PRECOMPILED_RUNTIME defined
which excludes all
 compiler-related code, not needed to run precompiled code from the binary.


BUG=
R=chinmaygarde@google.com

Committed:
https://github.com/dart-lang/sdk/commit/289157044a21ea999df9277095dba720848a8bfe
==========

[Florian Schneider, 2016-02-25 21:44:33]

Committed patchset #2 (id:80001) manually as
289157044a21ea999df9277095dba720848a8bfe (presubmit successful).

[Florian Schneider, 2016-02-25 21:45:05]

On 2016/02/25 21:27:23, Chinmay wrote:
> On 2016/02/25 at 17:32:06, fschneider wrote:
> > ptal
> 
> This looks good to me and I see the savings in size of the final binary. I am
> sure there is an easier way to make the configs inherit one another for a
> clearer GN file. But we can figure it out in a later patch if you want. lgtm
> (but I am not an owner or anything)

Thanks for testing it! I'll land this as is for now and try to simplify the gn
files in a separate CL.