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.