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

Side by Side Diff: skia/skia.gyp

Issue 1289623006: build: On Android, disable LTO for specific targets only when targeting GCC. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Clarify comment in build/common.gypi; add TODO to use -lto_library Created 5 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 | « sandbox/linux/sandbox_linux.gypi ('k') | skia/skia_library_opts.gyp » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 # Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 # Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 # Use of this source code is governed by a BSD-style license that can be 2 # Use of this source code is governed by a BSD-style license that can be
3 # found in the LICENSE file. 3 # found in the LICENSE file.
4 4
5 { 5 {
6 'conditions': [ 6 'conditions': [
7 # In component mode (shared_lib), we build all of skia as a single DLL. 7 # In component mode (shared_lib), we build all of skia as a single DLL.
8 # However, in the static mode, we need to build skia as multiple targets 8 # However, in the static mode, we need to build skia as multiple targets
9 # in order to support the use case where a platform (e.g. Android) may 9 # in order to support the use case where a platform (e.g. Android) may
10 # already have a copy of skia as a system library. 10 # already have a copy of skia as a system library.
11 ['component=="static_library"', { 11 ['component=="static_library"', {
12 'targets': [ 12 'targets': [
13 { 13 {
14 'target_name': 'skia_library', 14 'target_name': 'skia_library',
15 'type': 'static_library', 15 'type': 'static_library',
16 'includes': [ 16 'includes': [
17 'skia_common.gypi', 17 'skia_common.gypi',
18 'skia_library.gypi', 18 'skia_library.gypi',
19 '../build/android/increase_size_for_speed.gypi', 19 '../build/android/increase_size_for_speed.gypi',
20 # Disable LTO due to compiler error 20 # Disable LTO due to compiler error
21 # in mems_in_disjoint_alias_sets_p, at alias.c:393 21 # in mems_in_disjoint_alias_sets_p, at alias.c:393
22 # crbug.com/422255 22 # crbug.com/422255
23 '../build/android/disable_lto.gypi', 23 '../build/android/disable_gcc_lto.gypi',
24 ], 24 ],
25 }, 25 },
26 ], 26 ],
27 }], 27 }],
28 ['component=="static_library"', { 28 ['component=="static_library"', {
29 'targets': [ 29 'targets': [
30 { 30 {
31 'target_name': 'skia', 31 'target_name': 'skia',
32 'type': 'none', 32 'type': 'none',
33 'dependencies': [ 33 'dependencies': [
(...skipping 112 matching lines...) Expand 10 before | Expand all | Expand 10 after
146 '../third_party/mojo/mojom_bindings_generator.gypi', 146 '../third_party/mojo/mojom_bindings_generator.gypi',
147 ], 147 ],
148 'sources': [ 148 'sources': [
149 # Note: file list duplicated in GN build. 149 # Note: file list duplicated in GN build.
150 'public/interfaces/bitmap.mojom', 150 'public/interfaces/bitmap.mojom',
151 'public/type_converters.cc', 151 'public/type_converters.cc',
152 ], 152 ],
153 }, 153 },
154 ], 154 ],
155 } 155 }
OLDNEW
« no previous file with comments | « sandbox/linux/sandbox_linux.gypi ('k') | skia/skia_library_opts.gyp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698