[PATCH] D87418: [LLD] Allow configuring default ld.lld backend

Fangrui Song via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Sep 18 10:24:41 PDT 2020


MaskRay added a comment.

In D87418#2273236 <https://reviews.llvm.org/D87418#2273236>, @mstorsjo wrote:

> Pushed this one now.
>
> Just FTR, keep in mind that if LLD is configured this way, most existing ELF tests will fail, as they don't pass any explicit `-m` parameter. Not sure if it's worth adding such an option to all of them, or just require building with the option in the default state if one wants to work on LLD and actually run tests.

Not worth changing all RUN lines of all test/ELF/ tests. You can probably use lit.cfg.py to mark all test/ELF tests unsupported if LLD_DEFAULT_LD_LLD_IS_MINGW.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D87418



More information about the llvm-commits mailing list