1557507 - Default to lld linker for Android builds

Status: RESOLVED FIXED

(Firefox Build System :: General, task)

Comment 1

[Mike Hommey [:glandium]]

Attachment: Bug 1557507 - Default to lld linker for local builds in more cases.

The current logic is that if the compiler somehow uses gold rather than
BFD ld by default, we let it, but if it uses BFD ld, we default to lld.
When doing Android builds, the compiler finds the linker in the NDK, and
the default ld binary is gold. So we currently end up using fold for
Android builds.

Here, we change the logic such that we use lld when the default linker
the compiler uses is either BFD ld or gold. We can't go with "is not
lld" because the other possible kind, ld64, is what we actually want to
use on mac, since lld doesn't support mach-o fully just yet.

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/2d8ad105aac8
Default to lld linker for local builds in more cases. r=nalexander

Comment 3

[Bogdan Tara[:bogdan_tara | bogdant]]

Backed out changeset 2d8ad105aac8 (Bug 1557507) on Glandium's request

Backout link: https://hg.mozilla.org/integration/autoland/rev/2d0d626875641a4abaf0387d1d9c72c0b46cb605

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/5e8d4f11d199
Default to lld linker for local builds in more cases. r=nalexander

Comment 5

[BugBot [:suhaib / :marco/ :calixte]]

Bugbug thinks this bug is a task, but please change it back in case of error.

Comment 6

[Daniel Varga [:dvarga]]

https://hg.mozilla.org/mozilla-central/rev/5e8d4f11d199

Comment 7

[Daniel Varga [:dvarga]]

https://hg.mozilla.org/mozilla-central/rev/5e8d4f11d199