================
@@ -355,6 +349,9 @@ function(add_link_opts target_name)
     set_property(TARGET ${target_name} APPEND_STRING PROPERTY
                  LINK_FLAGS " -Wl,-brtl")
   endif()
+
+  check_linker_flag(CXX "-Wl,-Bsymbolic-functions"
----------------
rorth wrote:

**Please** don't go this route: it makes feature tests completely 
unmaintainable.  Consider the Solaris case where initially only `/bin/ld` was 
supported which didn't accept `-Bsymbolic-functions`.  Only later when GNU `ld` 
became an alternative linker that one **would** support that option.  How on 
earth is one supposed to know that I now have to revise the cmake check to 
detect this if it were restricted to some platforms?

https://github.com/llvm/llvm-project/pull/79539
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to