[PATCH] D118586: [C++20][Modules][3/8] Initial handling for module partitions.

Mon Feb 21 22:31:13 PST 2022

iains added a comment.

In D118586#3336612 <https://reviews.llvm.org/D118586#3336612>, @ChuanqiXu wrote:

> (May off the patch)

I am not sure what you mean here.

> BTW, I know you also work on GCC frontend. I want to consultant what's your opinion to uniform the command line options between Clang and GCC. Now it looks totally different. I feel it would be a problem for clang. Since the command line between clang and gcc are basically compatible so that user who decide to use clang could turn back to use gcc whenever he want. But I am worrying that it would be a harder decision for users if them are going to use clang. Since the cost to use GCC again would be higher.

We have the same objective: to keep user-facing options and functionality the same between clang and GCC (easier for both user and for build systems).
The content of this patch does not alter that. BTW GCC also keeps the module name as "Primary:Parition" internally and user-facing output.  //The module name in C++20 modules has no fixed mapping to the BMI/CMI filename on disk,.//

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D118586