[libcxx-commits] [PATCH] D103551: [libcxx][module-map] creates submodules for private headers

[Arthur O'Dwyer via Phabricator via libcxx-commits]

Quuxplusone added a comment.

Seems surprisingly painless.
You did miss the other "libc++ classic style" helper headers, e.g. `__functional_03`; did you want to get those in this same PR?



================
Comment at: libcxx/include/module.modulemap:303
   }
   module format {
     header "format"
----------------
`requires cplusplus20`? or what's the rationale for adding that on various headers?
E.g. I see `<latch>` is marked with `requires cplusplus14`, even though it's also new-in-C++20.


================
Comment at: libcxx/include/module.modulemap:469-476
+      module access                 { header "__ranges/access.h"                 }
+      module concepts               { header "__ranges/concepts.h"               }
+      module data                   { header "__ranges/data.h"                   }
+      module empty                  { header "__ranges/empty.h"                  }
+      module enabled_borrowed_range { header "__ranges/enabled_borrowed_range.h" }
+      module size                   { header "__ranges/size.h"                   }
+      module view_interface         { header "__ranges/view_interface.h"         }
----------------
I like this one-line-per-header formatting. Please also use it for `__utility` and `__format` and `__algorithm`.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D103551