[PATCH] D137609: [C++20] [Modules] Remove unmaintained header modules

Chuanqi Xu via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Nov 7 22:35:58 PST 2022


ChuanqiXu created this revision.
ChuanqiXu added reviewers: rsmith, iains, dblaikie, Bigcheese.
ChuanqiXu added a project: clang-modules.
Herald added a project: All.
ChuanqiXu requested review of this revision.
Herald added a project: clang.
Herald added a subscriber: cfe-commits.

Currently there is a `-emit-header-module` mode, which can combine several headers together as a module interface. However, this breaks our assumption (for standard c++ modules) about `module interface`. The `module interface` should come from a module interface unit. And if it is a header, it should be a header unit. And currently we have no ideas to combine several headers together.

So I think this mode is an experimental one and it is not maintained and it is not used. So it will be better to remove them.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D137609

Files:
  clang/include/clang/Frontend/FrontendActions.h
  clang/include/clang/Frontend/FrontendOptions.h
  clang/lib/Frontend/CompilerInvocation.cpp
  clang/lib/Frontend/FrontendActions.cpp
  clang/lib/FrontendTool/ExecuteCompilerInvocation.cpp
  clang/test/CXX/cpp/cpp.module/Inputs/attrs.h
  clang/test/CXX/cpp/cpp.module/Inputs/empty.h
  clang/test/CXX/cpp/cpp.module/p1.cpp
  clang/test/CXX/cpp/cpp.module/p2.cpp
  clang/test/CXX/module/module.interface/p2.cpp
  clang/test/Driver/header-module.cpp
  clang/test/Driver/modules.cpp
  clang/test/Misc/serialized-diags-emit-header-module-misconfig.c
  clang/test/Modules/no-module-map.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D137609.473876.patch
Type: text/x-patch
Size: 14695 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20221108/7a55a7e9/attachment-0001.bin>


More information about the cfe-commits mailing list