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

Unified Diff: chrome/chrome_dll.gypi

Issue 8685001: Remove the Syzygy optimization alternative from GYP configs. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 9 years, 1 month 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « chrome/chrome.gyp ('k') | chrome/chrome_dll_syzygy.gypi » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/chrome_dll.gypi
diff --git a/chrome/chrome_dll.gypi b/chrome/chrome_dll.gypi
index 83fde921ace10f40cd9ea2ef743144ff12698ad8..7389d9840673a27194f527426f742aae9c54e893 100644
--- a/chrome/chrome_dll.gypi
+++ b/chrome/chrome_dll.gypi
@@ -8,21 +8,8 @@
{
'variables': {
'enable_wexit_time_destructors': 1,
- 'conditions' : [
- ['OS=="win" and optimize_with_syzygy==1', {
- # On Windows we use build chrome_dll as an intermediate target
- # then have a subsequent step which either optimizes it to its
- # final location, or copies it to its final location, depending
- # on whether or not optimize_with_syzygy==1. Please, refer to
- # chrome_dll_syzygy.gypi for the subsequent defintion of the
- # Windows chrome_dll target.
- 'dll_target_name': 'chrome_dll_initial',
- }, {
- 'dll_target_name': 'chrome_dll',
- }],
- ],
},
- 'target_name': '<(dll_target_name)',
+ 'target_name': 'chrome_dll',
grt (UTC plus 2) 2011/11/23 22:03:11 ubernit: move target_name and type back above the
Sigurður Ásgeirsson 2011/11/23 22:25:52 Done.
'type': 'shared_library',
'dependencies': [
'<@(chromium_dependencies)',
@@ -113,13 +100,7 @@
# Set /SUBSYSTEM:WINDOWS for chrome.dll (for consistency).
'SubSystem': '2',
'conditions': [
- ['optimize_with_syzygy==1', {
- # When syzygy is enabled we use build chrome_dll as an
- # intermediate target then have a subsequent step which
- # optimizes it to its final location
- 'ProgramDatabaseFile': '$(OutDir)\\initial\\chrome_dll.pdb',
- 'OutputFile': '$(OutDir)\\initial\\chrome.dll',
- }], ['incremental_chrome_dll==1', {
+ ['incremental_chrome_dll==1', {
'OutputFile': '$(OutDir)\\initial\\chrome.dll',
'UseLibraryDependencyInputs': "true",
}],
@@ -146,7 +127,7 @@
},
},
'conditions': [
- ['incremental_chrome_dll==1 and optimize_with_syzygy==0', {
+ ['incremental_chrome_dll==1', {
# Linking to a different directory and then hardlinking back
# to OutDir is a workaround to avoid having the .ilk for
# chrome.exe and chrome.dll conflicting. See crbug.com/92528
« no previous file with comments | « chrome/chrome.gyp ('k') | chrome/chrome_dll_syzygy.gypi » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698