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

Side by Side Diff: build/common.gypi

Issue 155501: Add a variable that forces the release mac builds to use -O1 instead of -Os f... (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/
Patch Set: '' Created 11 years, 5 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 unified diff | Download patch | Annotate | Revision Log
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 # Copyright (c) 2009 The Chromium Authors. All rights reserved. 1 # Copyright (c) 2009 The Chromium Authors. All rights reserved.
2 # Use of this source code is governed by a BSD-style license that can be 2 # Use of this source code is governed by a BSD-style license that can be
3 # found in the LICENSE file. 3 # found in the LICENSE file.
4 4
5 { 5 {
6 'variables': { 6 'variables': {
7 # .gyp files should set chromium_code to 1 if they build Chromium-specific 7 # .gyp files should set chromium_code to 1 if they build Chromium-specific
8 # code, as opposed to external code. This variable is used to control 8 # code, as opposed to external code. This variable is used to control
9 # such things as the set of warnings to enable, and whether warnings are 9 # such things as the set of warnings to enable, and whether warnings are
10 # treated as errors. 10 # treated as errors.
(...skipping 173 matching lines...) Expand 10 before | Expand all | Expand 10 after
184 ], 184 ],
185 }, 185 },
186 'Release': { 186 'Release': {
187 'defines': [ 187 'defines': [
188 'NDEBUG', 188 'NDEBUG',
189 ], 189 ],
190 'conditions': [ 190 'conditions': [
191 [ 'OS=="mac"', { 191 [ 'OS=="mac"', {
192 'xcode_settings': { 192 'xcode_settings': {
193 'DEAD_CODE_STRIPPING': 'YES', 193 'DEAD_CODE_STRIPPING': 'YES',
194 'conditions': [
195 ['mac_release_optimization != "UNSET"',
196 {'GCC_OPTIMIZATION_LEVEL': '<(mac_release_optimization)'}],
197 ],
194 } 198 }
195 }], 199 }],
196 [ 'OS=="win" and msvs_use_common_release', { 200 [ 'OS=="win" and msvs_use_common_release', {
197 'configuration_platform': 'Win32', 201 'configuration_platform': 'Win32',
198 'msvs_props': ['release.vsprops'], 202 'msvs_props': ['release.vsprops'],
199 }], 203 }],
200 [ 'OS=="win"', { 204 [ 'OS=="win"', {
201 'msvs_settings': { 205 'msvs_settings': {
202 'VCLinkerTool': { 206 'VCLinkerTool': {
203 'LinkIncremental': '1', 207 'LinkIncremental': '1',
(...skipping 188 matching lines...) Expand 10 before | Expand all | Expand 10 after
392 ], 396 ],
393 }, 397 },
394 }], 398 }],
395 ['OS=="mac"', { 399 ['OS=="mac"', {
396 'target_defaults': { 400 'target_defaults': {
397 'variables': { 401 'variables': {
398 # This should be 'mac_real_dsym%', but there seems to be a bug 402 # This should be 'mac_real_dsym%', but there seems to be a bug
399 # with % in variables that are intended to be set to different 403 # with % in variables that are intended to be set to different
400 # values in different targets, like this one. 404 # values in different targets, like this one.
401 'mac_real_dsym': 0, # Fake .dSYMs are fine in most cases. 405 'mac_real_dsym': 0, # Fake .dSYMs are fine in most cases.
406 # Release defaults to the Xcode optimization default, this var
407 # lets you force the value.
408 'mac_release_optimization%': 'UNSET'
402 }, 409 },
403 'mac_bundle': 0, 410 'mac_bundle': 0,
404 'xcode_settings': { 411 'xcode_settings': {
405 'ALWAYS_SEARCH_USER_PATHS': 'NO', 412 'ALWAYS_SEARCH_USER_PATHS': 'NO',
406 'GCC_C_LANGUAGE_STANDARD': 'c99', 413 'GCC_C_LANGUAGE_STANDARD': 'c99',
407 'GCC_CW_ASM_SYNTAX': 'NO', 414 'GCC_CW_ASM_SYNTAX': 'NO',
408 'GCC_DYNAMIC_NO_PIC': 'NO', 415 'GCC_DYNAMIC_NO_PIC': 'NO',
409 'GCC_ENABLE_PASCAL_STRINGS': 'NO', 416 'GCC_ENABLE_PASCAL_STRINGS': 'NO',
410 'GCC_INLINES_ARE_PRIVATE_EXTERN': 'YES', 417 'GCC_INLINES_ARE_PRIVATE_EXTERN': 'YES',
411 'GCC_OBJC_CALL_CXX_CDTORS': 'YES', 418 'GCC_OBJC_CALL_CXX_CDTORS': 'YES',
(...skipping 176 matching lines...) Expand 10 before | Expand all | Expand 10 after
588 # settings in target dicts. SYMROOT is a special case, because many other 595 # settings in target dicts. SYMROOT is a special case, because many other
589 # Xcode variables depend on it, including variables such as 596 # Xcode variables depend on it, including variables such as
590 # PROJECT_DERIVED_FILE_DIR. When a source group corresponding to something 597 # PROJECT_DERIVED_FILE_DIR. When a source group corresponding to something
591 # like PROJECT_DERIVED_FILE_DIR is added to a project, in order for the 598 # like PROJECT_DERIVED_FILE_DIR is added to a project, in order for the
592 # files to appear (when present) in the UI as actual files and not red 599 # files to appear (when present) in the UI as actual files and not red
593 # red "missing file" proxies, the correct path to PROJECT_DERIVED_FILE_DIR, 600 # red "missing file" proxies, the correct path to PROJECT_DERIVED_FILE_DIR,
594 # and therefore SYMROOT, needs to be set at the project level. 601 # and therefore SYMROOT, needs to be set at the project level.
595 'SYMROOT': '<(DEPTH)/xcodebuild', 602 'SYMROOT': '<(DEPTH)/xcodebuild',
596 }, 603 },
597 } 604 }
OLDNEW
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698