[PATCH] D58107: [MinGW] Add the profiling library when necessary

Martin Storsjö via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Feb 13 05:10:38 PST 2019


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





More information about the cfe-commits mailing list