This include doesn't seem to be needed for the standalone build (it's
not being used by libc++ build either), but introduces unnecessary
dependency because HandleOutOfTreeLLVM performs checks that require
a working C++ library. We shouldn't require a working C++ library to
build libc++abi or libc++ (it's what we're building after all).
Details
Details
Diff Detail
Diff Detail
- Repository
- rL LLVM
Event Timeline
Comment Actions
I also think that this one is better than D58331, for the reasons stated in the summary.
Do you know if there's any reason why HandleLLVMOptions was included here?
HandleLLVMOptions seems to setup quite a few things, but I think most of them don't apply for libc++abi and libc++ (like LTO, assertions, -std=c++11). From looking over the code I guess that warnings and LLVM_USE_LINKER could be relevant?
Comment Actions
Thank you. I'd been try to land this myself (See discussion in https://reviews.llvm.org/D33753).