Start by moving some utilities to it. It will eventually house dumping
of individual nodes (after indentation etc has already been accounted
for).
Details
Details
Diff Detail
Diff Detail
- Repository
- rC Clang
Event Timeline
Comment Actions
Basically looking good, modulo namespace questions from D55188 and a few other organizational questions.
include/clang/AST/ASTTextNodeDumper.h | ||
---|---|---|
1 ↗ | (On Diff #176322) | Perhaps the file should be named TextNodeDumper.h to match the class name? |
44 ↗ | (On Diff #176322) | Should these implementations live in the header file? I feel like they belong in a .cpp file instead. |
Comment Actions
LGTM!
lib/AST/ASTDumper.cpp | ||
---|---|---|
90 | Another nice cleanup for later would be to replace these call sites with the NodeDumper.foo() version. Same for dumpBareDeclRef() below. |
lib/AST/ASTDumper.cpp | ||
---|---|---|
90 | These methods are called from AttrDump.inc. They'll be removed when the commit to port the Attr dumping goes in. |
Another nice cleanup for later would be to replace these call sites with the NodeDumper.foo() version. Same for dumpBareDeclRef() below.