cjdb added a comment. A few notes to reviewers:
- The patch assumes everything is under a top-level `__libcxx` directory, because it seems there are other system headers that could be prefixed with `__`. libc++ currently has many top-level directories, following the trend of `__${STD_LIB_HEADER}/`. I see two solutions going forward: 1. libc++ moves everything into a top-level `__libcxx` directory, which minimises the chances of clashing with pre-existing top-level directories that we don't own. Once this patch is committed, it probably won't need to be updated (sans bug fixes). 2. This patch adopts a set of "blessed" directories that needs to be consistently updated as we splice more and more headers (and as WG21 adds more headers to C++). - The patch could probably do with more testing, but I'm coming up dry :-( Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D105439/new/ https://reviews.llvm.org/D105439 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits