[cfe-commits] [PATCH] Implement AST dumper for Decls

Philip Craig philipjcraig at gmail.com
Tue Nov 13 01:45:29 PST 2012


  This is an updated patch showing my current progress. I still have a few things to fix or cleanup, so it's not worth reviewing yet.

  I do have a question though. Having separate StmtDumper and DeclDumper classes hasn't been useful, so I have merged them into a single ASTDumper class. The implementation is still spread over 4 files though: ASTDumper.cpp/h, DeclDumper.cpp, and StmtDumper.cpp. Is this okay, or is it better to put them all in a single file, say Dump.cpp?

Hi alexfh, doug.gregor,

http://llvm-reviews.chandlerc.com/D52

CHANGE SINCE LAST DIFF
  http://llvm-reviews.chandlerc.com/D52?vs=151&id=307#toc

Files:
  include/clang/AST/CMakeLists.txt
  include/clang/AST/Makefile
  include/clang/AST/PrettyPrinter.h
  lib/AST/ASTDumper.cpp
  lib/AST/ASTDumper.h
  lib/AST/CMakeLists.txt
  lib/AST/DeclDumper.cpp
  lib/AST/DeclPrinter.cpp
  lib/AST/StmtDumper.cpp
  lib/AST/StmtPrinter.cpp
  lib/Frontend/ASTConsumers.cpp
  test/CXX/dcl.decl/dcl.init/dcl.init.ref/p5-examples.cpp
  test/CodeGen/bitfield-2.c
  test/Misc/ast-dump-attr.cpp
  test/Misc/ast-dump-decl.c
  test/Misc/ast-dump-decl.cpp
  test/Misc/ast-dump-decl.m
  test/Misc/ast-dump-decl.mm
  test/Misc/ast-dump-stmt.c
  test/Misc/ast-dump-stmt.m
  test/Misc/ast-dump-templates.cpp
  test/Misc/ast-dump-wchar.cpp
  test/PCH/objc_stmts.m
  test/SemaTemplate/default-expr-arguments-2.cpp
  test/Tooling/clang-check-ast-dump.cpp
  utils/TableGen/ClangAttrEmitter.cpp
  utils/TableGen/TableGen.cpp
  utils/TableGen/TableGenBackends.h
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D52.5.patch
Type: text/x-patch
Size: 99239 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20121113/420bf149/attachment.bin>


More information about the cfe-commits mailing list