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

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

Reply via email to