[PATCH] D62056: Use ASTDumper to dump the AST from clang-query

2019-12-06 Thread Stephen Kelly via Phabricator via cfe-commits
steveire closed this revision. steveire added a comment. Committed in b22d8ae7f436bfe63 . I don't know why this review didn't automatically update. Repository: rCTE Clang Tools Extra CHANGES SINCE LAST ACTION https://rev

[PATCH] D62056: Use ASTDumper to dump the AST from clang-query

2019-05-20 Thread Aaron Ballman via Phabricator via cfe-commits
aaron.ballman added a comment. If you wanted to expand test coverage for the changes covered in this, that would be a good thing. However, I'm not certain how testable the newly colorized output is, which is why this LG as-is. Repository: rCTE Clang Tools Extra CHANGES SINCE LAST ACTION h

[PATCH] D62056: Use ASTDumper to dump the AST from clang-query

2019-05-20 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 Repository: rCTE Clang Tools Extra CHANGES SINCE LAST ACTION https://reviews.llvm.org/D62056/new/ https://reviews.llvm.org/D62056 __

[PATCH] D62056: Use ASTDumper to dump the AST from clang-query

2019-05-17 Thread Stephen Kelly via Phabricator via cfe-commits
steveire created this revision. steveire added a reviewer: aaron.ballman. Herald added a project: clang. Herald added a subscriber: cfe-commits. This way, the output is not limited by the various API differences between the dump() member functions. For example, all dumps are now in color, while t