[all-commits] [llvm/llvm-project] 1782e8: [C++20] [Modules] Allow -fmodule-file=<module-name...

Chuanqi Xu via All-commits all-commits at lists.llvm.org
Wed Feb 8 00:45:43 PST 2023


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 1782e8f9e882e8f4fb59968ff555c8c93827ea02
      https://github.com/llvm/llvm-project/commit/1782e8f9e882e8f4fb59968ff555c8c93827ea02
  Author: Chuanqi Xu <yedeng.yd at linux.alibaba.com>
  Date:   2023-02-08 (Wed, 08 Feb 2023)

  Changed paths:
    M clang/docs/StandardCPlusPlusModules.rst
    M clang/lib/Frontend/CompilerInstance.cpp
    M clang/lib/Sema/SemaModule.cpp
    M clang/test/CXX/modules-ts/basic/basic.link/module-declaration.cpp
    M clang/test/CXX/modules-ts/dcl.dcl/dcl.module/p2.cpp
    A clang/test/Modules/cxx20-impl-module-conditionally-load.cppm
    A clang/test/Modules/cxx20-named-conditionally-load.cppm

  Log Message:
  -----------
  [C++20] [Modules] Allow -fmodule-file=<module-name>=<BMI-Path> for implementation unit and document the behavior

Close https://github.com/llvm/llvm-project/issues/57293.

Previsouly we can't use `-fmodule-file=<module-name>=<BMI-Path>` for
implementation units, it is a bug. Also the behavior of the above option
is not tested nor documented for C++20 Modules. This patch addresses the
2 problems.




More information about the All-commits mailing list