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

Side by Side Diff: gyp/common_conditions.gypi

Issue 163363004: Gyp flag to disable inlining. (Closed) Base URL: http://skia.googlecode.com/svn/trunk/
Patch Set: Created 6 years, 10 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 | « gm/shadertext2.cpp ('k') | gyp/common_variables.gypi » ('j') | 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_DISTANCEFIELD_FONTS=<(skia_distancefield_fonts)', 8 'SK_DISTANCEFIELD_FONTS=<(skia_distancefield_fonts)',
9 ], 9 ],
10 'conditions' : [ 10 'conditions' : [
(...skipping 124 matching lines...) Expand 10 before | Expand all | Expand 10 after
135 'cflags_cc': [ 135 'cflags_cc': [
136 '-fno-rtti', 136 '-fno-rtti',
137 '-Wnon-virtual-dtor', 137 '-Wnon-virtual-dtor',
138 ], 138 ],
139 'conditions': [ 139 'conditions': [
140 [ 'skia_warnings_as_errors', { 140 [ 'skia_warnings_as_errors', {
141 'cflags': [ 141 'cflags': [
142 '-Werror', 142 '-Werror',
143 ], 143 ],
144 }], 144 }],
145 [ 'skia_disable_inlining', {
scroggo 2014/02/13 17:46:18 Why does this depend on being on a linux-ish OS?
mtklein 2014/02/13 17:55:39 Maybe tack on # Handy for profiling. ?
146 'cflags': [
147 '-fno-inline',
148 '-fno-default-inline',
149 '-finline-limit=0',
150 '-fno-omit-frame-pointer',
151 ],
152 }],
145 [ 'skia_arch_type == "arm" and arm_thumb == 1', { 153 [ 'skia_arch_type == "arm" and arm_thumb == 1', {
146 'cflags': [ 154 'cflags': [
147 '-mthumb', 155 '-mthumb',
148 ], 156 ],
149 # The --fix-cortex-a8 switch enables a link-time workaround for 157 # The --fix-cortex-a8 switch enables a link-time workaround for
150 # an erratum in certain Cortex-A8 processors. The workaround is 158 # an erratum in certain Cortex-A8 processors. The workaround is
151 # enabled by default if you target the ARM v7-A arch profile. 159 # enabled by default if you target the ARM v7-A arch profile.
152 # It can be enabled otherwise by specifying --fix-cortex-a8, or 160 # It can be enabled otherwise by specifying --fix-cortex-a8, or
153 # disabled unconditionally by specifying --no-fix-cortex-a8. 161 # disabled unconditionally by specifying --no-fix-cortex-a8.
154 # 162 #
(...skipping 347 matching lines...) Expand 10 before | Expand all | Expand 10 after
502 'defines': [ 510 'defines': [
503 'SK_USE_POSIX_THREADS', 511 'SK_USE_POSIX_THREADS',
504 ], 512 ],
505 }], 513 }],
506 ], # end 'conditions' 514 ], # end 'conditions'
507 # The Xcode SYMROOT must be at the root. See build/common.gypi in chromium for more details 515 # The Xcode SYMROOT must be at the root. See build/common.gypi in chromium for more details
508 'xcode_settings': { 516 'xcode_settings': {
509 'SYMROOT': '<(DEPTH)/xcodebuild', 517 'SYMROOT': '<(DEPTH)/xcodebuild',
510 }, 518 },
511 } 519 }
OLDNEW
« no previous file with comments | « gm/shadertext2.cpp ('k') | gyp/common_variables.gypi » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698