Author: [EMAIL PROTECTED]
Date: Tue Sep 23 05:32:43 2008
New Revision: 365

Modified:
    branches/bleeding_edge/SConstruct
    branches/bleeding_edge/tools/visual_studio/release.vsprops

Log:
Enable string pooling in the VS release build.  This saves 46k for a  
v8_shell_sample.exe, I didn't measure performance or the overall change to  
a Chromium build.  I think this option should always make sense anyway,  
unless we're using some string constants incorrectly.

Modified: branches/bleeding_edge/SConstruct
==============================================================================
--- branches/bleeding_edge/SConstruct   (original)
+++ branches/bleeding_edge/SConstruct   Tue Sep 23 05:32:43 2008
@@ -77,7 +77,7 @@
        'LINKFLAGS':    ['/DEBUG']
      },
      'mode:release': {
-      'CCFLAGS':      ['/Ox', '/MT'],
+      'CCFLAGS':      ['/Ox', '/MT', '/GF'],
        'LINKFLAGS':    ['/OPT:REF', '/OPT:ICF']
      }
    }
@@ -202,7 +202,7 @@
        'CPPDEFINES': ['USING_V8_SHARED']
      },
      'mode:release': {
-      'CCFLAGS':   ['/Ox', '/MT'],
+      'CCFLAGS':   ['/Ox', '/MT', '/GF'],
        'LINKFLAGS': ['/OPT:REF', '/OPT:ICF']
      },
      'mode:debug': {

Modified: branches/bleeding_edge/tools/visual_studio/release.vsprops
==============================================================================
--- branches/bleeding_edge/tools/visual_studio/release.vsprops  (original)
+++ branches/bleeding_edge/tools/visual_studio/release.vsprops  Tue Sep 23  
05:32:43 2008
@@ -12,6 +12,7 @@
                EnableIntrinsicFunctions="true"
                FavorSizeOrSpeed="0"
                OmitFramePointers="true"
+               StringPooling="true"
        />
        <Tool
                Name="VCLinkerTool"

--~--~---------~--~----~------------~-------~--~----~
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev
-~----------~----~----~----~------~----~------~--~---

Reply via email to