Commit 09169eee authored by brettw@chromium.org's avatar brettw@chromium.org

Optionally use new scheme for optimization configs.

The GN optimization config setup is being moved. This patch allows blink to switch between the new and old ones.

Review URL: https://codereview.chromium.org/1330693002

git-svn-id: svn://svn.chromium.org/blink/trunk@201835 bbb929c8-8fbe-4397-9dbb-9b2b20218538
parent de8f5516
...@@ -22,17 +22,19 @@ rel_blink_core_gen_dir = rebase_path(blink_core_output_dir, root_build_dir) ...@@ -22,17 +22,19 @@ rel_blink_core_gen_dir = rebase_path(blink_core_output_dir, root_build_dir)
# speed-over-size optimization for official builds on Windows only. The GN's # speed-over-size optimization for official builds on Windows only. The GN's
# build optimize_max config applies this optimization on all platforms, so # build optimize_max config applies this optimization on all platforms, so
# compute how to modify the config list to duplicate the GYP behavior. # compute how to modify the config list to duplicate the GYP behavior.
if (is_debug) { if (is_debug || !is_win || !is_official_build) {
core_config_remove = [ "//build/config/compiler:no_optimize" ] # NOP.
core_config_add = core_config_remove # NOP core_config_add = []
core_config_remove = []
} else { } else {
core_config_remove = [ "//build/config/compiler:optimize" ] if (using_new_global_compiler_configs) {
# GN build is moving to a single default optimization config that prevents
if (is_win && is_official_build) { # us from having to break out the debug/release conditions to override.
core_config_add = [ "//build/config/compiler:optimize_max" ] core_config_remove = [ "//build/config/compiler:default_optimization" ]
} else { } else {
core_config_add = core_config_remove # NOP core_config_remove = [ "//build/config/compiler:optimize" ]
} }
core_config_add = [ "//build/config/compiler:optimize_max" ]
} }
# Core targets also get wexit time destructors. # Core targets also get wexit time destructors.
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment