Skip to content

Commit

Permalink
scons/svga: remove opt from the list of valid build types
Browse files Browse the repository at this point in the history
This reverts commit a5fd54f.

The whole point was to add a way to pass -DVMX86_STATS to the build,
but we can do that with a command line argument when we invoke scons.

Reviewed-by: José Fonseca <[email protected]>
  • Loading branch information
brianpaul committed Oct 26, 2018
1 parent 5bcf479 commit d6be0b5
Show file tree
Hide file tree
Showing 3 changed files with 4 additions and 8 deletions.
2 changes: 1 addition & 1 deletion common.py
Original file line number Diff line number Diff line change
Expand Up @@ -86,7 +86,7 @@ def AddOptions(opts):
from SCons.Options.EnumOption import EnumOption
opts.Add(EnumOption('build', 'build type', 'debug',
allowed_values=('debug', 'checked', 'profile',
'release', 'opt')))
'release')))
opts.Add(BoolOption('verbose', 'verbose output', 'no'))
opts.Add(EnumOption('machine', 'use machine-specific assembly code',
default_machine,
Expand Down
8 changes: 3 additions & 5 deletions scons/gallium.py
Original file line number Diff line number Diff line change
Expand Up @@ -279,7 +279,7 @@ def generate(env):
if env['build'] == 'profile':
env['debug'] = False
env['profile'] = True
if env['build'] in ('release', 'opt'):
if env['build'] == 'release':
env['debug'] = False
env['profile'] = False

Expand Down Expand Up @@ -325,8 +325,6 @@ def generate(env):
cppdefines += ['NDEBUG']
if env['build'] == 'profile':
cppdefines += ['PROFILE']
if env['build'] in ('opt', 'profile'):
cppdefines += ['VMX86_STATS']
if env['platform'] in ('posix', 'linux', 'freebsd', 'darwin'):
cppdefines += [
'_POSIX_SOURCE',
Expand Down Expand Up @@ -477,7 +475,7 @@ def generate(env):
ccflags += [
'/O2', # optimize for speed
]
if env['build'] in ('release', 'opt'):
if env['build'] == 'release':
if not env['clang']:
ccflags += [
'/GL', # enable whole program optimization
Expand Down Expand Up @@ -588,7 +586,7 @@ def generate(env):
shlinkflags += ['-Wl,--enable-stdcall-fixup']
#shlinkflags += ['-Wl,--kill-at']
if msvc:
if env['build'] in ('release', 'opt') and not env['clang']:
if env['build'] == 'release' and not env['clang']:
# enable Link-time Code Generation
linkflags += ['/LTCG']
env.Append(ARFLAGS = ['/LTCG'])
Expand Down
2 changes: 0 additions & 2 deletions src/gallium/drivers/svga/svga_screen.c
Original file line number Diff line number Diff line change
Expand Up @@ -92,8 +92,6 @@ svga_get_name( struct pipe_screen *pscreen )
*/
build = "build: DEBUG;";
mutex = "mutex: " PIPE_ATOMIC ";";
#elif defined(VMX86_STATS)
build = "build: OPT;";
#else
build = "build: RELEASE;";
#endif
Expand Down

0 comments on commit d6be0b5

Please sign in to comment.