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

Side by Side Diff: build/android/increase_size_for_speed.gypi

Issue 646873004: Add Link-Time Optimizations support for Android. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 6 years, 2 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
OLDNEW
1 # Copyright (c) 2014 The Chromium Authors. All rights reserved. 1 # Copyright (c) 2014 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 # This file is meant to be included to optimize a target for speed 5 # This file is meant to be included to optimize a target for speed
6 # rather than for size on Android. 6 # rather than for size on Android.
7 # This is used in some carefully tailored targets and is not meant 7 # This is used in some carefully tailored targets and is not meant
8 # to be included everywhere. Before adding the template to another target, 8 # to be included everywhere. Before adding the template to another target,
9 # please ask in chromium-dev@. See crbug.com/411909 9 # please ask in chromium-dev@. See crbug.com/411909
10 10
11 { 11 {
12 'configurations': { 12 'configurations': {
13 'Release': { 13 'Release': {
14 'target_conditions': [ 14 'target_conditions': [
15 ['_toolset=="target"', { 15 ['_toolset=="target"', {
16 'conditions': [ 16 'conditions': [
17 ['OS=="android"', { 17 ['OS=="android"', {
18 'cflags!': ['-Os'], 18 'cflags!': ['-Os'],
19 'cflags': ['-O2'], 19 'cflags': ['-O2'],
20 }], 20 }],
21 # Do not merge -Os and -O2 in LTO
pasko 2014/10/10 11:59:21 s/LTO/LTO./ Also you can briefly explain, like: A
Fabrice (no longer in Chrome) 2014/10/13 15:45:12 Done.
22 ['use_lto==1', {
pasko 2014/10/10 11:59:21 maybe this should be called use_lto_os? I understa
Fabrice (no longer in Chrome) 2014/10/13 15:45:12 I think it's too late for that at this point. The
23 'cflags!': [
24 '-flto',
25 '-ffat-lto-objects',
26 ],
27 }],
28 ['use_lto_o2==1', {
29 'cflags': [
30 '-flto',
31 '-ffat-lto-objects',
32 ],
33 }],
21 ], 34 ],
22 }], 35 }],
23 ], 36 ],
24 }, 37 },
25 }, 38 },
26 } 39 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698