mstorsjo added a comment. In D58107#1396063 <https://reviews.llvm.org/D58107#1396063>, @davezarzycki wrote:
> This change breaks building/testing the compiler with `CLANG_DEFAULT_LINKER` > set to `lld`. Was this intentional? What should people do if they want to use > `CLANG_DEFAULT_LINKER` and run the test suite? It wasn't intentional. Originally I added `-fuse-ld=lld` to the test, but this broke buildbots that didn't have lld available. My fixup in rC353922 <https://reviews.llvm.org/rC353922> avoided this by removing the `-fuse-ld=lld` (where I noted in the commit message that I guess that it would break such configurations). At that point I thought the other tests in this same file also would be equally broken in such a setup then. But I see that most other tests use `-fuse-ld=ld`, to work around this. (This option doesn't check if the named tool exists, contrary to `-fuse-ld=<any-other-string>`.) But if a linker named `<triplet>-ld` exists, that will be used instead, so I'll need to make the regex a bit looser as well. I'll commit a fix for this soon. Repository: rL LLVM CHANGES SINCE LAST ACTION https://reviews.llvm.org/D58107/new/ https://reviews.llvm.org/D58107 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
