diff options
author | thestig@chromium.org <thestig@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2014-04-29 01:22:10 +0000 |
---|---|---|
committer | thestig@chromium.org <thestig@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2014-04-29 01:22:10 +0000 |
commit | ac296392d7ab509cda68f9465e0f3f23bc383879 (patch) | |
tree | 55ce5bcb9d27ef248508f1e300a54ada53751bd5 /build/common.gypi | |
parent | b71e2501b2ab530bcb61c6e812fc1f5a4132a019 (diff) | |
download | chromium_src-ac296392d7ab509cda68f9465e0f3f23bc383879.zip chromium_src-ac296392d7ab509cda68f9465e0f3f23bc383879.tar.gz chromium_src-ac296392d7ab509cda68f9465e0f3f23bc383879.tar.bz2 |
Linux: Switch 32-bit official builds to use the bundled 64-bit linker.
It is assumed that even though the buildbots have 32-bit userland,
they also have a 64-bit kernel and a 64-bit libc installed.
BUG=366523
Review URL: https://codereview.chromium.org/255023003
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@266729 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'build/common.gypi')
-rw-r--r-- | build/common.gypi | 24 |
1 files changed, 16 insertions, 8 deletions
diff --git a/build/common.gypi b/build/common.gypi index 23ea727..c0b73e7 100644 --- a/build/common.gypi +++ b/build/common.gypi @@ -729,8 +729,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. ['OS=="linux" and (target_arch=="x64" or target_arch=="arm")', { 'linux_use_bundled_gold%': 1, }, { @@ -1445,6 +1445,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" @@ -3795,16 +3805,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', ], }], ], |