[PATCH] D76452: Use LLD by default for Android.

Mon Mar 30 11:57:12 PDT 2020

danalbert added a comment.

In D76452#1945084 <https://reviews.llvm.org/D76452#1945084>, @srhines wrote:

> In D76452#1945029 <https://reviews.llvm.org/D76452#1945029>, @MaskRay wrote:
>
> > To cross build ELF object on macOS, another alternative is a wrapper named `ld` which invokes `lld -flavor gnu "$@"`
>
>
> @danalbert Would this kind of idea work with your other reverted patch? I'm not sure exactly what broke on those builds.

Not really. Windows complicates things here, since it operates based on file extensions rather than shebangs. `ld.exe` is not `ld.cmd`, and `ld` (with no extension) doesn't work. Any system that expects one form won't work with the other. Beyond that, process creation is expensive on Windows so we should be avoiding as many superfluous processes as we can.

Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D76452/new/

https://reviews.llvm.org/D76452