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

Unified Diff: build/common.gypi

Issue 177353002: Add use_allocator instead of linux_use_tcmalloc to switch the allocator. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: ready with use_allocator Created 6 years, 10 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
Index: build/common.gypi
diff --git a/build/common.gypi b/build/common.gypi
index 01e7b0701cf4cf6f6aacfae9950b623a1a18148b..559803dd3cc504275b1d5a9155c30f5a10de8d71 100644
--- a/build/common.gypi
+++ b/build/common.gypi
@@ -753,7 +753,10 @@
['OS=="linux" and target_arch=="arm" and chromeos==0', {
# Set some defaults for arm/linux chrome builds
+ # TODO(dmikurube): Change the default of use_allocator to "none".
+ # TODO(dmikurube): Kill linux_use_tcmalloc. http://crbug.com/345554
'linux_use_tcmalloc%': 0,
+ 'use_allocator%': 'see_use_tcmalloc',
# sysroot needs to be an absolute path otherwise it generates
# incorrect results when passed to pkg-config
'sysroot%': '<!(cd <(DEPTH) && pwd -P)/arm-sysroot',
@@ -1143,8 +1146,14 @@
'release_unwind_tables%': 1,
# Enable TCMalloc.
+ # TODO(dmikurube): Change the default of use_allocator to "tcmalloc".
+ # TODO(dmikurube): Kill linux_use_tcmalloc. http://crbug.com/345554
+ # {linux|android}_use_tcmalloc are to be replaced with use_allocator.
+ # They are now used only if use_allocator=="see_use_tcmalloc" (default).
+ # TODO(dmikurube): Assert when linux_use_tcmalloc is explicitly specified.
'linux_use_tcmalloc%': 1,
'android_use_tcmalloc%': 0,
+ 'use_allocator%': 'see_use_tcmalloc',
# Set to 1 to link against libgnome-keyring instead of using dlopen().
'linux_link_gnome_keyring%': 0,
@@ -1322,7 +1331,10 @@
'werror%': '',
'disable_nacl%': 1,
'nacl_untrusted_build%': 0,
+ # TODO(dmikurube): Change the default of use_allocator to "none".
+ # TODO(dmikurube): Kill linux_use_tcmalloc. http://crbug.com/345554
'linux_use_tcmalloc%': 0,
+ 'use_allocator%': 'see_use_tcmalloc',
}],
['OS=="linux" and target_arch=="mipsel"', {
'sysroot%': '<(sysroot)',
@@ -1477,7 +1489,10 @@
'enable_automation%': 0,
'java_bridge%': 1,
'build_ffmpegsumo%': 0,
+ # TODO(dmikurube): Change the default of use_allocator to "none".
+ # TODO(dmikurube): Kill linux_use_tcmalloc. http://crbug.com/345554
'linux_use_tcmalloc%': 0,
+ 'use_allocator%': 'see_use_tcmalloc',
# Disable Native Client.
'disable_nacl%': 1,
@@ -1872,7 +1887,10 @@
'win_release_InlineFunctionExpansion': '0',
'win_release_OmitFramePointers': '0',
- 'linux_use_tcmalloc': 1,
+ # TODO(dmikurube): Change the default of use_allocator to "tcmalloc".
+ # TODO(dmikurube): Kill linux_use_tcmalloc. http://crbug.com/345554
+ 'linux_use_tcmalloc%': 1,
+ 'use_allocator': 'see_use_tcmalloc',
'release_valgrind_build': 1,
'werror': '',
'component': 'static_library',
@@ -2119,7 +2137,8 @@
'<(DEPTH)/build/mac/asan.gyp:asan_dynamic_runtime',
],
}],
- ['OS=="linux" and linux_use_tcmalloc==1 and clang_type_profiler==1', {
+ # TODO(dmikurube): Kill linux_use_tcmalloc. http://crbug.com/345554
+ ['OS=="linux" and ((use_allocator!="none" and use_allocator!="see_use_tcmalloc") or (use_allocator=="see_use_tcmalloc" and linux_use_tcmalloc==1)) and clang_type_profiler==1', {
'cflags_cc!': ['-fno-rtti'],
'cflags_cc+': [
'-frtti',
@@ -3667,7 +3686,8 @@
}],
],
}],
- ['linux_use_tcmalloc==0 and android_use_tcmalloc==0', {
+ # TODO(dmikurube): Kill linux_use_tcmalloc. http://crbug.com/345554
+ ['(use_allocator!="tcmalloc" and (use_allocator!="see_use_tcmalloc" or linux_use_tcmalloc==0)) and android_use_tcmalloc==0', {
'defines': ['NO_TCMALLOC'],
}],
['linux_use_gold_flags==1', {
« ash/ash.gyp ('K') | « base/security_unittest.cc ('k') | build/gyp_chromium » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698