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

Unified Diff: tools/gyp/v8.gyp

Issue 5716001: Add gyp target to build preparser as stand-alone library. (Closed)
Patch Set: Created 10 years 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 | « src/v8preparserdll-main.cc ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: tools/gyp/v8.gyp
diff --git a/tools/gyp/v8.gyp b/tools/gyp/v8.gyp
index df8f5473a17924779ae55de4c40815bb51f891ee..39abd7ad8dd16601403fda1fea43f2d20b852ed9 100644
--- a/tools/gyp/v8.gyp
+++ b/tools/gyp/v8.gyp
@@ -30,6 +30,7 @@
'msvs_use_common_release': 0,
'gcc_version%': 'unknown',
'v8_target_arch%': '<(target_arch)',
+ 'host_arch%': '<(target_arch)',
Søren Thygesen Gjesse 2010/12/09 14:08:00 I don't think we should set host_arch here it shou
Lasse Reichstein 2010/12/13 09:19:49 Removed.
'v8_use_snapshot%': 'true',
},
'target_defaults': {
@@ -196,6 +197,43 @@
},
},
{
+ 'target_name': 'v8-preparser',
Søren Thygesen Gjesse 2010/12/09 14:08:00 - -> _, other targets use underscore.
Lasse Reichstein 2010/12/13 09:19:49 Done.
+ 'include_dirs': [
+ '../../include',
+ '../../src',
+ ],
+ 'sources': [
+ '../../src/allocation.cc',
+ '../../src/hashmap.cc',
+ '../../src/preparse-data.cc',
+ '../../src/preparser.cc',
+ '../../src/preparser-api.cc',
+ '../../src/scanner-base.cc',
+ '../../src/token.cc',
+ '../../src/unicode.cc',
+ ],
+ 'conditions': [
+ ['OS=="win" and component=="shared_library"', {
+ 'sources': [ '../../src/v8preparserdll-main.cc' ],
+ 'defines': [ 'BUILDING_V8_SHARED' ],
+ 'direct_dependent_settings': {
+ 'defines': [ 'USING_V8_SHARED' ]
+ },
+ 'type': '<(component)',
+ }],
+ ['OS!="win"', {
+ 'type': '<(library)'
+ }],
+ ['OS=="linux" and library=="shared_library"', {
Søren Thygesen Gjesse 2010/12/09 14:08:00 I also think -fPIC should be provided by the consu
Lasse Reichstein 2010/12/13 09:19:49 Done.
+ 'cflags': ['-fPIC']
+ }],
+ ['v8_target_arch!="x64" and host_arch=="x64"', {
Søren Thygesen Gjesse 2010/12/09 14:08:00 Does the preparser actually care about the v8_targ
Lasse Reichstein 2010/12/13 09:19:49 Removed.
+ 'cflags': ['-m32'],
+ 'ldflags': ['-m32'],
+ }],
+ ]
+ },
+ {
'target_name': 'v8_snapshot',
'type': '<(library)',
'conditions': [
« no previous file with comments | « src/v8preparserdll-main.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698