[PATCH] D80492: Avoid linking libdl unless needed

Roman Lebedev via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Jun 3 14:20:44 PDT 2020


lebedev.ri added a comment.

FYI as reported on IRC, this likely breaks standalone build of clang

  <mceier> standalone builds of clang are still supported right ? commit 0073c293a401774ac96b4b3d27f05e13f379f98e seems to break them, since HAVE_LIBDL is not defined (it requires config-ix include from llvm) causing linking libclang.so to fail with undefined reference to dladdr due to missing -ldl
  <LebedevRI> assuming the versions match, sounds like a bug
  <mceier> LebedevRI: by adding "-DHAVE_LIBDL -DCMAKE_DL_LIBS=dl" to cmake I was able to build standalone clang; so it is a bug

Would be good to either have a fix or a revert.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D80492/new/

https://reviews.llvm.org/D80492





More information about the cfe-commits mailing list