[PATCH] D44499: [Scudo] Avoid accidental references of C++ library

Petr Hosek via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Mar 15 16:22:12 PDT 2018


phosek added a comment.

The missing `-Wl,` is a mistake on my side. This came up while cleaning up the libc++ installation process which made the Scudo build to fail because Scudo couldn't find libc++ headers which was expected since Scudo doesn't have an explicit dependency on it. So either it needs to have that dependency (at least on Fuchsia where libc++ is being built as part of LLVM and doesn't come from sysroot) or it should disable it altogether which is what this patch does. Having scudo-minimal sounds like something we probably want although it wasn't the original motivation behind this patch.


https://reviews.llvm.org/D44499





More information about the llvm-commits mailing list