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

Unified Diff: SConstruct

Issue 6529032: Merge 6168:6800 from bleeding_edge to experimental/gc branch. (Closed) Base URL: http://v8.googlecode.com/svn/branches/experimental/gc/
Patch Set: Created 9 years, 10 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 | « MERGE ('k') | include/v8.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: SConstruct
===================================================================
--- SConstruct (revision 6800)
+++ SConstruct (working copy)
@@ -32,7 +32,7 @@
from os.path import join, dirname, abspath
from types import DictType, StringTypes
root_dir = dirname(File('SConstruct').rfile().abspath)
-sys.path.append(join(root_dir, 'tools'))
+sys.path.insert(0, join(root_dir, 'tools'))
import js2c, utils
# ANDROID_TOP is the top of the Android checkout, fetched from the environment
@@ -124,6 +124,13 @@
},
'debuggersupport:on': {
'CPPDEFINES': ['ENABLE_DEBUGGER_SUPPORT'],
+ },
+ 'inspector:on': {
+ 'CPPDEFINES': ['INSPECTOR'],
+ },
+ 'liveobjectlist:on': {
+ 'CPPDEFINES': ['ENABLE_DEBUGGER_SUPPORT', 'INSPECTOR',
+ 'LIVE_OBJECT_LIST', 'OBJECT_PRINT'],
}
},
'gcc': {
@@ -227,8 +234,8 @@
'CCFLAGS': ['-m64'],
'LINKFLAGS': ['-m64'],
},
- 'prof:oprofile': {
- 'CPPDEFINES': ['ENABLE_OPROFILE_AGENT']
+ 'gdbjit:on': {
+ 'CPPDEFINES': ['ENABLE_GDB_JIT_INTERFACE']
}
},
'msvc': {
@@ -320,7 +327,7 @@
},
'msvc': {
'all': {
- 'WARNINGFLAGS': ['/W3', '/WX', '/wd4355', '/wd4800']
+ 'WARNINGFLAGS': ['/W3', '/WX', '/wd4351', '/wd4355', '/wd4800']
},
'library:shared': {
'CPPDEFINES': ['BUILDING_V8_SHARED'],
@@ -529,10 +536,6 @@
'CCFLAGS': ['-g', '-O0'],
'CPPDEFINES': ['DEBUG']
},
- 'prof:oprofile': {
- 'LIBPATH': ['/usr/lib32', '/usr/lib32/oprofile'],
- 'LIBS': ['opagent']
- }
},
'msvc': {
'all': {
@@ -663,6 +666,8 @@
if os == 'win32' and toolchain == 'gcc':
# MinGW can't do it.
return 'default'
+ elif os == 'solaris':
+ return 'default'
else:
return 'hidden'
@@ -700,10 +705,15 @@
'help': 'build using snapshots for faster start-up'
},
'prof': {
- 'values': ['on', 'off', 'oprofile'],
+ 'values': ['on', 'off'],
'default': 'off',
'help': 'enable profiling of build target'
},
+ 'gdbjit': {
+ 'values': ['on', 'off'],
+ 'default': 'off',
+ 'help': 'enable GDB JIT interface'
+ },
'library': {
'values': ['static', 'shared'],
'default': 'static',
@@ -734,6 +744,16 @@
'default': 'on',
'help': 'enable debugging of JavaScript code'
},
+ 'inspector': {
+ 'values': ['on', 'off'],
+ 'default': 'off',
+ 'help': 'enable inspector features'
+ },
+ 'liveobjectlist': {
+ 'values': ['on', 'off'],
+ 'default': 'off',
+ 'help': 'enable live object list features in the debugger'
+ },
'soname': {
'values': ['on', 'off'],
'default': 'off',
@@ -870,8 +890,8 @@
return False
if env['os'] == 'win32' and env['library'] == 'shared' and env['prof'] == 'on':
Abort("Profiling on windows only supported for static library.")
- if env['prof'] == 'oprofile' and env['os'] != 'linux':
- Abort("OProfile is only supported on Linux.")
+ if env['gdbjit'] == 'on' and (env['os'] != 'linux' or (env['arch'] != 'ia32' and env['arch'] != 'x64')):
+ Abort("GDBJIT interface is supported only for Intel-compatible (ia32 or x64) Linux target.")
if env['os'] == 'win32' and env['soname'] == 'on':
Abort("Shared Object soname not applicable for Windows.")
if env['soname'] == 'on' and env['library'] == 'static':
@@ -989,6 +1009,13 @@
# Print a warning if native regexp is specified for mips
print "Warning: forcing regexp to interpreted for mips"
options['regexp'] = 'interpreted'
+ if options['liveobjectlist'] == 'on':
+ if (options['debuggersupport'] != 'on') or (options['mode'] == 'release'):
+ # Print a warning that liveobjectlist will implicitly enable the debugger
+ print "Warning: forcing debuggersupport on for liveobjectlist"
+ options['debuggersupport'] = 'on'
+ options['inspector'] = 'on'
+ options['objectprint'] = 'on'
def ParseEnvOverrides(arg, imports):
« no previous file with comments | « MERGE ('k') | include/v8.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698