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

Side by Side Diff: gyp/common_conditions.gypi

Issue 537713002: Revert to enabling all of -O2 as it exposes a bug in the toolchain. (Closed) Base URL: https://skia.googlesource.com/skia.git@master
Patch Set: shorter Created 6 years, 3 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
« no previous file with comments | « gm/tilemodes_scaled.cpp ('k') | 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 # conditions used in both common.gypi and skia.gyp in chromium 1 # conditions used in both common.gypi and skia.gyp in chromium
2 # 2 #
3 { 3 {
4 'defines': [ 4 'defines': [
5 'SK_ALLOW_STATIC_GLOBAL_INITIALIZERS=<(skia_static_initializers)', 5 'SK_ALLOW_STATIC_GLOBAL_INITIALIZERS=<(skia_static_initializers)',
6 'SK_SUPPORT_GPU=<(skia_gpu)', 6 'SK_SUPPORT_GPU=<(skia_gpu)',
7 'SK_SUPPORT_OPENCL=<(skia_opencl)', 7 'SK_SUPPORT_OPENCL=<(skia_opencl)',
8 'SK_FORCE_DISTANCEFIELD_FONTS=<(skia_force_distancefield_fonts)', 8 'SK_FORCE_DISTANCEFIELD_FONTS=<(skia_force_distancefield_fonts)',
9 'SK_PICTURE_USE_SK_RECORD', 9 'SK_PICTURE_USE_SK_RECORD',
10 ], 10 ],
(...skipping 222 matching lines...) Expand 10 before | Expand all | Expand 10 after
233 }], 233 }],
234 # For profiling; reveals some costs, exaggerates others (e.g. trivial setters & getters). 234 # For profiling; reveals some costs, exaggerates others (e.g. trivial setters & getters).
235 [ 'skia_disable_inlining', { 235 [ 'skia_disable_inlining', {
236 'cflags': [ 236 'cflags': [
237 '-fno-inline', 237 '-fno-inline',
238 '-fno-default-inline', 238 '-fno-default-inline',
239 '-finline-limit=0', 239 '-finline-limit=0',
240 '-fno-omit-frame-pointer', 240 '-fno-omit-frame-pointer',
241 ], 241 ],
242 }], 242 }],
243 [ 'skia_arch_type == "arm64"', {
244 # this flag causes tiling errors with GCC 4.9 (pre-release) toolchai n
245 # see skbug.com/
246 'cflags': [
247 '-fno-expensive-optimizations'
248 ],
249 }],
250 [ 'skia_arch_type == "arm" and arm_thumb == 1', { 243 [ 'skia_arch_type == "arm" and arm_thumb == 1', {
251 'cflags': [ 244 'cflags': [
252 '-mthumb', 245 '-mthumb',
253 ], 246 ],
254 # The --fix-cortex-a8 switch enables a link-time workaround for 247 # The --fix-cortex-a8 switch enables a link-time workaround for
255 # an erratum in certain Cortex-A8 processors. The workaround is 248 # an erratum in certain Cortex-A8 processors. The workaround is
256 # enabled by default if you target the ARM v7-A arch profile. 249 # enabled by default if you target the ARM v7-A arch profile.
257 # It can be enabled otherwise by specifying --fix-cortex-a8, or 250 # It can be enabled otherwise by specifying --fix-cortex-a8, or
258 # disabled unconditionally by specifying --no-fix-cortex-a8. 251 # disabled unconditionally by specifying --no-fix-cortex-a8.
259 # 252 #
(...skipping 376 matching lines...) Expand 10 before | Expand all | Expand 10 after
636 [ 'skia_is_bot', { 629 [ 'skia_is_bot', {
637 'defines': [ 'SK_CRASH_HANDLER' ], 630 'defines': [ 'SK_CRASH_HANDLER' ],
638 }], 631 }],
639 632
640 ], # end 'conditions' 633 ], # end 'conditions'
641 # The Xcode SYMROOT must be at the root. See build/common.gypi in chromium for more details 634 # The Xcode SYMROOT must be at the root. See build/common.gypi in chromium for more details
642 'xcode_settings': { 635 'xcode_settings': {
643 'SYMROOT': '<(DEPTH)/xcodebuild', 636 'SYMROOT': '<(DEPTH)/xcodebuild',
644 }, 637 },
645 } 638 }
OLDNEW
« no previous file with comments | « gm/tilemodes_scaled.cpp ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698