Reviewers: Søren Gjesse, Description: Add compilation flags for optimizing samples built in release mode. Remove /Ob2 /Oi /Oy from VS build because they are already enabled by /Ox.
Please review this at http://codereview.chromium.org/4012 Affected files: M SConstruct Index: SConstruct =================================================================== --- SConstruct (revision 350) +++ SConstruct (working copy) @@ -77,7 +77,7 @@ 'LINKFLAGS': ['/DEBUG'] }, 'mode:release': { - 'CCFLAGS': ['/Ox', '/MT', '/Ob2', '/Oi', '/Oy'], + 'CCFLAGS': ['/Ox', '/MT'], 'LINKFLAGS': ['/OPT:REF', '/OPT:ICF'] } } @@ -187,6 +187,9 @@ 'CCFLAGS': ['-m32'], 'LINKFLAGS': ['-m32'] }, + 'mode:release': { + 'CCFLAGS': ['-O2'] + }, 'mode:debug': { 'CCFLAGS': ['-g', '-O0'] } @@ -199,10 +202,12 @@ 'CPPDEFINES': ['USING_V8_SHARED'] }, 'mode:release': { - 'CCFLAGS': ['/MT'], + 'CCFLAGS': ['/Ox', '/MT'], + 'LINKFLAGS': ['/OPT:REF', '/OPT:ICF'] }, 'mode:debug': { - 'CCFLAGS': ['/MTd'] + 'CCFLAGS': ['/Od', '/MTd'], + 'LINKFLAGS': ['/DEBUG'] } } } --~--~---------~--~----~------------~-------~--~----~ v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev -~----------~----~----~----~------~----~------~--~---
