[PATCH] D40448: Add a printing policy for AST dumping

Hal Finkel via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Dec 20 13:54:04 PST 2017


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

In https://reviews.llvm.org/D40448#961398, @aaron.ballman wrote:

> P-p-p-power ping! :-)


LGTM



================
Comment at: lib/AST/ASTDumper.cpp:218
+        : ASTDumper(OS, Traits, SM, ShowColors, LangOptions()) {}
+    //ASTDumper(raw_ostream &OS, const CommandTraits *Traits,
+    //          const SourceManager *SM, const PrintingPolicy &PrintPolicy)
----------------
Remove commented-out code.


https://reviews.llvm.org/D40448





More information about the cfe-commits mailing list