krasimir added inline comments.
================
Comment at: include/clang/CrossTU/CrossTUDiagnostic.h:16
+namespace clang {
+ namespace diag {
+ enum {
----------------
LLVM Style uses no indent for namespaces. Reformat with `clang-format`.
================
Comment at: include/clang/CrossTU/CrossTranslationUnit.h:70
+ llvm::StringMap<std::unique_ptr<clang::ASTUnit>> FileASTUnitMap;
+ llvm::StringMap<clang::ASTUnit *> FunctionAstUnitMap;
+ llvm::StringMap<std::string> FunctionFileMap;
----------------
Maybe rename to `FunctionASTUnitMap` for consistency with the previous line?
https://reviews.llvm.org/D34512
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
