ChuanqiXu added inline comments.
================ Comment at: clang/lib/Format/TokenAnnotator.cpp:3226 + // No space between module :. + if (Left.isOneOf(Keywords.kw_module, tok::kw_export, Keywords.kw_import) && + Right.is(TT_ModulePartitionColon)) ---------------- owenpan wrote: > You can remove `kw_export` as it must be followed by `import` or `module`, > based on how `TT_ModulePartitionColon` is set on Line 908. `export` could be followed by many other declarations. (Maybe the logic on line 908 is not right, I am not sure) CHANGES SINCE LAST ACTION https://reviews.llvm.org/D114151/new/ https://reviews.llvm.org/D114151 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits