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

Side by Side Diff: gyp/common_conditions.gypi

Issue 1919873003: Make clang the default Android compiler. (Closed) Base URL: https://skia.googlesource.com/skia.git@master
Patch Set: Created 4 years, 7 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 | gyp/freetype.gyp » ('j') | gyp/freetype.gyp » ('J')
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 # Copyright 2015 Google Inc. 1 # Copyright 2015 Google Inc.
2 # 2 #
3 # Use of this source code is governed by a BSD-style license that can be 3 # Use of this source code is governed by a BSD-style license that can be
4 # found in the LICENSE file. 4 # found in the LICENSE file.
5 5
6 # conditions used in both common.gypi and skia.gyp in chromium 6 # conditions used in both common.gypi and skia.gyp in chromium
7 # 7 #
8 { 8 {
9 'defines': [ 9 'defines': [
10 'SK_ALLOW_STATIC_GLOBAL_INITIALIZERS=<(skia_static_initializers)', 10 'SK_ALLOW_STATIC_GLOBAL_INITIALIZERS=<(skia_static_initializers)',
(...skipping 203 matching lines...) Expand 10 before | Expand all | Expand 10 after
214 }, 214 },
215 }], 215 }],
216 ], 216 ],
217 }, 217 },
218 ], 218 ],
219 219
220 # The following section is common to linux + derivatives and android 220 # The following section is common to linux + derivatives and android
221 [ 'skia_os in ["linux", "freebsd", "openbsd", "solaris", "chromeos", "androi d"]', 221 [ 'skia_os in ["linux", "freebsd", "openbsd", "solaris", "chromeos", "androi d"]',
222 { 222 {
223 'cflags': [ 223 'cflags': [
224 '-g', 224 '-g',
djsollen 2016/04/25 18:18:26 yeah, it was defined here.
225 '-fno-exceptions', 225 '-fno-exceptions',
226 '-fstrict-aliasing', 226 '-fstrict-aliasing',
227 227
228 '-Wall', 228 '-Wall',
229 '-Wextra', 229 '-Wextra',
230 '-Winit-self', 230 '-Winit-self',
231 '-Wpointer-arith', 231 '-Wpointer-arith',
232 '-Wsign-compare', 232 '-Wsign-compare',
233 233
234 '-Wno-unused-parameter', 234 '-Wno-unused-parameter',
(...skipping 378 matching lines...) Expand 10 before | Expand all | Expand 10 after
613 613
614 # Android Text Tuning 614 # Android Text Tuning
615 'SK_GAMMA_EXPONENT=1.4', 615 'SK_GAMMA_EXPONENT=1.4',
616 'SK_GAMMA_CONTRAST=0.0', 616 'SK_GAMMA_CONTRAST=0.0',
617 ], 617 ],
618 # Android defines a fixed gamma exponent instead of using SRGB 618 # Android defines a fixed gamma exponent instead of using SRGB
619 'defines!': [ 619 'defines!': [
620 'SK_GAMMA_SRGB', 620 'SK_GAMMA_SRGB',
621 ], 621 ],
622 'configurations': { 622 'configurations': {
623 'Debug': {
624 'cflags': ['-g']
mtklein 2016/04/25 17:44:17 Was this redundant?
625 },
626 'Release': { 623 'Release': {
627 'cflags': ['-O2'], 624 'cflags': ['-O2'],
625 'conditions': [
626 [ 'skia_clang_build', {
627 'cflags!': ['-g'],
628 'cflags': [ '-gline-tables-only' ],
629 }],
630 ],
628 }, 631 },
629 }, 632 },
630 'libraries': [ 633 'libraries': [
631 '-llog', 634 '-llog',
632 ], 635 ],
633 'cflags': [ 636 'cflags': [
634 '-fuse-ld=gold', 637 '-fuse-ld=gold',
635 ], 638 ],
636 'conditions': [ 639 'conditions': [
637 [ '"x86" in skia_arch_type', { 640 [ '"x86" in skia_arch_type', {
(...skipping 48 matching lines...) Expand 10 before | Expand all | Expand 10 after
686 'LD_RUNPATH_SEARCH_PATHS': ['@executable_path/.'], 689 'LD_RUNPATH_SEARCH_PATHS': ['@executable_path/.'],
687 }, 690 },
688 }], 691 }],
689 692
690 ], # end 'conditions' 693 ], # end 'conditions'
691 # The Xcode SYMROOT must be at the root. See build/common.gypi in chromium for more details 694 # The Xcode SYMROOT must be at the root. See build/common.gypi in chromium for more details
692 'xcode_settings': { 695 'xcode_settings': {
693 'SYMROOT': '<(DEPTH)/xcodebuild', 696 'SYMROOT': '<(DEPTH)/xcodebuild',
694 }, 697 },
695 } 698 }
OLDNEW
« no previous file with comments | « no previous file | gyp/freetype.gyp » ('j') | gyp/freetype.gyp » ('J')

Powered by Google App Engine
This is Rietveld 408576698