[PATCH] D134267: [C++] [Modules] Support one phase compilation model for named modules

Chuanqi Xu via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Oct 17 01:59:27 PDT 2022


ChuanqiXu updated this revision to Diff 468138.
ChuanqiXu added a comment.

Address @tschuett's opinion:

- Use new introduced `-fc++-module-cache-path=` instead of `-fc++-module-path` to avoid many logics about modules cache in clang modules.
- Use new introduced `-fmodule-bmi-output=` instead of `-fmodule-name` for the same reason. Also `-fmodule-bmi-output=` is more flexible since `-fmodule-name` require its argument to be the same with the actual module name.


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

https://reviews.llvm.org/D134267

Files:
  clang/docs/ReleaseNotes.rst
  clang/include/clang/Basic/DiagnosticCommonKinds.td
  clang/include/clang/Driver/Options.td
  clang/lib/Driver/Driver.cpp
  clang/lib/Driver/ToolChains/Clang.cpp
  clang/test/Driver/create_module_cache.cpp
  clang/test/Modules/one-phase-compilation-named-modules.cppm

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D134267.468138.patch
Type: text/x-patch
Size: 7115 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20221017/e294a9e5/attachment-0001.bin>


More information about the cfe-commits mailing list