vsapsai added a comment.

In D120874#3404420 <https://reviews.llvm.org/D120874#3404420>, @ChuanqiXu wrote:

> @vsapsai @dblaikie I want to make sure if this one would affect clang 
> modules. Or simply, would the name of clang modules contain `:`? (`:` is the 
> separator of C++20 modules partitions.)

Module names in .modulemap should be valid C identifiers, if I remember 
ModuleMapParser code correctly. You can have submodules that are separated by 
`.` (no such meaning for C++20 modules). So in .modulemap you cannot specify a 
module with `:`. I'm not sure how you can specify a module name when you do 
that from the command line, need to ask around about that.

Also I think some team internally has a custom machinery with 
`-fprebuilt-module-path`. I'm trying to find them, so they can make sure it 
works with their approach.


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

https://reviews.llvm.org/D120874

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to