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

Side by Side Diff: chrome/chrome_dll.gypi

Issue 7541080: Enable /PROFILE for Release builds and remove it from Debug builds. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/
Patch Set: fixed a small typo in a comment Created 9 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 | Annotate | Revision Log
« build/common.gypi ('K') | « build/common.gypi ('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
1 # Copyright (c) 2011 The Chromium Authors. All rights reserved. 1 # Copyright (c) 2011 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 'target_defaults': { 5 'target_defaults': {
6 'variables': { 6 'variables': {
7 'chrome_dll_target': 0, 7 'chrome_dll_target': 0,
8 }, 8 },
9 'target_conditions': [ 9 'target_conditions': [
10 ['chrome_dll_target==1', { 10 ['chrome_dll_target==1', {
(...skipping 134 matching lines...) Expand 10 before | Expand all | Expand 10 after
145 ], 145 ],
146 'msvs_settings': { 146 'msvs_settings': {
147 'VCLinkerTool': { 147 'VCLinkerTool': {
148 'ImportLibrary': '$(OutDir)\\lib\\chrome_dll.lib', 148 'ImportLibrary': '$(OutDir)\\lib\\chrome_dll.lib',
149 # On Windows we use build chrome_dll as an intermediate target 149 # On Windows we use build chrome_dll as an intermediate target
150 # then have a subsequent step which either optimizes it to its 150 # then have a subsequent step which either optimizes it to its
151 # final location, or copies it to its final location, based 151 # final location, or copies it to its final location, based
152 # on whether or not optimize_with_syzygy==1. 152 # on whether or not optimize_with_syzygy==1.
153 'ProgramDatabaseFile': '$(OutDir)\\initial\\chrome_dll.pdb', 153 'ProgramDatabaseFile': '$(OutDir)\\initial\\chrome_dll.pdb',
154 'OutputFile': '$(OutDir)\\initial\\chrome.dll', 154 'OutputFile': '$(OutDir)\\initial\\chrome.dll',
155 'conditions': [
156 ['fastbuild==0', {
157 # This corresponds to the /PROFILE flag which ensures the PDB
158 # file contains FIXUP information. This information is use d
159 # by the Syzygy optimization tool. Syzygy optimization is
160 # disabled when fastbuild!=0.
161 'Profile': 'true',
162 }],
163 ],
164 }, 155 },
165 }, 156 },
166 }], # OS=="win" 157 }], # OS=="win"
167 ['OS=="mac"', { 158 ['OS=="mac"', {
168 # The main browser executable's name is <(mac_product_name). 159 # The main browser executable's name is <(mac_product_name).
169 # Certain things will get confused if two modules in the 160 # Certain things will get confused if two modules in the
170 # executable share the same name, so append " Framework" to the 161 # executable share the same name, so append " Framework" to the
171 # product_name used for the framework. This will result in 162 # product_name used for the framework. This will result in
172 # a name like "Chromium Framework.framework". 163 # a name like "Chromium Framework.framework".
173 'product_name': '<(mac_product_name) Framework', 164 'product_name': '<(mac_product_name) Framework',
(...skipping 561 matching lines...) Expand 10 before | Expand all | Expand 10 after
735 'LinkIncremental': '<(msvs_debug_link_nonincremental)', 726 'LinkIncremental': '<(msvs_debug_link_nonincremental)',
736 }, 727 },
737 }, 728 },
738 }, 729 },
739 }, 730 },
740 }, # target chrome_dll 731 }, # target chrome_dll
741 ], 732 ],
742 }], 733 }],
743 ], 734 ],
744 } 735 }
OLDNEW
« build/common.gypi ('K') | « build/common.gypi ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698