[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