Eugene.Zelenko added a comment.

I may be mistaken, but Clang source code didn't use llvm namespace by default. 
Insetad you should include clang/Basic/LLVM.h and use llvm:: for rest of things.



================
Comment at: clang-doc/tool/ClangDocMain.cpp:42
 
 namespace {
 
----------------
There is no need to enclose static definitions in anonymous namespace.


https://reviews.llvm.org/D43667



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

Reply via email to