[PATCH] D120874: [C++20] [Modules] Use '-' as the separator of partitions when searching in filesystems

[Volodymyr Sapsai via Phabricator via cfe-commits]

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