[all-commits] [llvm/llvm-project] e9a787: [C++20] [Modules] Chose BMI from for module m with...

Chuanqi Xu via All-commits all-commits at lists.llvm.org
Fri Oct 27 01:53:29 PDT 2023


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: e9a7876c2c81423f2289aa85eeac32d7a55cd3c8
      https://github.com/llvm/llvm-project/commit/e9a7876c2c81423f2289aa85eeac32d7a55cd3c8
  Author: Chuanqi Xu <yedeng.yd at linux.alibaba.com>
  Date:   2023-10-27 (Fri, 27 Oct 2023)

  Changed paths:
    M clang/lib/Frontend/CompilerInvocation.cpp
    A clang/test/Modules/duplicated-module-file-eq-module-name.cppm

  Log Message:
  -----------
  [C++20] [Modules] Chose BMI from for module m with the last
-fmodule-file=<module-name>= option

Currently if we have multiple `-fmodule-file=<module-name>=<BMI-path>`
flags for the same `<module-name>`, we will pick the BMI-path from the
first flag. And this is inconsistent with what users generally expect.
e.g, we might expect the latter flags can override the former ones.

This patch changes the behavior to match user's expectation.




More information about the All-commits mailing list