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

Side by Side Diff: base/allocator/allocator.gyp

Issue 437163006: base/allocator: Hardcode TCMalloc optimization level to 2 on Windows. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Address Dale's comments. Also ditch mac settings. Created 6 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) 2012 The Chromium Authors. All rights reserved. 1 # Copyright (c) 2012 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 'target_defaults': {
7 'variables': {
8 # This code gets run a lot and debugged rarely, so it should be fast
9 # by default. See http://crbug.com/388949.
10 'debug_optimize': '2',
11 'win_debug_Optimization': '2',
12 # Run time checks are incompatible with any level of optimizations.
13 'win_debug_RuntimeChecks': '0',
14 },
15 },
6 'variables': { 16 'variables': {
7 'tcmalloc_dir': '../../third_party/tcmalloc/chromium', 17 'tcmalloc_dir': '../../third_party/tcmalloc/chromium',
8 'use_vtable_verify%': 0, 18 'use_vtable_verify%': 0,
9 }, 19 },
10 'targets': [ 20 'targets': [
11 # Only executables and not libraries should depend on the 21 # Only executables and not libraries should depend on the
12 # allocator target; only the application (the final executable) 22 # allocator target; only the application (the final executable)
13 # knows what allocator makes sense. 23 # knows what allocator makes sense.
14 { 24 {
15 'target_name': 'allocator', 25 'target_name': 'allocator',
(...skipping 283 matching lines...) Expand 10 before | Expand all | Expand 10 after
299 'msvs_settings': { 309 'msvs_settings': {
300 'VCCLCompilerTool': { 310 'VCCLCompilerTool': {
301 'RuntimeLibrary': '0', 311 'RuntimeLibrary': '0',
302 }, 312 },
303 }, 313 },
304 'variables': { 314 'variables': {
305 # Provide a way to force disable debugallocation in Debug builds, 315 # Provide a way to force disable debugallocation in Debug builds,
306 # e.g. for profiling (it's more rare to profile Debug builds, 316 # e.g. for profiling (it's more rare to profile Debug builds,
307 # but people sometimes need to do that). 317 # but people sometimes need to do that).
308 'disable_debugallocation%': 0, 318 'disable_debugallocation%': 0,
309 # This code gets run a lot and debugged rarely, so it should be fast
310 # by default. See http://crbug.com/388949.
311 'debug_optimize': 2,
312 }, 319 },
313 'conditions': [ 320 'conditions': [
314 # TODO(phajdan.jr): Also enable on Windows. 321 # TODO(phajdan.jr): Also enable on Windows.
315 ['disable_debugallocation==0 and OS!="win"', { 322 ['disable_debugallocation==0 and OS!="win"', {
316 'defines': [ 323 'defines': [
317 # Use debugallocation for Debug builds to catch problems early 324 # Use debugallocation for Debug builds to catch problems early
318 # and cleanly, http://crbug.com/30715 . 325 # and cleanly, http://crbug.com/30715 .
319 'TCMALLOC_FOR_DEBUGALLOCATION', 326 'TCMALLOC_FOR_DEBUGALLOCATION',
320 ], 327 ],
321 }], 328 }],
(...skipping 301 matching lines...) Expand 10 before | Expand all | Expand 10 after
623 'sources': [ 630 'sources': [
624 'type_profiler_map_unittest.cc', 631 'type_profiler_map_unittest.cc',
625 '<(tcmalloc_dir)/src/gperftools/type_profiler_map.h', 632 '<(tcmalloc_dir)/src/gperftools/type_profiler_map.h',
626 '<(tcmalloc_dir)/src/type_profiler_map.cc', 633 '<(tcmalloc_dir)/src/type_profiler_map.cc',
627 ], 634 ],
628 }, 635 },
629 ], 636 ],
630 }], 637 }],
631 ], 638 ],
632 } 639 }
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