[PATCH] D41111: [profile] Solaris ld supports __start___llvm_prof_data etc. labels
Rainer Orth via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Wed Dec 13 02:44:07 PST 2017
ro added a comment.
In https://reviews.llvm.org/D41111#952623, @vsk wrote:
> Is there a way to gate the change on the Solaris OS version (maybe it's in the triple somewhere)? Apart from that, this patch lgtm.
Unfortunately not: all `interesting' Solaris versions identify themselves as *-*-solaris2.11. In gcc land, one would use a configure test
looking at the ld version number in cases like this, since it has turned out that actual linker feature tests are often hard and involved.
The same issue applies to whether or not to use the libclang_rt.sancov_{begin,end}-*.a libraries in https://reviews.llvm.org/D40899 and
https://reviews.llvm.org/D40903.
Repository:
rL LLVM
https://reviews.llvm.org/D41111
More information about the llvm-commits
mailing list