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

Unified Diff: SConstruct

Issue 4064: Added a profiling option (prof) to the SCons build. Currently... (Closed) Base URL: http://v8.googlecode.com/svn/branches/bleeding_edge/
Patch Set: Created 12 years, 3 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 | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: SConstruct
===================================================================
--- SConstruct (revision 366)
+++ SConstruct (working copy)
@@ -201,6 +201,9 @@
'library:shared': {
'CPPDEFINES': ['USING_V8_SHARED']
},
+ 'prof:on': {
+ 'LINKFLAGS': ['/MAP']
+ },
'mode:release': {
'CCFLAGS': ['/Ox', '/MT', '/GF'],
'LINKFLAGS': ['/OPT:REF', '/OPT:ICF']
@@ -280,6 +283,11 @@
'default': 'off',
'help': 'build using snapshots for faster start-up'
},
+ 'prof': {
+ 'values': ['on', 'off'],
+ 'default': 'off',
+ 'help': 'enable profiling of build target'
+ },
'library': {
'values': ['static', 'shared'],
'default': 'static',
@@ -337,6 +345,8 @@
return False
if not IsLegal(env, 'sample', ["shell", "process"]):
return False
+ if env['os'] == 'win32' and env['library'] == 'shared' and env['prof'] == 'on':
+ Abort("Profiling on windows only supported for static library.")
for (name, option) in SIMPLE_OPTIONS.iteritems():
if (not option.get('default')) and (name not in ARGUMENTS):
message = ("A value for option %s must be specified (%s)." %
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698