[PATCH] D11230: libunwind: Introduce __libunwind_config.h.
Dan Albert
danalbert at google.com
Thu Jul 16 10:30:58 PDT 2015
danalbert accepted this revision.
danalbert added a comment.
This revision is now accepted and ready to land.
Actually, I don't think there's an issue here from the libc++ side of things; we already don't install any of the libunwind headers (though libcxxrt does, so maybe we should?).
My concern was: https://github.com/llvm-mirror/libcxx/blob/master/cmake/Modules/HandleLibCXXABI.cmake#L92
We install libc++abi headers there, and for libcxxrt we also install the unwind headers there. Since this isn't already broken, presumably libunwind is already properly separated to not need them there.
Still, as Renato says, we should be installing the libunwind headers //somewhere//. Since we don't do any of that yet though, I think it's reasonable to save that for a different patch.
http://reviews.llvm.org/D11230
More information about the cfe-commits
mailing list