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. |