aaron.ballman closed this revision.
aaron.ballman marked an inline comment as done.
aaron.ballman added a comment.
Thanks, Hal! Committed in r321223.
Comment at: lib/AST/ASTDumper.cpp:218
+: ASTDumper(OS, Traits, SM, ShowColors, LangOptions()) {}
+//ASTDumper(raw_os
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,
aaron.ballman added a comment.
P-p-p-power ping! :-)
https://reviews.llvm.org/D40448
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
aaron.ballman added a comment.
Ping
https://reviews.llvm.org/D40448
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
aaron.ballman updated this revision to Diff 126016.
aaron.ballman added a comment.
Ping. Added more context to the patch.
https://reviews.llvm.org/D40448
Files:
include/clang/AST/Type.h
lib/AST/ASTDumper.cpp
lib/AST/TypePrinter.cpp
lib/StaticAnalyzer/Checkers/PaddingChecker.cpp
test/C
aaron.ballman added a comment.
Ping.
https://reviews.llvm.org/D40448
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
aaron.ballman created this revision.
The AST dump functionality does not currently make use of a printing policy, so
when dumping C++ code, you will see `_Bool`, `struct Foo` and other C-isms in
the output. This patch adds a `PrintingPolicy` object to the AST dumper to get
slightly improved out
