[libcxx-commits] [PATCH] D127127: [libc++] Fix modules builds when features are removed

Louis Dionne via Phabricator via libcxx-commits libcxx-commits at lists.llvm.org
Mon Jun 6 10:03:33 PDT 2022


ldionne created this revision.
Herald added subscribers: arichardson, mgorny.
Herald added a project: All.
ldionne requested review of this revision.
Herald added a project: libc++.
Herald added a subscriber: libcxx-commits.
Herald added a reviewer: libc++.

When some headers are not available because we removed features like
localization or threads, the compiler should not try to include these
headers when building modules. To avoid that from happening, add a
requires-declaration that is never satisfied when the configuration
in use doesn't support a header.

rdar://93777687


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D127127

Files:
  libcxx/docs/Contributing.rst
  libcxx/include/CMakeLists.txt
  libcxx/include/csignal
  libcxx/include/module.modulemap
  libcxx/include/module.modulemap.in
  libcxx/test/libcxx/lint/lint_headers.sh.py
  libcxx/test/libcxx/lint/lint_modulemap.sh.py
  libcxx/test/libcxx/modules_include.sh.cpp
  libcxx/utils/generate_header_tests.py
  libcxx/utils/libcxx/test/dsl.py
  libcxx/utils/libcxx/test/features.py

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D127127.434514.patch
Type: text/x-patch
Size: 14925 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/libcxx-commits/attachments/20220606/073968a6/attachment-0001.bin>


More information about the libcxx-commits mailing list