Page MenuHomeSolus

Enable support for llvm's LLD linker
AbandonedPublic

Authored by joebonrichie on Dec 11 2017, 12:39 AM.

Details

Reviewers
None
Group Reviewers
Triage Team
Summary

Drop-in, very-speedy replacement for other linkers such as ld and ld.gold.

Signed-off-by: Joey Riches <josephriches@gmail.com>

Test Plan

Built against chromium 63 stable, which now requires LLD by default.

Diff Detail

Repository
R1972 llvm
Branch
master
Lint
No Linters Available
Unit
No Unit Test Coverage

Event Timeline

joebonrichie created this revision.Dec 11 2017, 12:39 AM

So this ends up in llvm-clang right? Have to be careful that it doesn't turn up in normal builds as stuff shits itself when ld.gold is present. But I do want to test this as the default clang linker

Yeah it does, only reason ld.gold and lld don't work out of the box AFAIK is our use of --copy-dt-needed-entries whilst other distros use --no-copy-dt-needed-entries, generally packages will use the system default linker unless they are explicitly looking another linker such as ld.gold e.g. webkit2, also AFAIK. Still would like to test this against other packages that are built with clang I think lld is still too new for packages to look for it explicitly.

ikey abandoned this revision.Jan 26 2018, 12:53 PM
ikey added a subscriber: ikey.

Rebased it for ya