[libcxx-commits] [PATCH] D97572: [libc++] Include <__config_site> from <__config>

[Petr Hosek via Phabricator via libcxx-commits]

phosek added a comment.

In D97572#2651312 <https://reviews.llvm.org/D97572#2651312>, @ldionne wrote:

> In D97572#2648233 <https://reviews.llvm.org/D97572#2648233>, @phosek wrote:
>
>> Looks like presubmit checks are passing now.
>
> I'm pretty sure we'll need the `cxx-headers` change in libcxxabi (D98367 <https://reviews.llvm.org/D98367>) to make this change, no? Otherwise, we might break some people building and testing libc++abi in funky ways. I'd also wait until https://reviews.llvm.org/D97888 has landed so that the pre-commit tests include the Runtimes build. But except for that, I think this is good and I'm excited/scared to try this again.

This should be ready to land now.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D97572/new/

https://reviews.llvm.org/D97572