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

Unified Diff: gyp/common_variables.gypi

Issue 17370005: Add gyp define for optimization level (Closed) Base URL: http://skia.googlecode.com/svn/trunk/
Patch Set: Created 7 years, 6 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 | « gyp/common_conditions.gypi ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: gyp/common_variables.gypi
===================================================================
--- gyp/common_variables.gypi (revision 9698)
+++ gyp/common_variables.gypi (working copy)
@@ -91,6 +91,11 @@
'skia_profile_enabled%': 0,
'skia_win_debuggers_path%': '',
'skia_shared_lib%': 0,
+
+ # These variables determine the default optimization level for different
+ # compilers.
+ 'skia_default_vs_optimization_level': 3, # full (/Ox)
+ 'skia_default_gcc_optimization_level': 3, # -O3
},
'conditions': [
@@ -101,6 +106,16 @@
}, {
'skia_warnings_as_errors%': 0,
}],
+
+ # This variable allows the user to customize the optimization level used
+ # by the compiler. The user should be aware that this has different
+ # meanings for different compilers and should exercise caution when
+ # overriding it.
+ [ 'skia_os == "win"', {
+ 'skia_release_optimization_level%': '<(skia_default_vs_optimization_level)',
+ }, {
+ 'skia_release_optimization_level%': '<(skia_default_gcc_optimization_level)',
+ }],
],
# Re-define all variables defined within the level-2 'variables' dict,
@@ -128,8 +143,6 @@
'ios_sdk_version%': '6.0',
'skia_win_debuggers_path%': '<(skia_win_debuggers_path)',
- 'skia_cros_target%': '',
-
# These are referenced by our .gypi files that list files (e.g. core.gypi)
#
'skia_src_path%': '../src',
« no previous file with comments | « gyp/common_conditions.gypi ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698