Chromium Code Reviews| Index: build/common.gypi |
| =================================================================== |
| --- build/common.gypi (revision 266327) |
| +++ build/common.gypi (working copy) |
| @@ -725,8 +725,8 @@ |
| # linux_use_bundled_gold: whether to use the gold linker binary checked |
| # into third_party/binutils. Force this off via GYP_DEFINES when you |
| # are using a custom toolchain and need to control -B in ldflags. |
| - # Gold is not used for 32-bit linux builds as it runs out of address |
| - # space. |
| + # Do not use 32-bit gold on 32-bit hosts as it runs out address space |
| + # for component=static_library builds. |
|
mithro-old
2014/04/28 19:01:21
Tab.
Nico
2014/04/28 19:01:46
:set expandtab | retab
Lei Zhang
2014/04/28 19:13:22
Done.
|
| ['OS=="linux" and (target_arch=="x64" or target_arch=="arm")', { |
| 'linux_use_bundled_gold%': 1, |
| }, { |
| @@ -1440,6 +1440,16 @@ |
| # Omit unwind support in official release builds to save space. We |
| # can use breakpad for these builds. |
| 'release_unwind_tables%': 0, |
| + |
| + 'conditions': [ |
| + # For official builds, use a 64-bit linker to avoid running out |
| + # of address space. The buildbots should have a 64-bit kernel |
| + # and a 64-bit libc installed. |
| + ['host_arch=="ia32" and target_arch=="ia32"', { |
| + 'linux_use_bundled_gold%': '1', |
| + 'binutils_dir%': 'third_party/binutils/Linux_x64/Release/bin', |
| + }], |
| + ], |
| }], |
| ], |
| }], # os_posix==1 and OS!="mac" and OS!="ios" |
| @@ -3790,16 +3800,14 @@ |
| ['linux_dump_symbols==1', { |
| 'cflags': [ '-g' ], |
| 'conditions': [ |
| - # TODO(thestig) We should not need to specify chromeos==0 here, |
| - # but somehow ChromeOS uses gold despite linux_use_bundled_gold==0. |
| - # http://crbug.com./360082 |
| - ['linux_use_bundled_gold==0 and chromeos==0 and OS!="android"', { |
| + ['OS=="linux" and host_arch=="ia32" and linux_use_bundled_gold==0', { |
| 'target_conditions': [ |
| ['_toolset=="target"', { |
| 'ldflags': [ |
| - # Workarounds for linker OOM. |
| + # Attempt to use less memory to prevent the linker from |
| + # running out of address space. Considering installing a |
| + # 64-bit kernel and switching to a 64-bit linker. |
| '-Wl,--no-keep-memory', |
| - '-Wl,--reduce-memory-overheads', |
|
Lei Zhang
2014/04/28 18:51:56
--reduce-memory-overheads is a PITA because it's G
|
| ], |
| }], |
| ], |