[libcxx-commits] [PATCH] D155141: [libc++][Modules] Group the private detail headers into larger modules

Ian Anderson via Phabricator via libcxx-commits libcxx-commits at lists.llvm.org
Wed Jul 12 17:22:17 PDT 2023


iana created this revision.
iana added reviewers: ldionne, Mordante, Bigcheese.
Herald added subscribers: jdoerfert, ributzka, arichardson.
Herald added a project: All.
iana requested review of this revision.
Herald added a reviewer: jdoerfert.
Herald added subscribers: libcxx-commits, jplehr, sstefan1.
Herald added a project: libc++.
Herald added a reviewer: libc++.

It's not really in the spirit of clang modules to make a top level module for every single header. That's necessary for the public headers, but the private detail headers could be grouped into bigger modules as long as they remain acyclic. Make a Python script to do the grouping.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D155141

Files:
  libcxx/docs/Contributing.rst
  libcxx/include/CMakeLists.txt
  libcxx/include/generate_module_map.py
  libcxx/include/internal_module_attributes.json.in
  libcxx/include/module.modulemap.in
  libcxx/test/libcxx/lint/lint_headers.sh.py
  libcxx/utils/libcxx/test/header_information.py

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D155141.539804.patch
Type: text/x-patch
Size: 180789 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/libcxx-commits/attachments/20230713/2617b939/attachment-0001.bin>


More information about the libcxx-commits mailing list