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

Unified Diff: build/common.gypi

Issue 13411003: Introduce fastbuild=2, which completely disables debug information. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: fixes Created 7 years, 9 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 | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: build/common.gypi
diff --git a/build/common.gypi b/build/common.gypi
index 49551ffabcc1301f102b64cefc0a6c3ce5543b68..6b6f0279d9054b65a3f5bc7b17936b3c2953191c 100644
--- a/build/common.gypi
+++ b/build/common.gypi
@@ -186,8 +186,9 @@
# Override branding to select the desired branding flavor.
'branding%': 'Chromium',
- # Set to 1 to enable fast builds. It disables debug info for fastest
- # compilation.
+ # Set to 1 to enable fast builds. Set to 2 for even faster builds
+ # (it disables debug info for fastest compilation - only for use
+ # on compile-only bots).
'fastbuild%': 0,
# Set to 1 to enable dcheck in release without having to use the flag.
@@ -1798,11 +1799,11 @@
'defines': ['ENABLE_HIDPI=1'],
}],
['fastbuild!=0', {
- # Clang creates chubby debug information, which makes linking very
- # slow. For now, don't create debug information with clang. See
- # http://crbug.com/70000
'conditions': [
['clang==1', {
+ # Clang creates chubby debug information, which makes linking very
+ # slow. For now, don't create debug information with clang. See
+ # http://crbug.com/70000
'conditions': [
['OS=="linux"', {
'variables': {
@@ -1819,8 +1820,18 @@
],
}, { # else clang!=1
'conditions': [
- # For Windows and Mac, we don't genererate debug information.
- ['OS=="win"', {
+ ['OS=="win" and fastbuild==2', {
+ # Completely disable debug information.
+ 'msvs_settings': {
+ 'VCLinkerTool': {
+ 'GenerateDebugInformation': 'false',
+ },
+ 'VCCLCompilerTool': {
+ 'DebugInformationFormat': '0',
+ },
+ },
+ }],
+ ['OS=="win" and fastbuild==1', {
'msvs_settings': {
'VCLinkerTool': {
# This tells the linker to generate .pdbs, so that
@@ -1838,12 +1849,23 @@
'GCC_GENERATE_DEBUGGING_SYMBOLS': 'NO',
},
}],
- ['OS=="linux"', {
+ ['OS=="linux" and fastbuild==2', {
+ 'variables': {
+ 'debug_extra_cflags': '-g0',
+ },
+ }],
+ ['OS=="linux" and fastbuild==1', {
'variables': {
'debug_extra_cflags': '-g1',
},
}],
- ['OS=="android"', {
+ ['OS=="android" and fastbuild==2', {
+ 'variables': {
+ 'debug_extra_cflags': '-g0',
+ 'release_extra_cflags': '-g0',
+ },
+ }],
+ ['OS=="android" and fastbuild==1', {
'variables': {
'debug_extra_cflags': '-g1',
'release_extra_cflags': '-g1',
« 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