[PATCH] D44964: Change order of libclang_rt.profile link for freebsd

2020-10-30 Thread Dimitry Andric via Phabricator via cfe-commits
dim added a comment. Yes, this looks pretty fine to me, but indeed needs a test. Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D44964/new/ https://reviews.llvm.org/D44964 ___ cfe-commits mailing list cfe-commits@lists.l

[PATCH] D44964: Change order of libclang_rt.profile link for freebsd

2018-03-28 Thread Vedant Kumar via Phabricator via cfe-commits
vsk added a comment. Could you add a test in test/Driver/instrprof-ld.c? Repository: rC Clang https://reviews.llvm.org/D44964 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[PATCH] D44964: Change order of libclang_rt.profile link for freebsd

2018-03-27 Thread Tom Rix via Phabricator via cfe-commits
trixirt created this revision. trixirt added reviewers: emaste, rsmith. Herald added a subscriber: cfe-commits. clang -static -coverage foo.c fails because the -lclang_rt.profile-*.a has a dependency on libc but is placed after libc in the link line. This change place -lclang_rt.profile before -