[PATCH] D43168: Don't use -ldl on BSD

Dimitry Andric via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun Feb 11 07:39:56 PST 2018


dim added a comment.

In https://reviews.llvm.org/D43168#1004536, @krytarowski wrote:

> CMAKE_DL_LIBS allows us to fix it once for everybody and forever.
>
> CMAKE_DL_LIBS is already used in llvm, e.g.
>
>   if( HAVE_LIBDL )
>     set(system_libs ${system_libs} ${CMAKE_DL_LIBS})
>   endif()
>   
>
> in llvm/lib/Support/CMakeLists.txt
>
> I don't want to repeat this process for AIX, SmartOS (they are now migrating to Clang at least for pkgsrc) etc.


Hm, well my CMake knowledge is limited, so if you could show me how to do this, please let me know.  Otherwise I will have to drop this revision, and keep on patching manually :(


Repository:
  rT test-suite

https://reviews.llvm.org/D43168





More information about the llvm-commits mailing list