rmaprath added a comment. In https://reviews.llvm.org/D27204#613122, @EricWF wrote:
> My main issue with this patch (and https://reviews.llvm.org/D27206) is that > there are now two different CMake options for building two different external > threading libraries. I would much prefer having libc++abi use libc++'s > `__threading_support` header and `cxx_external_threads` library. I would like to do that too! But I always viewed `libcxx` and `libcxxabi` as two independent libraries that should be usable on their own. In this case, binding `libcxxabi` to `__threading_support` from `libcxx` sounded wrong, because `__threading_support` is something internal to `libcxx` (e.g. all the functions there are prefixed with `__libcpp_` - there may be other `libcpp`-isms arising from `__config` in there too). Is that not a concern? I could give it a shot and try to merge the two APIs into one if not. https://reviews.llvm.org/D27204 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
