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

Side by Side Diff: gyp/common_conditions.gypi

Issue 535113004: Disable -fexpensive-optimizations on 64-bit ARM. (Closed) Base URL: https://skia.googlesource.com/skia.git@master
Patch Set: 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 | « 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 # 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 }],
243 [ 'skia_arch_type == "arm" and arm_thumb == 1', { 250 [ 'skia_arch_type == "arm" and arm_thumb == 1', {
244 'cflags': [ 251 'cflags': [
245 '-mthumb', 252 '-mthumb',
246 ], 253 ],
247 # The --fix-cortex-a8 switch enables a link-time workaround for 254 # The --fix-cortex-a8 switch enables a link-time workaround for
248 # an erratum in certain Cortex-A8 processors. The workaround is 255 # an erratum in certain Cortex-A8 processors. The workaround is
249 # enabled by default if you target the ARM v7-A arch profile. 256 # enabled by default if you target the ARM v7-A arch profile.
250 # It can be enabled otherwise by specifying --fix-cortex-a8, or 257 # It can be enabled otherwise by specifying --fix-cortex-a8, or
251 # disabled unconditionally by specifying --no-fix-cortex-a8. 258 # disabled unconditionally by specifying --no-fix-cortex-a8.
252 # 259 #
(...skipping 318 matching lines...) Expand 10 before | Expand all | Expand 10 after
571 # Android defines a fixed gamma exponent instead of using SRGB 578 # Android defines a fixed gamma exponent instead of using SRGB
572 'defines!': [ 579 'defines!': [
573 'SK_GAMMA_SRGB', 580 'SK_GAMMA_SRGB',
574 ], 581 ],
575 'configurations': { 582 'configurations': {
576 'Debug': { 583 'Debug': {
577 'cflags': ['-g'] 584 'cflags': ['-g']
578 }, 585 },
579 'Release': { 586 'Release': {
580 'cflags': ['-O2'], 587 'cflags': ['-O2'],
581 'defines': [ 'NDEBUG' ],
582 }, 588 },
583 }, 589 },
584 'libraries': [ 590 'libraries': [
585 '-llog', 591 '-llog',
586 ], 592 ],
587 'cflags': [ 593 'cflags': [
588 '-fuse-ld=gold', 594 '-fuse-ld=gold',
589 ], 595 ],
590 'conditions': [ 596 'conditions': [
591 [ 'skia_arch_type == "x86"', { 597 [ 'skia_arch_type == "x86"', {
(...skipping 38 matching lines...) Expand 10 before | Expand all | Expand 10 after
630 [ 'skia_is_bot', { 636 [ 'skia_is_bot', {
631 'defines': [ 'SK_CRASH_HANDLER' ], 637 'defines': [ 'SK_CRASH_HANDLER' ],
632 }], 638 }],
633 639
634 ], # end 'conditions' 640 ], # end 'conditions'
635 # The Xcode SYMROOT must be at the root. See build/common.gypi in chromium for more details 641 # The Xcode SYMROOT must be at the root. See build/common.gypi in chromium for more details
636 'xcode_settings': { 642 'xcode_settings': {
637 'SYMROOT': '<(DEPTH)/xcodebuild', 643 'SYMROOT': '<(DEPTH)/xcodebuild',
638 }, 644 },
639 } 645 }
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