steveire marked an inline comment as done.
steveire added inline comments.
================
Comment at: include/clang/AST/ASTDumperUtils.h:22
+
+namespace ast_dumper {
+// Colors used for various parts of the AST dump
----------------
aaron.ballman wrote:
> I'm not certain this namespace is useful, especially when it gets imported at
> TU scope in ASTDumper.cpp.
> it gets imported at TU scope in ASTDumper.cpp
Today that's the only place it is used. In the future that won't be true.
Repository:
rC Clang
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D55188/new/
https://reviews.llvm.org/D55188
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
