[PATCH] D55340: NFC: Move dump of individual Stmts to TextNodeDumper

Aaron Ballman via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Dec 6 06:31:59 PST 2018


aaron.ballman accepted this revision.
aaron.ballman added a comment.
This revision is now accepted and ready to land.

LGTM aside from some small nits.



================
Comment at: lib/AST/TextNodeDumper.cpp:685
+void TextNodeDumper::VisitObjCSelectorExpr(const ObjCSelectorExpr *Node) {
+
+  OS << " ";
----------------
Extra whitespace that wasn't there before.


================
Comment at: lib/AST/TextNodeDumper.cpp:691
+void TextNodeDumper::VisitObjCProtocolExpr(const ObjCProtocolExpr *Node) {
+
+  OS << ' ' << *Node->getProtocol();
----------------
Extra whitespace that wasn't there before.


Repository:
  rC Clang

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D55340/new/

https://reviews.llvm.org/D55340





More information about the cfe-commits mailing list