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

Side by Side Diff: chrome/chrome_browser.gypi

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, 4 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 | « build/common.gypi ('k') | chrome/chrome_browser_ui.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 # Copyright 2012 The Chromium Authors. All rights reserved. 1 # Copyright 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 'variables': { 6 'variables': {
7 # All .cc, .h, .m, and .mm files under browser except for: 7 # All .cc, .h, .m, and .mm files under browser except for:
8 # * tests and mocks. 8 # * tests and mocks.
9 # * code below browser/chromeos 9 # * code below browser/chromeos
10 # * code below browser/extensions 10 # * code below browser/extensions
(...skipping 3202 matching lines...) Expand 10 before | Expand all | Expand 10 after
3213 '../sql/sql.gyp:sql', 3213 '../sql/sql.gyp:sql',
3214 '../sync/sync.gyp:sync', 3214 '../sync/sync.gyp:sync',
3215 ], 3215 ],
3216 # sources applied to all configurations 3216 # sources applied to all configurations
3217 'sources': [ 3217 'sources': [
3218 '<@(chrome_browser_undo_sources)', 3218 '<@(chrome_browser_undo_sources)',
3219 ], 3219 ],
3220 'includes': [ 3220 'includes': [
3221 # Disable LTO due to ELF section name out of range 3221 # Disable LTO due to ELF section name out of range
3222 # crbug.com/422251 3222 # crbug.com/422251
3223 '../build/android/disable_lto.gypi', 3223 '../build/android/disable_gcc_lto.gypi',
3224 ], 3224 ],
3225 'conditions': [ 3225 'conditions': [
3226 ['OS != "ios"', { 3226 ['OS != "ios"', {
3227 'dependencies': [ 3227 'dependencies': [
3228 'apps', 3228 'apps',
3229 'common_mojo_bindings', 3229 'common_mojo_bindings',
3230 'debugger', 3230 'debugger',
3231 'installer_util', 3231 'installer_util',
3232 '../third_party/re2/re2.gyp:re2', 3232 '../third_party/re2/re2.gyp:re2',
3233 '../cc/cc.gyp:cc', 3233 '../cc/cc.gyp:cc',
(...skipping 843 matching lines...) Expand 10 before | Expand all | Expand 10 after
4077 '../components/components.gyp:omnibox_browser', 4077 '../components/components.gyp:omnibox_browser',
4078 '../components/components.gyp:rlz', 4078 '../components/components.gyp:rlz',
4079 '../components/components.gyp:search_engines', 4079 '../components/components.gyp:search_engines',
4080 '../rlz/rlz.gyp:rlz_lib', 4080 '../rlz/rlz.gyp:rlz_lib',
4081 ], 4081 ],
4082 }, 4082 },
4083 ], 4083 ],
4084 }], 4084 }],
4085 ], 4085 ],
4086 } 4086 }
OLDNEW
« no previous file with comments | « build/common.gypi ('k') | chrome/chrome_browser_ui.gypi » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698