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

Issue 5664: Changed optimization from /Ox to the (more) commonly used /O2 (which includes... (Closed)

Created:
12 years, 2 months ago by Søren Thygesen Gjesse
Modified:
9 years, 7 months ago
CC:
v8-dev
Visibility:
Public.

Description

Changed optimization from /Ox to the (more) commonly used /O2 (which includes /GF) in both SCons and Visual Studio build. There is not mesurable difference neither in speed nor in code size. Added whole program optimization to the SCons release build. This clutters the linking of the samples a little with the option /LTCG. This option is not strictly needed for linking, but if it is not specified the linker complaints as it can see object files compiled with /GL and then restarts itself with /LTCG which dosen't look very pretty. Committed: http://code.google.com/p/v8/source/detail?r=407

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+6 lines, -5 lines) Patch
M SConstruct View 2 chunks +5 lines, -4 lines 0 comments Download
M tools/visual_studio/release.vsprops View 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 1 (0 generated)
Søren Thygesen Gjesse
12 years, 2 months ago (2008-10-02 08:44:21 UTC) #1

          

Powered by Google App Engine
This is Rietveld 408576698