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

Issues Search
    My Issues | Starred     Open | Closed | All

Unified Diff: build/common.gypi

Issue 9789003: lto Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: gets a bit further Created 8 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | tools/clang/scripts/update.sh » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: build/common.gypi
diff --git a/build/common.gypi b/build/common.gypi
index a098a1c451fa9b7c8fa88da3c04ed0b62751a22a..8c7f0e6b19fbf06b2409ebf276b90de8b1480fb9 100644
--- a/build/common.gypi
+++ b/build/common.gypi
@@ -1610,9 +1610,15 @@
'NDEBUG',
],
'xcode_settings': {
- 'DEAD_CODE_STRIPPING': 'YES', # -Wl,-dead_strip
+ #'DEAD_CODE_STRIPPING': 'YES', # -Wl,-dead_strip # lto + this = ld crash
'GCC_OPTIMIZATION_LEVEL': '<(mac_release_optimization)',
- 'OTHER_CFLAGS': [ '<@(release_extra_cflags)', ],
+ 'OTHER_CFLAGS': [ '<@(release_extra_cflags)',
+ '-Qunused-arguments', # -flto for .S files for example.
+ ],
+ 'LLVM_LTO': 'YES', # -flto, XXX
+ # makes libtool complain like
+ # "descriptor.o is not an object file (not allowed in a library)"
+ # with xcodebuild? at least with xcode 3.2.6.
},
'msvs_settings': {
'VCCLCompilerTool': {
« no previous file with comments | « no previous file | tools/clang/scripts/update.sh » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698