Chromium Code Reviews| Index: base/allocator/allocator.gyp |
| diff --git a/base/allocator/allocator.gyp b/base/allocator/allocator.gyp |
| index d76a31f4cf010fb7ec58c12a084ec8c1794757ef..93e61563443b8fcf651aef316b56a3bb6e3374b3 100644 |
| --- a/base/allocator/allocator.gyp |
| +++ b/base/allocator/allocator.gyp |
| @@ -3,6 +3,28 @@ |
| # found in the LICENSE file. |
| { |
| + 'target_defaults': { |
| + 'variables': { |
| + # This code gets run a lot and debugged rarely, so it should be fast |
| + # by default. See http://crbug.com/388949. |
| + 'debug_optimize': '2', |
| + 'mac_debug_optimization': '2', |
| + # In addition to the above reasons, /Od optimization won't remove symbols |
|
DaleCurtis
2014/08/04 18:58:17
While true, this comment is probably not relevant
willchan no longer on Chromium
2014/08/04 19:05:22
Done.
|
| + # that are under "if (0)" style sections. Which lead to link time errors |
| + # when for example it tries to link an ARM symbol on X86. |
| + 'win_debug_Optimization': '2', |
| + # Run time checks are incompatible with any level of optimizations. |
| + 'win_debug_RuntimeChecks': '0', |
| + 'conditions': [ |
| + ['OS == "win"', { |
| + # Setting the optimizations to 'speed' or to 'max' results in a lot of |
|
DaleCurtis
2014/08/04 18:58:17
I'm not sure you want this setting, it's only need
willchan no longer on Chromium
2014/08/04 19:05:22
Done.
|
| + # unresolved symbols. The only supported mode is 'size' (see |
| + # crbug.com/264459). |
| + 'optimize' :'size', |
| + }], |
| + ], |
| + }, |
| + }, |
| 'variables': { |
| 'tcmalloc_dir': '../../third_party/tcmalloc/chromium', |
| 'use_vtable_verify%': 0, |
| @@ -306,9 +328,6 @@ |
| # e.g. for profiling (it's more rare to profile Debug builds, |
| # but people sometimes need to do that). |
| 'disable_debugallocation%': 0, |
| - # This code gets run a lot and debugged rarely, so it should be fast |
| - # by default. See http://crbug.com/388949. |
| - 'debug_optimize': 2, |
| }, |
| 'conditions': [ |
| # TODO(phajdan.jr): Also enable on Windows. |