[PATCH] D37631: [libFuzzer] Support using libc++

Petr Hosek via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Sep 20 10:31:50 PDT 2017


phosek added a comment.

In https://reviews.llvm.org/D37631#871554, @kcc wrote:

> I would rather require libc++ for libFuzzer to build. Isn't it simpler?


There still the case when you're building compiler-rt without having libc++ around in which case you need to fallback onto whatever C++ library is available is on host. Using libc++ whenever the source is available and we're building with Clang, independently of other options, would be fine with me.

> We'll also need tests for any such change.

I was thinking about the right testing strategy. The lit setup already links explicitly against libstdc++, so if we use libc++ whenever possible, the existing tests should be sufficient to check the two libraries work together. We may consider adding additional lit configuration which also uses libc++.


Repository:
  rL LLVM

https://reviews.llvm.org/D37631





More information about the llvm-commits mailing list