[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