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

Side by Side Diff: chrome/chrome_pgo.gypi

Issue 187813008: Add PGO targets to Chrome. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 6 years, 9 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 | « chrome/chrome.gyp ('k') | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
(Empty)
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
3 # found in the LICENSE file.
4 {
5 'conditions': [
6 ['OS=="win"', {
7 'targets': [
8 # In the PGO (Profile-Guided Optimization) build flow, we need to
9 # build the target binary multiple times. To implement this flow with
10 # gyp, here we define multiple targets, each of which represents one
11 # build particular build/profile stage.
12 {
13 'target_name': 'chrome_pgo_main',
14 'type': 'none',
yukawa 2014/03/07 05:25:22 Does this really work? The 'type' is 'none' here.
15 'dependencies': [ 'chrome_main_dll', ],
16 'msvs_settings': {
17 'VCCLCompilerTool': {
18 'WholeProgramOptimization': 'true', # /GL
19 },
20 'VCLibrarianTool': {
21 'LinkTimeCodeGeneration': 'true',
22 },
23 },
24 'link_settings': {
25 'msvs_settings': {
26 'VCLinkerTool': {
27 # Tell ninja generator not to pass /ManifestFile:<filename>
yukawa 2014/03/07 05:25:22 You might be able to drop this hack. I added this
28 # option to the linker, because it causes LNK1268 error in PGO
29 # build.
30 'GenerateManifest': 'false',
31 },
32 },
33 },
34 }, {
35 'target_name': 'chrome_pgo_instrument',
36 'type': 'none',
37 'msvs_settings': {
38 'VCLinkerTool': {
39 'LinkTimeCodeGeneration': '2',
40 },
41 },
42 'dependencies': [
43 'chrome_pgo_main',
44 ],
45 }, {
46 'target_name': 'chrome_pgo_optimize',
47 'type': 'none',
48 'msvs_settings': {
49 'VCLinkerTool': {
50 'LinkTimeCodeGeneration': '3',
51 },
52 },
53 'dependencies': [
54 # TODO(sebmarchand): Add a target that run some profile test cases.
55 'chrome_pgo_instrument',
56 'chrome_pgo_main',
57 ],
58 }, {
59 'target_name': 'chrome_pgo_update',
60 'type': 'none',
61 'msvs_settings': {
62 'VCLinkerTool': {
63 'LinkTimeCodeGeneration': '4',
64 },
65 },
66 'dependencies': [
67 'chrome_pgo_main',
68 ],
69 },
70 ],
71 }],
72 ],
73 }
OLDNEW
« no previous file with comments | « chrome/chrome.gyp ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698