[PATCH] D55490: Add dumpMethodDeclOverrides to NodeDumper

2018-12-21 Thread Stephen Kelly via Phabricator via cfe-commits
steveire abandoned this revision. steveire added a comment. Abandoning this because it's not really the right approach as it appears in the interface between the generic traverser and the generic node dumper, which it really shouldn't. Repository: rC Clang CHANGES SINCE LAST ACTION https:

[PATCH] D55490: Add dumpMethodDeclOverrides to NodeDumper

2018-12-09 Thread Aaron Ballman via Phabricator via cfe-commits
aaron.ballman accepted this revision. aaron.ballman added a comment. This revision is now accepted and ready to land. LGTM, aside from a minor nit. Comment at: lib/AST/ASTDumper.cpp:921 if (const CXXMethodDecl *MD = dyn_cast(D)) { +dumpMethodDeclOverrides(MD);

[PATCH] D55490: Add dumpMethodDeclOverrides to NodeDumper

2018-12-09 Thread Stephen Kelly via Phabricator via cfe-commits
steveire created this revision. steveire added a reviewer: aaron.ballman. Herald added a subscriber: cfe-commits. Repository: rC Clang https://reviews.llvm.org/D55490 Files: include/clang/AST/TextNodeDumper.h lib/AST/ASTDumper.cpp lib/AST/TextNodeDumper.cpp Index: lib/AST/TextNodeDumpe