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

Side by Side Diff: chrome/chrome_dll.gypi

Issue 1045113004: Update LTO flags for Mac. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 years, 8 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') | 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) 2012 The Chromium Authors. All rights reserved. 1 # Copyright (c) 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 'conditions': [ 5 'conditions': [
6 ['OS=="mac" or OS=="win"', { 6 ['OS=="mac" or OS=="win"', {
7 'targets': [ 7 'targets': [
8 { 8 {
9 'target_name': 'chrome_dll', 9 'target_name': 'chrome_dll',
10 'type': 'none', 10 'type': 'none',
(...skipping 277 matching lines...) Expand 10 before | Expand all | Expand 10 after
288 'app/chrome_crash_reporter_client_mac.mm', 288 'app/chrome_crash_reporter_client_mac.mm',
289 ], 289 ],
290 'xcode_settings': { 290 'xcode_settings': {
291 # Define the order of symbols within the framework. This 291 # Define the order of symbols within the framework. This
292 # sets -order_file. 292 # sets -order_file.
293 'ORDER_FILE': 'app/framework.order', 293 'ORDER_FILE': 'app/framework.order',
294 }, 294 },
295 'include_dirs': [ 295 'include_dirs': [
296 '<(grit_out_dir)', 296 '<(grit_out_dir)',
297 ], 297 ],
298 }],
299 # This step currently fails when using LTO. TODO(pcc): Re-enable.
300 ['OS=="mac" and use_lto==0', {
298 'postbuilds': [ 301 'postbuilds': [
299 { 302 {
300 # This step causes an error to be raised if the .order file 303 # This step causes an error to be raised if the .order file
301 # does not account for all global text symbols. It 304 # does not account for all global text symbols. It
302 # validates the completeness of the .order file. 305 # validates the completeness of the .order file.
303 'postbuild_name': 'Verify global text symbol order', 306 'postbuild_name': 'Verify global text symbol order',
304 'variables': { 307 'variables': {
305 'verify_order_path': 'tools/build/mac/verify_order', 308 'verify_order_path': 'tools/build/mac/verify_order',
306 }, 309 },
307 'action': [ 310 'action': [
(...skipping 60 matching lines...) Expand 10 before | Expand all | Expand 10 after
368 'dependencies': [ 371 'dependencies': [
369 '../pdf/pdf.gyp:pdf', 372 '../pdf/pdf.gyp:pdf',
370 ], 373 ],
371 }], 374 }],
372 ], 375 ],
373 }, # target chrome_child_dll 376 }, # target chrome_child_dll
374 ], 377 ],
375 }], 378 }],
376 ], 379 ],
377 } 380 }
OLDNEW
« no previous file with comments | « build/common.gypi ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698