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

Side by Side Diff: build/common.gypi

Issue 159621: Add a way to inject extra cflags in release builds.... (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/
Patch Set: '' Created 11 years, 4 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 315 matching lines...) Expand 10 before | Expand all | Expand 10 after
326 # One can use '-gstabs' to enable building the debugging 326 # One can use '-gstabs' to enable building the debugging
327 # information in STABS format for breakpad's dumpsyms. 327 # information in STABS format for breakpad's dumpsyms.
328 ], 328 ],
329 'ldflags': [ 329 'ldflags': [
330 '-rdynamic', # Allows backtrace to resolve symbols. 330 '-rdynamic', # Allows backtrace to resolve symbols.
331 ], 331 ],
332 }, 332 },
333 'Release': { 333 'Release': {
334 'variables': { 334 'variables': {
335 'release_optimize%': '2', 335 'release_optimize%': '2',
336 'release_extra_cflags%': '',
336 }, 337 },
337 'cflags': [ 338 'cflags': [
338 '-O<(release_optimize)', 339 '-O<(release_optimize)',
339 # Don't emit the GCC version ident directives, they just end up 340 # Don't emit the GCC version ident directives, they just end up
340 # in the .comment section taking up binary size. 341 # in the .comment section taking up binary size.
341 '-fno-ident', 342 '-fno-ident',
342 # Put data and code in their own sections, so that unused symbols 343 # Put data and code in their own sections, so that unused symbols
343 # can be removed at link time with --gc-sections. 344 # can be removed at link time with --gc-sections.
344 '-fdata-sections', 345 '-fdata-sections',
345 '-ffunction-sections', 346 '-ffunction-sections',
347 '<(release_extra_cflags)',
346 ], 348 ],
347 }, 349 },
348 }, 350 },
349 'variants': { 351 'variants': {
350 'coverage': { 352 'coverage': {
351 'cflags': ['-fprofile-arcs', '-ftest-coverage'], 353 'cflags': ['-fprofile-arcs', '-ftest-coverage'],
352 'ldflags': ['-fprofile-arcs'], 354 'ldflags': ['-fprofile-arcs'],
353 }, 355 },
354 'profile': { 356 'profile': {
355 'cflags': ['-pg', '-g'], 357 'cflags': ['-pg', '-g'],
(...skipping 253 matching lines...) Expand 10 before | Expand all | Expand 10 after
609 # settings in target dicts. SYMROOT is a special case, because many other 611 # settings in target dicts. SYMROOT is a special case, because many other
610 # Xcode variables depend on it, including variables such as 612 # Xcode variables depend on it, including variables such as
611 # PROJECT_DERIVED_FILE_DIR. When a source group corresponding to something 613 # PROJECT_DERIVED_FILE_DIR. When a source group corresponding to something
612 # like PROJECT_DERIVED_FILE_DIR is added to a project, in order for the 614 # like PROJECT_DERIVED_FILE_DIR is added to a project, in order for the
613 # files to appear (when present) in the UI as actual files and not red 615 # files to appear (when present) in the UI as actual files and not red
614 # red "missing file" proxies, the correct path to PROJECT_DERIVED_FILE_DIR, 616 # red "missing file" proxies, the correct path to PROJECT_DERIVED_FILE_DIR,
615 # and therefore SYMROOT, needs to be set at the project level. 617 # and therefore SYMROOT, needs to be set at the project level.
616 'SYMROOT': '<(DEPTH)/xcodebuild', 618 'SYMROOT': '<(DEPTH)/xcodebuild',
617 }, 619 },
618 } 620 }
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