Commit ac92dd69 authored by George Burgess IV's avatar George Burgess IV Committed by Commit Bot

build: remove an obsolete workaround

Chrome OS swapped its default linker to lld for Chrome quite a while
ago. At this point, we no longer support anything _but_ lld in our SDK
for linking Chrome. Hence, this workaround now seems unnecessary.

Bug: 837095
Change-Id: I4fb50552c9f88c230683a29d8135c77cec3fa7d6
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1960181
Commit-Queue: Nico Weber <thakis@chromium.org>
Reviewed-by: default avatarNico Weber <thakis@chromium.org>
Cr-Commit-Position: refs/heads/master@{#723588}
parent cc70a5e3
...@@ -413,14 +413,7 @@ config("compiler") { ...@@ -413,14 +413,7 @@ config("compiler") {
# Linux-specific compiler flags setup. # Linux-specific compiler flags setup.
# ------------------------------------ # ------------------------------------
if (((is_posix || is_fuchsia) && use_lld) || if ((is_posix || is_fuchsia) && use_lld) {
(target_os == "chromeos" && is_android)) {
# NOTE: Some Chrome OS builds globally disable LLD, but they also build some
# targets against Android toolchains which should use LLD. Therefore we
# explicitly select LLD in these cases.
#
# TODO(https://crbug.com/837095): This should be cleaned up if/when LLD can
# work properly for Chrome OS builds.
ldflags += [ "-fuse-ld=lld" ] ldflags += [ "-fuse-ld=lld" ]
if (current_cpu == "arm64") { if (current_cpu == "arm64") {
# Reduce the page size from 65536 in order to reduce binary size slightly # Reduce the page size from 65536 in order to reduce binary size slightly
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment