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

[Chuanqi Xu via Phabricator via cfe-commits]

ChuanqiXu added a comment.

In D120874#3406954 <https://reviews.llvm.org/D120874#3406954>, @vsapsai wrote:

> By the way, tried a module name with a colon and ModuleMapParser doesn't accept it
>
>   include/module.modulemap:1:12: error: skipping stray token
>   module Test:Colon {
>              ^
>   include/module.modulemap:1:13: error: expected '{' to start module 'Test'
>   module Test:Colon {
>               ^
>   include/module.modulemap:1:13: error: expected module declaration
>   include/module.modulemap:1:19: error: expected module declaration
>   module Test:Colon {
>                     ^
>   include/module.modulemap:2:1: error: expected module declaration
>   }
>   ^

So it looks like this wouldn't be conflict with clang modules, do I understand right?


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

https://reviews.llvm.org/D120874