Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(585)

Unified Diff: runtime/bin/bin.gypi

Issue 1661703002: VM: Replace dart --noopt with new binary target dart_noopt. (Closed) Base URL: git@github.com:dart-lang/sdk.git@master
Patch Set: addressed comments Created 4 years, 11 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « dart.gyp ('k') | runtime/bin/main.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: runtime/bin/bin.gypi
diff --git a/runtime/bin/bin.gypi b/runtime/bin/bin.gypi
index fa814894d1ccbb61ade71dc50ca33d535adf2bd6..f271fbd0ea70e72b5b0a215fdaedfc4526e5ec6c 100644
--- a/runtime/bin/bin.gypi
+++ b/runtime/bin/bin.gypi
@@ -366,6 +366,7 @@
'defines': [
'DART_SHARED_LIB',
'DART_NO_SNAPSHOT',
+ 'DART_PRECOMPILER',
],
},
{
@@ -628,6 +629,64 @@
},
},
{
+ # dart binary with a snapshot of corelibs built in and support for testing
+ # precompilation (aka --noopt)
+ 'target_name': 'dart_noopt',
+ 'type': 'executable',
+ 'dependencies': [
+ 'libdart_noopt',
+ 'libdart_builtin',
+ 'libdart_io',
+ 'build_observatory#host',
+ 'generate_snapshot_file#host',
+ 'generate_resources_cc_file#host',
+ 'generate_observatory_assets_cc_file#host',
+ ],
+ 'include_dirs': [
+ '..',
+ '../../third_party/', # Zlib
+ ],
+ 'sources': [
+ 'main.cc',
+ 'builtin_common.cc',
+ 'builtin_natives.cc',
+ 'builtin_nolib.cc',
+ 'builtin.h',
+ 'io_natives.h',
+ 'vmservice_impl.cc',
+ 'vmservice_impl.h',
+ '<(snapshot_cc_file)',
+ '<(resources_cc_file)',
+ '<(observatory_assets_cc_file)',
+ ],
+ 'defines': [
+ 'DART_PRECOMPILER',
+ ],
+ 'conditions': [
+ ['OS=="win"', {
+ 'link_settings': {
+ 'libraries': [ '-lws2_32.lib', '-lRpcrt4.lib', '-lwinmm.lib' ],
+ },
+ # Generate an import library on Windows, by exporting a function.
+ # Extensions use this import library to link to the API in dart.exe.
+ 'msvs_settings': {
+ 'VCLinkerTool': {
+ 'AdditionalOptions': [ '/EXPORT:Dart_True' ],
+ },
+ },
+ }],
+ ],
+ 'configurations': {
+ 'Dart_Linux_Base': {
+ # Have the linker add all symbols to the dynamic symbol table
+ # so that extensions can look them up dynamically in the binary.
+ 'ldflags': [
+ '-rdynamic',
+ ],
+ },
+ },
+ },
+ {
# dart binary for running precompiled snapshots without the compiler.
'target_name': 'dart_precompiled_runtime',
'type': 'executable',
@@ -760,6 +819,10 @@
'<(observatory_assets_cc_file)',
'snapshot_empty.cc',
],
+ 'defines': [
+ 'DART_NO_SNAPSHOT',
+ 'DART_PRECOMPILER',
+ ],
'conditions': [
['OS=="win"', {
'link_settings': {
« no previous file with comments | « dart.gyp ('k') | runtime/bin/main.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698