ChuanqiXu added inline comments.
================ Comment at: clang/lib/Sema/SemaModule.cpp:177 + if (IsPartition) { + ModuleName += ":"; + ModuleName += stringFromPath(Partition); ---------------- I chose '-' in my implementation since here ModuleName refers to the name in filesystem, right? And I remember '-' is the choice of GCC. So let's keep consistency. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D118586/new/ https://reviews.llvm.org/D118586 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits