[PATCH] D40326: Fix static link on debian

Chris Bieneman via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Nov 27 10:16:50 PST 2017


beanz added a comment.

I don't see anything wrong with this patch as a workaround, but it seems like there might be a bigger issue here.

Is there a reason why std::thread is implemented with weak references to pthread? Is this libstdc++'s implementation or libcxx? Should we consider changes to the clang driver as well?

It's undesirable for user code that is using std::thread to require the user to list pthread as a dependency, and even worse if the pthread library needs to be linked with extra flags.


https://reviews.llvm.org/D40326





More information about the llvm-commits mailing list